SRE-3829 ci: Remove EL8 support for jenkins and GHA#18537
Conversation
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
|
Errors are Unable to load ticket data |
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
osalyk
changed the title
daltonbohning
left a comment
daltonbohning
left a comment
There was a problem hiding this comment.
Are we sure nobody is using EL8 internally and installing DAOS RPMs? If so, maybe we need to make the team aware of this change.
|
Daily and weekly tests for master / release 2.8 may have VM testing stages for EL-8, and if so, they will need to be removed before this is landing. Using EL-8 for hardware CI will cause issues that I have not yet deployed a bug fix for that. CI has not tripped on that bug, but some of the reservable LAB systems have. |
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
| DISTRO_VERSION='${{ env.CP_EL9_VERSION && | ||
| env.CP_EL9_VERSION || env.EL9_VERSION }}' |
There was a problem hiding this comment.
I am not exactly familiar with this part of the code but what just happened with CP_EL9_TARGET?
| description: 'Continue testing if a previous stage is Unstable') | ||
| booleanParam(name: 'CI_UNIT_TEST', | ||
| defaultValue: true, | ||
| description: 'Run the Cancel Previous Builds stage.') |
There was a problem hiding this comment.
This description seems inaccurate.
| string(name: 'CI_UBUNTU20.04_TARGET', | ||
| defaultValue: '', | ||
| description: 'Image to used for Ubuntu 20 CI tests. I.e. ubuntu20.04, etc.') | ||
| booleanParam(name: bashName('Cancel Previous Builds'), |
There was a problem hiding this comment.
Why you are getting rid of this parameter?
| booleanParam(name: 'CI_FUNCTIONAL_TEST_SKIP', | ||
| defaultValue: false, | ||
| description: 'Skip all functional test stages (Test)') | ||
| booleanParam(name: 'CI_MORE_FUNCTIONAL_PR_TESTS', | ||
| defaultValue: false, | ||
| description: 'Enable more distros for functional CI tests') | ||
| booleanParam(name: 'CI_FUNCTIONAL_el9_TEST', |
There was a problem hiding this comment.
How these variables are related to this change? They look unfamiliar.
5 participants
Steps for the author:
After all prior steps are complete:
Required: