Skip to content

Commit 0452c14

Browse files
rsafronovyzigold
andauthored
Fix kepler metrics tests (#224)
Kepler metrics tests were added in [1] though the merged commit was not the correct one. This PR aplies changes from [2] that had more correct code of the tests. Fixes OSPRH-14029 [1] #193 [2] 0b5d856 Co-authored-by: Jaromír Wysoglad <[email protected]>
1 parent b240e82 commit 0452c14

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

roles/telemetry_verify_metrics/tasks/verify_kepler_metrics.yml

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,6 @@
66
common_cr_list:
77
- kind: scrapeconfigs.monitoring.rhobs
88
name: telemetry-kepler
9-
kepler_dashboard_metrics:
10-
- kepler_vm_platform_joules_total
11-
- kepler_node_platform_joules_total
12-
- kepler_process_joules_total
13-
- kepler_container_joules_total
14-
- kepler_node_info
159

1610
- name: Verify Kepler container is up
1711
ansible.builtin.include_tasks:
@@ -27,10 +21,18 @@
2721
- name: |
2822
TEST Use openstack observabilityclient to verify kepler metrics are stored in prometheus
2923
RHOSO-1216
24+
vars:
25+
kepler_metrics:
26+
- kepler_vm_platform_joules_total
27+
- kepler_node_platform_joules_total
28+
- kepler_process_joules_total
29+
- kepler_container_joules_total
30+
- kepler_node_info
3031
ansible.builtin.shell: |
31-
{{ openstack_cmd }} metric show --disable-rbac kepler_node_info
32+
{{ openstack_cmd }} metric show --disable-rbac {{ item }}
3233
register: result
3334
delay: 30
3435
retries: 10
3536
changed_when: false
36-
until: result.rc == 0 and "kepler_node_info" in result.stdout
37+
until: result.rc == 0 and item in result.stdout
38+
loop: "{{ kepler_metrics }}"

0 commit comments

Comments
 (0)