[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 |
|
If you installed uv using curl, it is as easy as runing What are the consequences you are afraid of? The only breaking change I can imagine is failing the upgrade or failing to build the environment after upgrade. Both can be tested in advance on different platforms if need be. The only possible change to the runtime environment of the weathergenerator itself is a potentially higher version of numcodes. (@tjhunter is this critical or not? might be easier to revert the version constraint to |
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