Skip to content
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

Ask for confirmation before disabling OpenShift CVO #222

Merged
merged 1 commit into from
Mar 4, 2020

Conversation

skitt
Copy link
Member

@skitt skitt commented Mar 2, 2020

When subctl join is run against an OpenShift cluster with a deployed
cluster version operator, and service discovery is enabled, we now ask
for confirmation before continuing, because installing service
discovery will disable the CVO.

Fixes: #167
Signed-off-by: Stephen Kitt skitt@redhat.com

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr222/skitt/confirm-cvo-disabling

@skitt skitt force-pushed the confirm-cvo-disabling branch from 432b948 to fc7091b Compare March 2, 2020 15:24
@submariner-bot
Copy link
Contributor

🤖 Updated branch: z_pr222/skitt/confirm-cvo-disabling

@skitt skitt force-pushed the confirm-cvo-disabling branch from fc7091b to a0ae575 Compare March 2, 2020 15:33
@submariner-bot
Copy link
Contributor

🤖 Updated branch: z_pr222/skitt/confirm-cvo-disabling

When subctl join is run against an OpenShift cluster with a deployed
cluster version operator, and service discovery is enabled, we now ask
for confirmation before continuing, because installing service
discovery will disable the CVO.

Fixes: submariner-io#167
Signed-off-by: Stephen Kitt <skitt@redhat.com>
@skitt skitt force-pushed the confirm-cvo-disabling branch from a0ae575 to f3997bb Compare March 2, 2020 15:38
@submariner-bot
Copy link
Contributor

🤖 Updated branch: z_pr222/skitt/confirm-cvo-disabling

@skitt skitt added this to the v0.1.1 milestone Mar 3, 2020
@mangelajo mangelajo linked an issue Mar 4, 2020 that may be closed by this pull request
@skitt skitt merged commit 9b6b450 into submariner-io:release-0.1 Mar 4, 2020
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr222/skitt/confirm-cvo-disabling]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

subctl should ask for confirmation before degrading OCP clusters
4 participants