Skip to content

Commit af0c35d

Browse files
committed
Fixed issues from code review
1 parent b0b47e5 commit af0c35d

File tree

4 files changed

+5
-4
lines changed

4 files changed

+5
-4
lines changed

roles/edpm_iscsid/tasks/run.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@
5656
state: restarted
5757

5858
- name: Remove iscsid container restart sentinel file
59+
become: true
5960
ansible.builtin.file:
6061
path: /etc/iscsi/.iscsid_restart_required
6162
state: absent

roles/edpm_ovn/tasks/healthchecks.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,4 +27,4 @@
2727
setype: container_file_t
2828
owner: "{{ ansible_user | default(ansible_user_id) }}"
2929
group: "{{ ansible_user | default(ansible_user_id) }}"
30-
mode: '0700'
30+
mode: '0700'

roles/edpm_telemetry/defaults/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ edpm_telemetry_healthcheck_sources:
4949
# kepler: exporter
5050
# If telemetry services should have health checks enabled
5151
edpm_telemetry_healthcheck: true
52-
# List of ceilometer agents to be stopped during EDPM adoption
52+
# List of exporters to be deployed in the compute node
5353
edpm_telemetry_enabled_exporters:
5454
- ceilometer_agent_compute
55-
- node_exporter
55+
- node_exporter

roles/edpm_telemetry/tasks/exporter.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,4 @@
2323
become: true
2424
ansible.builtin.systemd:
2525
state: restarted
26-
name: "edpm_{{ exporter }}.service"
26+
name: "edpm_{{ exporter }}.service"

0 commit comments

Comments
 (0)