Skip to content

Clarify when cleanup needs to be run#2954

Merged
ahardin-rh merged 1 commit intoopenshift:masterfrom
knobunc:fix/clarify-change-plugin
Sep 28, 2016
Merged

Clarify when cleanup needs to be run#2954
ahardin-rh merged 1 commit intoopenshift:masterfrom
knobunc:fix/clarify-change-plugin

Conversation

@knobunc
Copy link
Copy Markdown
Contributor

@knobunc knobunc commented Sep 28, 2016

There is no need to clean up several things when changing between
variants of the OpenShift SDN plugin.

@knobunc
Copy link
Copy Markdown
Contributor Author

knobunc commented Sep 28, 2016

@danwinship PTAL

@danwinship
Copy link
Copy Markdown
Contributor

I'd say something more like "if you are switching from an OpenShift plugin to a third-party plugin, then ..."

There is no need to clean up several things when changing between
variants of the OpenShift SDN plugin.
@knobunc knobunc force-pushed the fix/clarify-change-plugin branch from dd0ab66 to 92c1c8e Compare September 28, 2016 14:36
@knobunc
Copy link
Copy Markdown
Contributor Author

knobunc commented Sep 28, 2016

Thanks @danwinship . Changed.

@danwinship
Copy link
Copy Markdown
Contributor

lgtm

@pravisankar
Copy link
Copy Markdown

LGTM

@ahardin-rh ahardin-rh added this to the Next Release milestone Sep 28, 2016
@ahardin-rh ahardin-rh self-assigned this Sep 28, 2016
@ahardin-rh
Copy link
Copy Markdown
Contributor

Looks good to me. Thanks!

@ahardin-rh ahardin-rh merged commit 5e2d273 into openshift:master Sep 28, 2016
@ahardin-rh
Copy link
Copy Markdown
Contributor

ahardin-rh commented Sep 28, 2016

[rev_history]
|xref:../install_config/configuring_sdn.adoc#install-config-configuring-sdn[Configuring the SDN]
|Added clarifying details to the xref:../install_config/configuring_sdn.adoc#migrating-between-sdn-plugins[Migrating Between SDN Plug-ins] section about when to clean up SDN-specific artifacts.
%

@bfallonf bfallonf modified the milestones: Next Release, Staging Oct 4, 2016
@vikram-redhat vikram-redhat modified the milestones: Staging, OCP 3.3 GA Oct 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants