openstack/openstack-ansible-os_nova
Code Issues Proposed changes

Remove systemd conditionals

Browse Source 
All operating systems supported by the role have systemd and these
conditionals are no longer needed.

Change-Id: I35500f7eec993b2bcdb245a995a05cacf2c596f8
This commit is contained in:
Major Hayden 2018-02-15 08:35:19 -06:00 committed by Jesse Pretorius (odyssey4me)
parent d53626d3bc
commit ff26ba2158
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
handlers/main.yml
View File

@ -18,14 +18,14 @@
name: "{{ libvirt_service_name }}" name: "{{ libvirt_service_name }}"
enabled: yes enabled: yes
state: "restarted" state: "restarted"
daemon_reload: "{{ (ansible_service_mgr == 'systemd') | ternary('yes', omit) }}" daemon_reload: yes
- name: Stop services - name: Stop services
service: service:
name: "{{ item.service_name }}" name: "{{ item.service_name }}"
enabled: yes enabled: yes
state: "stopped" state: "stopped"
daemon_reload: "{{ (ansible_service_mgr == 'systemd') | ternary('yes', omit) }}" daemon_reload: yes
with_items: "{{ filtered_nova_services }}" with_items: "{{ filtered_nova_services }}"
register: _stop register: _stop
until: _stop | success until: _stop | success
@ -56,7 +56,7 @@
name: "{{ item.service_name }}" name: "{{ item.service_name }}"
enabled: yes enabled: yes
state: "started" state: "started"
daemon_reload: "{{ (ansible_service_mgr == 'systemd') | ternary('yes', omit) }}" daemon_reload: yes
with_items: "{{ filtered_nova_services }}" with_items: "{{ filtered_nova_services }}"
register: _start register: _start
until: _start | success until: _start | success

1
tasks/drivers/kvm/nova_disable_smt.yml
View File

@ -29,7 +29,6 @@
owner: "root" owner: "root"
group: "root" group: "root"
when: when:
- ansible_service_mgr == 'systemd'
- smt_disable_result.rc is defined - smt_disable_result.rc is defined
- smt_disable_result.rc == 0 - smt_disable_result.rc == 0
register: smt_disable_service register: smt_disable_service