Skip to content

Commit

Permalink
OVN: Fix control plane metrics alert job name
Browse files Browse the repository at this point in the history
Following the change in PR openshift#581 the job name should be renamed to
"ovnkube-master-metrics".

Signed-off-by: Surya Seetharaman <suryaseetharaman.9@gmail.com>
  • Loading branch information
tssurya committed Aug 4, 2020
1 parent 13a7db3 commit a9ea052
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 9 deletions.
4 changes: 2 additions & 2 deletions bindata/network/ovn-kubernetes/alert-rules-control-plane.yaml
Expand Up @@ -15,9 +15,9 @@ spec:
- alert: NoRunningOvnMaster
annotations:
message: |
there is no running ovn-kubernetes master
There is no running ovn-kubernetes master
expr: |
absent(up{job="ovnkube-master",namespace="openshift-ovn-kubernetes"} ==1)
absent(up{job="ovnkube-master-metrics", namespace="openshift-ovn-kubernetes"} == 1)
for: 10m
labels:
severity: warning
Expand Down
10 changes: 5 additions & 5 deletions bindata/network/ovn-kubernetes/monitor.yaml
Expand Up @@ -3,7 +3,7 @@ apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
labels:
app: ovnkube-master-metrics
metrics: ovnkube-master-metrics
annotations:
networkoperator.openshift.io/ignore-errors: ""
name: monitor-ovn-master-metrics
Expand All @@ -17,26 +17,26 @@ spec:
tlsConfig:
caFile: /etc/prometheus/configmaps/serving-certs-ca-bundle/service-ca.crt
serverName: ovn-kubernetes-master-metrics.openshift-ovn-kubernetes.svc
jobLabel: app
jobLabel: metrics
namespaceSelector:
matchNames:
- openshift-ovn-kubernetes
selector:
matchLabels:
app: ovnkube-master-metrics
metrics: ovnkube-master-metrics
---
apiVersion: v1
kind: Service
metadata:
labels:
app: ovnkube-master-metrics
metrics: ovnkube-master-metrics
name: ovn-kubernetes-master-metrics
namespace: openshift-ovn-kubernetes
annotations:
service.beta.openshift.io/serving-cert-secret-name: ovn-master-metrics-cert
spec:
selector:
app: ovnkube-master-metrics
metrics: ovnkube-master-metrics
clusterIP: None
publishNotReadyAddresses: true
ports:
Expand Down
6 changes: 4 additions & 2 deletions bindata/network/ovn-kubernetes/ovnkube-master.yaml
Expand Up @@ -27,6 +27,7 @@ spec:
selector:
matchLabels:
app: ovnkube-master
metrics: ovnkube-master-metrics
strategy:
type: RollingUpdate
rollingUpdate:
Expand All @@ -38,6 +39,7 @@ spec:
metadata:
labels:
app: ovnkube-master
metrics: ovnkube-master-metrics
component: network
type: infra
openshift.io/component: network
Expand Down Expand Up @@ -619,13 +621,13 @@ metadata:
spec:
selector:
matchLabels:
app: ovnkube-master-metrics
metrics: ovnkube-master-metrics
strategy:
type: RollingUpdate
template:
metadata:
labels:
app: ovnkube-master-metrics
metrics: ovnkube-master-metrics
component: network
type: infra
openshift.io/component: network
Expand Down

0 comments on commit a9ea052

Please sign in to comment.