Deleting unwanted manual setup instructions #3984
Conversation
| @@ -124,26 +124,9 @@ For this example, we are going to use [Prometheus](https://prometheus.io/). We a | |||
|
|
|||
| Prometheus is available for deployment in the Rancher v2.0 catalog. Deploy it from Rancher catalog if it isn't already running in your cluster. | |||
There was a problem hiding this comment.
I think we should refer to the installation of the rancher-monitoring chart instead and link to the Monitoring docs.
There was a problem hiding this comment.
@axeal , while I'm happy to make the changes I think it would be good creating another section for the rancher-monitoring chart because this section is explicitly detailing using third-party tools such as Prometheus.
There was a problem hiding this comment.
@axeal : Gentle reminder on the above, please could you let me know your response?
There was a problem hiding this comment.
@divya-mohan0209 This section (https://github.com/rancher/docs/blob/b4f4bfb8cdcfd38bbaad86e1838d00a8933dd52f/content/rancher/v2.x/en/k8s-in-rancher/horitzontal-pod-autoscaler/_index.md) pre-dates the Rancher v2.2 release (https://github.com/rancher/rancher/releases/v2.2.0) that introduced monitoring as a feature into Rancher v2. With monitoring, including prometheus and prometheus-adapter, available as a feature out-of-the-box, documentation on installing a custom prometheus adapter instance for a custom Prometheus deployment do not seem relevant anymore.
| @@ -121,26 +121,9 @@ For this example, we are going to use [Prometheus](https://prometheus.io/). We a | |||
|
|
|||
| Prometheus is available for deployment in the Rancher v2.0 catalog. Deploy it from Rancher catalog if it isn't already running in your cluster. | |||
There was a problem hiding this comment.
Same as above for v2.5: "I think we should refer to the installation of the rancher-monitoring chart instead and link to the Monitoring docs."
Fixes rancher/rancher-docs#62