search

flux-framework / flux-sched

Fluxion Graph-based Scheduler
GNU Lesser General Public License v3.0
90 stars 41 forks source link

matrix: add explicit python path for el8 #1256

Closed trws closed 4 months ago

trws commented 4 months ago

This one is slightly urgent. None of our CI runs will pass on EL8 until this is dealt with, fallout from rebuilding the fluxrm/testenv image on el8 is it picked up a newer valgrind, which changed the python environment on the system in a way that interacts badly with the older cmake in there.

problem: upstream el8 has added a new version of valgrind, which pulls in python3.11. The older cmake logic on el8 finds the newer python but fails to find any development dependencies for it and errors out.

solution: force cmake to use the system python3.6, it must be an absolute path as well for some reason

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.4%. Comparing base (5b6058f) to head (85c40c8). Report is 129 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1256 +/- ## ====================================== Coverage 74.4% 74.4% ====================================== Files 104 104 Lines 14940 14940 ====================================== Hits 11121 11121 Misses 3819 3819 ```