Merge pull request #646 from abhishek-sa1/devel

Issue #645: Dynamic Inventory Fix

control_plane/roles/collect_device_info/files/create_inventory.yml

@@ -185,7 +185,6 @@
             --conf.insecure hosts list --inventory ethernet_inventory -f human --filter "name"
           changed_when: false
           no_log: true
-          run_once: true
           register: ethernet_switches
 
         - name: Assert ethernet switch
@@ -236,7 +235,6 @@
             --conf.insecure hosts list --inventory infiniband_inventory -f human --filter "name"
           changed_when: false
           no_log: true
-          run_once: true
           register: infiniband_switches
 
         - name: Authenticate infiniband Switch
@@ -311,7 +309,6 @@
             --conf.insecure hosts list --inventory powervault_me4_inventory -f human --filter "name"
           changed_when: false
           no_log: true
-          run_once: true
           register: me4_storage
 
         - name: Get auth string for powervault

control_plane/roles/collect_node_info/files/create_inventory.yml

@@ -100,20 +100,17 @@
       ignore_errors: true
 
     - name: Set the hostname from mapping file
-      hostname:
-        name: "{{ host_name.stdout + '.' + hostvars['localhost']['domain_name'] }}"
+      command: hostnamectl set-hostname "{{ host_name.stdout + '.' + hostvars['localhost']['domain_name'] }}"
       when: ('localhost' in hostname_check.stdout) and (mapping_file_present != "" ) and  (mapping_file | bool == true )
       ignore_errors: true
 
     - name: Set the hostname if hostname not present mapping file
-      hostname:
-        name: "compute{{ inventory_hostname.split('.')[-2] + '-' + inventory_hostname.split('.')[-1] + '.' + hostvars['localhost']['domain_name'] }}"
+      command: hostnamectl set-hostname "compute{{ inventory_hostname.split('.')[-2] + '-' + inventory_hostname.split('.')[-1] + '.' + hostvars['localhost']['domain_name'] }}"
       when: ('localhost' in hostname_check.stdout) and (file_present.rc != 0) and (mapping_file | bool == true )
       ignore_errors: true
 
     - name: Set the system hostname
-      hostname:
-        name: "compute{{ inventory_hostname.split('.')[-2] + '-' + inventory_hostname.split('.')[-1]+'.'+ hostvars['localhost']['domain_name'] }}"
+      command: hostnamectl set-hostname "compute{{ inventory_hostname.split('.')[-2] + '-' + inventory_hostname.split('.')[-1]+'.'+ hostvars['localhost']['domain_name'] }}"
       when: ('localhost' in hostname_check.stdout) and (mapping_file | bool == false)
       ignore_errors: true
 
@@ -222,4 +219,4 @@
     - name: Show unreachable hosts
       debug:
         msg: "{{ host_unreachable_msg }} + {{ groups['ungrouped'] }}"
-      when: "'ungrouped' in groups"
+      when: "'ungrouped' in groups"