System tests: Update reference versions and results, upgrade default to Ubuntu 24.04#619
System tests: Update reference versions and results, upgrade default to Ubuntu 24.04#619
Conversation
|
@valentin-seitz please remind me: how did this work before, but not now? It seems like the What do I forget? |
|
Related job that shows the Python 3.10 issue: https://github.com/precice/tutorials/actions/runs/13421240183/job/37494139990 I see that there is also Python 3.11 installed, which should be enough. I tried enforcing that in 62fefe5, but then I started getting random tutorials failing without output. I reverted the change (which I don't seem to have needed ever before), and I removed all Docker cache from the VM, and this job output is the result. I am tempted to upgrade the VM, but I would like first to understand why this only appears now. |
|
I see that the VM is set to not allow release upgrades (I guess because it should be externally managed), so I see two ways forwards:
I still don't understand how this worked last time, though. |
|
I changed the implementation of the script to be compatible with Python 3.10, and it worked. Job output: https://github.com/precice/tutorials/actions/runs/13435728598/job/37537442258 |
|
@MakisH This is one of the things that makes me think of the 6 stages of debugging. (where this would be the: why did it ever work stage). The VM has python 3.11 installed, and I remember that I implemented the hashing function in 3.11 because it was a bit more neat, and it did work at some point in time. But you solved it for now i guess? |
Could it be that some virtual environment was enabled, which was reset after a restart? I see that the uptime of the VM is only 9 days at the moment. In the bash history, I do see a venv enabled for certbot at some point (in a directory that does not exist anymore), as well as the package edit: I have also updated packages (a usual
I would still like to understand the root cause, but yes, the specific problem has been worked around for now. Do I correctly interpret your comment as a PR approval? |
In the context of precice/precice#2052 (comment)
Also upgrading to Ubuntu 24.04 as a base image because (1) we eventually need to do it, and (2) the Python script that updates the reference results seems to require Python 3.11. I am not sure how this worked last time.
Checklist:
changelog-entries/<PRnumber>.md. -> N/A