Skip to content

Commit 4219449

Browse files
committed
Add kuttl tests for prometheus configuration
This patch adds some required prometheus secret for kuttl jobs to run and adds validation checks. Also, it modifies the watcher_deploy make target to create a secret so that the watcher can be deployed until the secret is created by the telemetry operator.
1 parent b61921c commit 4219449

File tree

9 files changed

+60
-0
lines changed

9 files changed

+60
-0
lines changed

Makefile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -385,11 +385,13 @@ watcher: ## Install watcher operator via olm
385385

386386
.PHONY: watcher_deploy
387387
watcher_deploy: ## Deploy watcher service
388+
oc apply -f config/samples/watcher_requirements.yaml
388389
oc apply -f config/samples/watcher_v1beta1_watcher.yaml
389390

390391
.PHONY: watcher_deploy_cleanup
391392
watcher_deploy_cleanup: ## Undeploy watcher service
392393
oc delete -f config/samples/watcher_v1beta1_watcher.yaml
394+
oc delete -f config/samples/watcher_requirements.yaml
393395

394396
.PHONY: watcher_cleanup
395397
watcher_cleanup: export CATALOG_IMG=${CATALOG_IMAGE}
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
apiVersion: v1
2+
kind: Secret
3+
metadata:
4+
name: metric-storage-prometheus-config
5+
namespace: openstack
6+
stringData:
7+
host: metric-storage-prometheus.openstack.svc
8+
port: "9090"
9+
prometheus_ca.crt: ""

tests/kuttl/test-suites/default/common/cleanup-errors.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,3 +96,13 @@ apiVersion: keystone.openstack.org/v1beta1
9696
kind: KeystoneEndpoint
9797
metadata:
9898
name: watcher
99+
---
100+
apiVersion: v1
101+
kind: Secret
102+
metadata:
103+
name: metric-storage-prometheus-config
104+
---
105+
apiVersion: v1
106+
kind: Secret
107+
metadata:
108+
name: custom-prometheus-config

tests/kuttl/test-suites/default/common/cleanup-watcher.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,9 @@ delete:
44
- apiVersion: watcher.openstack.org/v1beta1
55
kind: Watcher
66
name: watcher-kuttl
7+
- apiVersion: v1
8+
kind: Secret
9+
name: metric-storage-prometheus-config
10+
- apiVersion: v1
11+
kind: Secret
12+
name: custom-prometheus-config

tests/kuttl/test-suites/default/common/deploy-with-defaults.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
apiVersion: v1
2+
kind: Secret
3+
metadata:
4+
name: metric-storage-prometheus-config
5+
namespace: watcher-kuttl-default
6+
stringData:
7+
host: metric-storage-prometheus.watcher-kuttl-default.svc
8+
port: "9090"
9+
prometheus_ca.crt: ""
10+
---
111
apiVersion: watcher.openstack.org/v1beta1
212
kind: Watcher
313
metadata:

tests/kuttl/test-suites/default/watcher-api-service-override/01-deploy-with-defaults.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
apiVersion: v1
2+
kind: Secret
3+
metadata:
4+
name: metric-storage-prometheus-config
5+
namespace: watcher-kuttl-default
6+
stringData:
7+
host: metric-storage-prometheus.watcher-kuttl-default.svc
8+
port: "9090"
9+
prometheus_ca.crt: ""
10+
---
111
apiVersion: watcher.openstack.org/v1beta1
212
kind: Watcher
313
metadata:

tests/kuttl/test-suites/default/watcher/01-assert.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ spec:
2020
apiServiceTemplate:
2121
replicas: 1
2222
resources: {}
23+
prometheusSecret: metric-storage-prometheus-config
2324
status:
2425
apiServiceReadyCount: 1
2526
conditions:

tests/kuttl/test-suites/default/watcher/04-assert.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -243,6 +243,7 @@ commands:
243243
if [ -n "${APIPOD}" ]; then
244244
[ $(echo $(oc rsh -c watcher-api ${APIPOD} cat /etc/watcher/watcher.conf.d/01-global-custom.conf) |grep -c "^# Global config") == 1 ]
245245
[ $(echo $(oc rsh -c watcher-api ${APIPOD} cat /etc/watcher/watcher.conf.d/02-service-custom.conf) |grep -c "^# Service config") == 1 ]
246+
[ $(echo $(oc rsh -c watcher-api ${APIPOD} cat /etc/watcher/watcher.conf.d/00-default.conf) |grep -czPo '\[prometheus_client\]\s+host\s+=\s+metric-storage-prometheus.watcher-kuttl-default.svc\s+port\s+=\s+9090') == 1 ]
246247
else
247248
exit 1
248249
fi

tests/kuttl/test-suites/default/watcher/04-deploy-with-precreated-account.yaml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
apiVersion: v1
2+
kind: Secret
3+
metadata:
4+
name: custom-prometheus-config
5+
namespace: watcher-kuttl-default
6+
stringData:
7+
host: metric-storage-prometheus.watcher-kuttl-default.svc
8+
port: "9090"
9+
prometheus_ca.crt: ""
10+
---
111
apiVersion: watcher.openstack.org/v1beta1
212
kind: Watcher
313
metadata:
@@ -6,6 +16,7 @@ metadata:
616
spec:
717
databaseInstance: "openstack"
818
databaseAccount: watcher-precreated
19+
prometheusSecret: custom-prometheus-config
920
tls:
1021
caBundleSecretName: "combined-ca-bundle"
1122
customServiceConfig: |

0 commit comments

Comments
 (0)