Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/master' into benc-remove-channels
Browse files Browse the repository at this point in the history
 Conflicts:
	docs/userguide/plugins.rst
	parsl/channels/base.py
	parsl/channels/local/local.py
	parsl/tests/test_providers/test_pbspro_template.py
	parsl/tests/test_providers/test_slurm_template.py
  • Loading branch information
benclifford committed Nov 7, 2024
2 parents a3fdfaa + b092195 commit 5d3b61e
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
5 changes: 3 additions & 2 deletions docs/userguide/plugins.rst
Original file line number Diff line number Diff line change
Expand Up @@ -101,5 +101,6 @@ interpreted as a Future.
Removed interfaces
------------------

Parsl had a ``Channel`` interface which was removed in 2024. See
`issue 3515 <https://github.com/Parsl/parsl/issues/3515>`_ for further discussion.
Parsl had a deprecated ``Channel`` abstraction. See
`issue 3515 <https://github.com/Parsl/parsl/issues/3515>`_
for further discussion on its removal.
1 change: 1 addition & 0 deletions parsl/tests/test_providers/test_pbspro_template.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ def test_submit_script_basic(tmp_path):
queue="debug"
)
provider.script_dir = tmp_path
provider.channel.script_dir = tmp_path
job_id = str(random.randint(55000, 59000))
provider.execute_wait = mock.Mock(spec=PBSProProvider.execute_wait)
provider.execute_wait.return_value = (0, job_id, "")
Expand Down
1 change: 1 addition & 0 deletions parsl/tests/test_providers/test_slurm_template.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ def test_submit_script_basic(tmp_path):
partition="debug"
)
provider.script_dir = tmp_path
provider.channel.script_dir = tmp_path
job_id = str(random.randint(55000, 59000))
provider.execute_wait = mock.MagicMock(spec=SlurmProvider.execute_wait)
provider.execute_wait.return_value = (0, f"Submitted batch job {job_id}", "")
Expand Down

0 comments on commit 5d3b61e

Please sign in to comment.