Skip to content

Commit e7a28b0

Browse files
Update Sample YAML in prometheus-remote-write-managed-identity.md
The sample YAML in the `Deploy Side car and configure remote write on the Prometheus server` section appears to be wrong. `livenessProbe` should contain `httpGet`, `initialDelaySeconds`, and `timeoutSeconds`. Presently, the latter two are values under `httpGet`, which makes this YAML invalid. This exact same issue occurs in the `readinessProbe` value below, too. The Prometheus spec can be seen here laying out what I am claiming here: https://docs.okd.io/latest/rest_api/monitoring_apis/prometheus-monitoring-coreos-com-v1.html#spec-containers-livenessprobe https://docs.okd.io/latest/rest_api/monitoring_apis/prometheus-monitoring-coreos-com-v1.html#spec-containers-readinessprobe
1 parent d8d2f96 commit e7a28b0

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

articles/azure-monitor/essentials/prometheus-remote-write-managed-identity.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,14 +105,14 @@ This step isn't required if you're using an AKS identity since it will already h
105105
httpGet:
106106
path: /health
107107
port: rw-port
108-
initialDelaySeconds: 10
109-
timeoutSeconds: 10
108+
initialDelaySeconds: 10
109+
timeoutSeconds: 10
110110
readinessProbe:
111111
httpGet:
112112
path: /ready
113113
port: rw-port
114-
initialDelaySeconds: 10
115-
timeoutSeconds: 10
114+
initialDelaySeconds: 10
115+
timeoutSeconds: 10
116116
env:
117117
- name: INGESTION_URL
118118
value: <INGESTION_URL>

0 commit comments

Comments
 (0)