馃敟 Terminate MSS with extreme prejudice #1162
Conversation
9f60392
to
72bf055
Compare
Running an installation & an upgrade on RHPDS clusters now. I forgot to log the fresh install one to a file, so I might not be able to add all of the logs from it (assuming it succeeds). |
Both install & upgrade worked successfully (logs included in description). @mikenairn @maleck13 it probably makes more sense for one of you from RHMI to review this than someone from my team, wdyt? |
/lgtm It all looks fine to me, but i haven't tested it, are we happy that it's been tested enough? We will also need a PR to remove the DeLorean jobs for mss and mss-operator https://github.com/integr8ly/ci-cd/tree/master/jobs/delorean |
I've created the following PR there just now: integr8ly/ci-cd#356 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mikenairn The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
JIRA: https://issues.jboss.org/browse/AEROGEAR-10077
Not removing the CRDs as part of the upgrade, since if people have created CRs for those, then I'd imagine that could break upgrade on that cluster.
Verification Steps
As the verifier of the PR the following process should be done:
Installation Verification
Installation log: https://gist.github.com/grdryn/5e6d60287e5f902a1ae7e6265734230f
RHPDS cluster GUID:
freshmss-4732
There should be no mobile-security-service namespace
Upgrade Verification
Upgrade log: https://gist.github.com/grdryn/401e6e610c099acc7c0b97fb662cd133
RHPDS cluster GUID:
upgrdryn-af79
Upgrade should have remove mobile-security-service.
Is an upgrade task required and are there additional steps needed to test this?
Why is Yes/No multiple choice? 馃榾