-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CASMTRIAGE-7294 upgrade cert-manager in prerequisites.sh if cray-drydock is not on the correct version #5379
Conversation
…ock is not on the correct version
cb434a3
to
6d15a5c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I tested the latest version of this block on gamora and observed that it worked properly. Here is a section of the output when run with 'set -x'
|
In early CSM 1.6 versions, helm is an earlier version and does not have the
|
c0c455b
to
db38ff7
Compare
Description
In CSM 1.6, the latest cray-drydock version is 2.18.4. In that version, cray-drydock creates the cert-manager namespace. If the system has already been upgraded to an early version on CSM 1.6, then cert-manager has likely been upgraded and cray-drydock has not. The cray-drydock chart fails to upgrade when cert-manager is installed. This PR adds logic in the case of a CSM 1.6 to CSM 1.6 upgrade. It allows cray-drydock to be upgraded by uninstalling cert-manager, then upgrading drydock, then reinstalling cert-manager. Again, this is only used on a CSM 1.6 to CSM 1.6 upgrade.
Checklist
.github/CODEOWNERS
with the corresponding team in Cray-HPE.