Skip to content

Commit 5db81ec

Browse files
author
Claudia
committed
requested changes
Signed-off-by: Claudia <[email protected]>
1 parent b17059d commit 5db81ec

File tree

5 files changed

+15
-6
lines changed

5 files changed

+15
-6
lines changed

setup.RHOAI-v2.13/CLUSTER-SETUP.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ Helm charts values and how-to for customization can be found [in the official do
9696

9797
```bash
9898
helm repo add autopilot https://ibm.github.io/autopilot/
99+
helm repo update
99100
```
100101

101102
- Install the chart (idempotent command). The config file is for customizing the helm values and it is optional.
@@ -107,12 +108,13 @@ helm upgrade autopilot autopilot/autopilot --install --namespace=autopilot --cre
107108
### Enabling Prometheus metrics
108109

109110
After completing the installation, manually label the namespace to enable metrics to be scraped by Prometheus with the following command:
110-
The `ServiceMonitor` labeling is not required.
111111

112112
```bash
113113
oc label ns autopilot openshift.io/cluster-monitoring=true
114114
```
115115

116+
The `ServiceMonitor` labeling is not required.
117+
116118
## Kueue Configuration
117119

118120
Create Kueue's default flavor:

setup.RHOAI-v2.16/CLUSTER-SETUP.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ Helm charts values and how-to for customization can be found [in the official do
8484

8585
```bash
8686
helm repo add autopilot https://ibm.github.io/autopilot/
87+
helm repo update
8788
```
8889

8990
- Install the chart (idempotent command). The config file is for customizing the helm values and it is optional.
@@ -95,12 +96,13 @@ helm upgrade autopilot autopilot/autopilot --install --namespace=autopilot --cre
9596
### Enabling Prometheus metrics
9697

9798
After completing the installation, manually label the namespace to enable metrics to be scraped by Prometheus with the following command:
98-
The `ServiceMonitor` labeling is not required.
9999

100100
```bash
101101
oc label ns autopilot openshift.io/cluster-monitoring=true
102102
```
103103

104+
The `ServiceMonitor` labeling is not required.
105+
104106
## Kueue Configuration
105107

106108
Create Kueue's default flavor:

setup.RHOAI-v2.17/CLUSTER-SETUP.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ Helm charts values and how-to for customization can be found [in the official do
8484

8585
```bash
8686
helm repo add autopilot https://ibm.github.io/autopilot/
87+
helm repo update
8788
```
8889

8990
- Install the chart (idempotent command). The config file is for customizing the helm values and it is optional.
@@ -95,12 +96,13 @@ helm upgrade autopilot autopilot/autopilot --install --namespace=autopilot --cre
9596
### Enabling Prometheus metrics
9697

9798
After completing the installation, manually label the namespace to enable metrics to be scraped by Prometheus with the following command:
98-
The `ServiceMonitor` labeling is not required.
9999

100100
```bash
101101
oc label ns autopilot openshift.io/cluster-monitoring=true
102102
```
103103

104+
The `ServiceMonitor` labeling is not required.
105+
104106
## Kueue Configuration
105107

106108
Create Kueue's default flavor:

setup.k8s/CLUSTER-SETUP.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@ Helm charts values and how-to for customization can be found [in the official do
8282

8383
```bash
8484
helm repo add autopilot https://ibm.github.io/autopilot/
85+
helm repo update
8586
```
8687

8788
- Install the chart (idempotent command). The config file is for customizing the helm values and it is optional.
@@ -98,7 +99,7 @@ If that is not the case in your cluster, the correct release label can be found
9899
Then, Autopilot's `ServiceMonitor` can be labeled with the following command
99100

100101
```bash
101-
kubectl label servicemonitors.monitoring.coreos.com -n autopilot autopilot-metrics-monitor release=<prometheus-release-name>
102+
kubectl label servicemonitors.monitoring.coreos.com -n autopilot autopilot-metrics-monitor release=<prometheus-release-name> --overwrite
102103
```
103104

104105
## Kueue Configuration

setup.tmpl/CLUSTER-SETUP.md.tmpl

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,6 +163,7 @@ Helm charts values and how-to for customization can be found [in the official do
163163

164164
```bash
165165
helm repo add autopilot https://ibm.github.io/autopilot/
166+
helm repo update
166167
```
167168

168169
- Install the chart (idempotent command). The config file is for customizing the helm values and it is optional.
@@ -175,19 +176,20 @@ helm upgrade autopilot autopilot/autopilot --install --namespace=autopilot --cre
175176

176177
{{ if .OPENSHIFT -}}
177178
After completing the installation, manually label the namespace to enable metrics to be scraped by Prometheus with the following command:
178-
The `ServiceMonitor` labeling is not required.
179179

180180
```bash
181181
{{ .KUBECTL }} label ns autopilot openshift.io/cluster-monitoring=true
182182
```
183+
184+
The `ServiceMonitor` labeling is not required.
183185
{{- else -}}
184186
The `ServiceMonitor` object is the one that enables Prometheus to scrape the metrics produced by Autopilot.
185187
In order for Prometheus to find the right objects, the `ServiceMonitor` needs to be annotated with the Prometheus' release name. It is usually `prometheus`, and that's the default added in the Autopilot release.
186188
If that is not the case in your cluster, the correct release label can be found by checking in the `ServiceMonitor` of Prometheus itself, or the name of Prometheus helm chart.
187189
Then, Autopilot's `ServiceMonitor` can be labeled with the following command
188190

189191
```bash
190-
{{ .KUBECTL }} label servicemonitors.monitoring.coreos.com -n autopilot autopilot-metrics-monitor release=<prometheus-release-name>
192+
{{ .KUBECTL }} label servicemonitors.monitoring.coreos.com -n autopilot autopilot-metrics-monitor release=<prometheus-release-name> --overwrite
191193
```
192194
{{- end }}
193195

0 commit comments

Comments
 (0)