search

flux-framework / flux-pmix

flux shell plugin to bootstrap openmpi v5+
GNU Lesser General Public License v3.0
2 stars 4 forks source link

ci: remove el7 build #88

Closed grondo closed 1 year ago

grondo commented 1 year ago

Problem: The el7 flux-core docker image will no longer be updated because it is being dropped from the matrix of test images in CI.

Drop the el7 build in flux-pmix CI as well. Move chain_lint=t to another builder so that at least one build continues to run with --chain-lint.

grondo commented 1 year ago

I've already updated branch protections to remove the el7 build as required. (though actually looks like a few other builds are missing as required) I'll try to update those.

codecov[bot] commented 1 year ago

Codecov Report

Merging #88 (79918d8) into main (b22804c) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #88   +/-   ##
=======================================
  Coverage   79.01%   79.01%           
=======================================
  Files          12       12           
  Lines        1382     1382           
=======================================
  Hits         1092     1092           
  Misses        290      290

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

grondo commented 1 year ago

Thanks! I'll set MWP.