Use systemd module instead of command

This patch changes the use of systemctl via the command module over to the Ansible systemd module. One conditional check for systemd was removed since all supported distributions deployed by OSA have systemd. Change-Id: I926e94d185f4999e13b33f7d035b9dd5cf732f0b
openstack · Feb 1, 2018 · ad29910 · ad29910
1 parent 1f5e998
commit ad29910
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/tasks/install_zypper.yml b/tasks/install_zypper.yml
@@ -95,7 +95,8 @@
     - rabbitmq-config
 
 - name: Reload the systemd daemon
-  command: "systemctl daemon-reload"
+  systemd:
+    daemon_reload: yes
   when:
     - rabbitmq_server_service_added | changed
   tags:

diff --git a/tasks/rabbitmq_install.yml b/tasks/rabbitmq_install.yml
@@ -22,17 +22,15 @@
     - rabbitmq_server-install
 
 - name: Reload the systemd daemon
-  command: "systemctl daemon-reload"
+  systemd:
+    daemon_reload: yes
   when:
     - install_rabbitmq | changed
-    - ansible_service_mgr == 'systemd'
   tags:
     - rabbitmq-apt-packages
     - rabbitmq-yum-packages
     - rabbimtq-zypper-packages
     - rabbitmq_server-install
-    # Avoid ANSIBLE0006 systemctl used in place of systemd module issue
-    - skip_ansible_lint
 
 - name: Ensure there are no policy-rc files
   file:

diff --git a/tasks/rabbitmq_post_install.yml b/tasks/rabbitmq_post_install.yml
@@ -80,7 +80,8 @@
     - rabbitmq-config
 
 - name: Reload the systemd daemon
-  command: "systemctl daemon-reload"
+  systemd:
+    daemon_reload: yes
   when:
     - ansible_service_mgr == 'systemd'
     - rabbit_resource_limit | changed