Skip to content

Commit 5edfad3

Browse files
committed
fix: Use of __keyserver_name needs metrics.keyserver role
Tests/checks using __keyserver_name need metrics.keyserver role to be included to ensure the variable is defined.
1 parent eaa56aa commit 5edfad3

12 files changed

+12
-0
lines changed

tests/tests_verify_auth.yml

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
# yamllint disable rule:line-length
1010
roles:
11+
- role: performancecopilot.metrics.keyserver
1112
- role: performancecopilot.metrics.pcp
1213
vars:
1314
pcp_accounts:

tests/tests_verify_basic.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.pcp
89

910
pre_tasks:

tests/tests_verify_bpftrace.yml

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
# yamllint disable rule:line-length
1010
roles:
11+
- role: performancecopilot.metrics.keyserver
1112
- role: performancecopilot.metrics.bpftrace
1213
vars:
1314
bpftrace_users:

tests/tests_verify_from_elasticsearch.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.elasticsearch
89
vars:
910
elasticsearch_agent: true

tests/tests_verify_grafana.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.pcp
89
vars:
910
pcp_rest_api: true

tests/tests_verify_into_elasticsearch.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.elasticsearch
89
vars:
910
elasticsearch_export_metrics: true

tests/tests_verify_mssql.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.mssql
89
- role: performancecopilot.metrics.pcp
910
vars:

tests/tests_verify_pmie_webhook.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.pcp
89
vars:
910
pcp_pmie_endpoint: https://example.com:12345/webhook

tests/tests_verify_postfix.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.postfix
89
- role: performancecopilot.metrics.pcp
910
vars:

tests/tests_verify_repository.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.repository
89
- role: performancecopilot.metrics.pcp
910

tests/tests_verify_retention.yml

+1
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
__test_retention: 137
77

88
roles:
9+
- role: performancecopilot.metrics.keyserver
910
- role: performancecopilot.metrics.pcp
1011
vars:
1112
pcp_pmlogger_discard: "{{ __test_retention }}"

tests/tests_verify_spark.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
hosts: all
55

66
roles:
7+
- role: performancecopilot.metrics.keyserver
78
- role: performancecopilot.metrics.pcp
89
vars:
910
pcp_rest_api: true

0 commit comments

Comments
 (0)