transition bionic containers to jammy #1057
Conversation
trws
force-pushed
the
jammy
branch
10 times, most recently
from
ea0b610
to
de52f4c
Compare
There was a problem hiding this comment.
LGTM, but honestly I'd consider just squashing all these commits into one big commit with a nice descriptive commit message saying something like ci: replace bionic with jammy and bookworm with a nice explanation in the message "bionic is out of LTS, etc" so update to jammy and add a bookworm image since flux-core has one, etc.
If you want to keep a couple separate commits that's fine too, but they'd need commit message to give slightly more detail.
And THANK YOU for doing this!
|
Thanks @grondo. Most of these are squashed out, all the container oriented commits are. There's a larger commit message, and updated to only build one jammy output along with using bookworm for most tests. |
This is an image update/rework for our CI. Since bionic has fallen out of LTS standard support, the original plan was to update to Jammy and move on. There's more here than that because Jammy doesn't provide a 386 container build anymore. To fill in that gap, we also have a Debian bookworm container, which is now the default target. There is also a Jammy build in here to keep an ubuntu LTS current container going, but the Ubuntu side is painful enough that most of the tests are now on bookworm. Additionally: * el7 is dropped * fedora outside of 34 (forked to el9) and current are dropped * docker-checks-run script updated from matching PR in core
trws
added
the
merge-when-passing
Codecov Report
@@ Coverage Diff @@
## master #1057 +/- ##
========================================
- Coverage 74.4% 72.6% -1.9%
========================================
Files 86 80 -6
Lines 9434 10243 +809
========================================
+ Hits 7020 7437 +417
- Misses 2414 2806 +392 |
transition bionic containers to jammy
This may end up having to merge before the tests on flux-core's version can pass, because of the flux-sched test run. Hoping to get a jump on it here, somehow need to break the cycle.