Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pin radical.utils version, to work with pinned radical.pilot #3431

Merged
merged 1 commit into from
May 16, 2024

Conversation

benclifford
Copy link
Collaborator

On 10th May 2024, a new version of radical.utils was released, which does not work with the pinned radical.pilot==1.52.1 used by Parsl.

I also tried upgrading radical.pilot to 1.60, released alongside the new radical.utils, but that does not work for other reasons documented in the below issue.

See Parsl issue #3429

cc @AymenFJA @andre-merzky

Changed Behaviour

radical.utils will no longer follow the latest (or at least, the latest under whatever radical.pilot constrains things to) - that's a good thing from this PR's perspective.

Fixes

Fixes #3429

Type of change

  • Bug fix

On 10th May 2024, a new version of radical.utils was released, which does not
work with the pinned radical.pilot==1.52.1 used by Parsl.

I also tried upgrading radical.pilot to 1.60, released alongside the new
radical.utils, but that does not work for other reasons documented in the
below issue.

See Parsl issue #3429
Copy link
Collaborator

@khk-globus khk-globus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦

Copy link
Collaborator

@khk-globus khk-globus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird, I thought I approved it. Alas. Clicky clicky.

@benclifford benclifford merged commit 1bf6c22 into master May 16, 2024
6 checks passed
@benclifford benclifford deleted the benc-radical-pin-utils branch May 16, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Radical Pilot test failure in CI
2 participants