Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

ref(docs): update uninstall guide #3003

Merged
merged 1 commit into from
Apr 1, 2021

Conversation

michelleN
Copy link
Contributor

@michelleN michelleN commented Mar 25, 2021

Signed-off-by: Michelle Noorali minooral@microsoft.com

Please mark with X for applicable areas.

-->
Affected area:

  • New Functionality [ ]
  • Documentation [ X]
  • Install [ ]
  • Control Plane [ ]
  • CLI Tool [ ]
  • Certificate Management [ ]
  • Networking [ ]
  • Metrics [ ]
  • SMI Policy [ ]
  • Security [ ]
  • Tests [ ]
  • CI System [ ]
  • Demo [ ]
  • Performance [ ]
  • Other [ ]

Please answer the following questions with yes/no.

  • Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution? No

@michelleN michelleN force-pushed the uninstall branch 2 times, most recently from 7defd42 to 051c322 Compare March 26, 2021 16:07
@michelleN michelleN marked this pull request as ready for review March 26, 2021 16:07
@michelleN michelleN requested a review from a team as a code owner March 26, 2021 16:07
Copy link
Contributor

@nojnhuh nojnhuh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM aside from a couple nits.

@@ -8,44 +8,136 @@ weight: 3

# Uninstallation Guide

This guide describes how to uninstall Open Service Mesh (OSM) from a Kubernetes cluster using the `osm` CLI.
This guide describes how to uninstall Open Service Mesh (OSM) from a Kubernetes cluster. This guide assumes there is a single OSM control plane (mesh) running. If there are multiple meshes in a cluster, repeat the process described for each control plane in the cluster before uninstalling any cluster wide resources at the end of the guide. Taking into consideration both the control plane and dataplane, this guide aims to walk through uninstalling all reminents of OSM with minimal downtime.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This guide describes how to uninstall Open Service Mesh (OSM) from a Kubernetes cluster. This guide assumes there is a single OSM control plane (mesh) running. If there are multiple meshes in a cluster, repeat the process described for each control plane in the cluster before uninstalling any cluster wide resources at the end of the guide. Taking into consideration both the control plane and dataplane, this guide aims to walk through uninstalling all reminents of OSM with minimal downtime.
This guide describes how to uninstall Open Service Mesh (OSM) from a Kubernetes cluster. This guide assumes there is a single OSM control plane (mesh) running. If there are multiple meshes in a cluster, repeat the process described for each control plane in the cluster before uninstalling any cluster wide resources at the end of the guide. Taking into consideration both the control plane and dataplane, this guide aims to walk through uninstalling all remnants of OSM with minimal downtime.

once part of the mesh. Identify these resources by using the `kubectl get ingress -n <namespace>` command in each namespace that was
removed from the mesh earlier.

If there are ingress resources and they are configured to allow HTTPS traffic, SSL related annotations will need to be update appropriately.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
If there are ingress resources and they are configured to allow HTTPS traffic, SSL related annotations will need to be update appropriately.
If there are ingress resources and they are configured to allow HTTPS traffic, SSL related annotations will need to be updated appropriately.


If there are ingress resources and they are configured to allow HTTPS traffic, SSL related annotations will need to be update appropriately.

If ingress resources need to be reconfigured, this may result in some down time of traffic from outside the cluster not being able to get to the applications.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might be a bit of a cleaner way to say the same thing:

Suggested change
If ingress resources need to be reconfigured, this may result in some down time of traffic from outside the cluster not being able to get to the applications.
Applications may be unavailable from outside the cluster if ingress resources need to be reconfigured.


## Resource Management
Once the sidecar is removed, there is no need for the Envoy bootstrap config secrets OSM created. These are stored in the application namespace and can be deleted manually with `kubectl`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to give a hint here as to what those secrets look like, like a label, annotation, or example name.


```
Error: rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "osm" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-namespace" must equal "osm-system2": current value is "osm-system"
As a result, one may see this error during a subsequent install of a new mesh with the same name but different namespace:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this may fit better in the install troubleshooting doc since it's an error that occurs at install time.

@michelleN
Copy link
Contributor Author

Thanks for the feedback @nojnhuh. Please re-review when you have a few minutes.

Copy link
Contributor

@SanyaKochhar SanyaKochhar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, left comments for a couple nits

docs/content/docs/install/uninstallation_guide.md Outdated Show resolved Hide resolved
docs/content/docs/install/uninstallation_guide.md Outdated Show resolved Hide resolved
docs/content/docs/install/uninstallation_guide.md Outdated Show resolved Hide resolved
+ update uninstall guide
+ update uninstall troubleshooting guide
+ resolves openservicemesh#2880

Signed-off-by: Michelle Noorali <minooral@microsoft.com>
@michelleN michelleN merged commit d5bf349 into openservicemesh:main Apr 1, 2021
@michelleN michelleN deleted the uninstall branch April 1, 2021 17:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doc: uninstalling OSM and removing all related components from a cluster
4 participants