[2298] Fix CI#2299
Conversation
grassesi
left a comment
There was a problem hiding this comment.
Tested the modified commands (sync-safe, lint) on Juptir: works fine. Note for people on Jupiter or juwels-booster who are using the uv package supplied by the module stack need to switch to standalone uv installation since the module stack only contains uv 0.7.4/0.8.17 . Alternatively we need to update the module stack.
sophie-xhonneux
left a comment
There was a problem hiding this comment.
This is a breaking change, let's wait until after June 10th when we present results.
What will be broken by this change? |
|
From my understanding, it will force everyone to change their uv installation or be blocked and with insufficiently tested consequences. It is simply not worth doing this before June 10th |
Description
This PR should fix the CI issue. It makes several changes:
sync-safecommand that creates a venv in a more robust way. that should make sure that SCRATCH cache is touched and refreshed enough that it does not get cleanedIssue Number
Closes #2298
Checklist before asking for review
./scripts/actions.sh lint./scripts/actions.sh unit-test./scripts/actions.sh integration-testlaunch-slurm.py --time 60