diff options
| author | Abhijeet Kasurde <akasurde@redhat.com> | 2018-01-15 22:16:43 +0530 |
|---|---|---|
| committer | Adrian Likins <alikins@redhat.com> | 2018-01-15 11:46:43 -0500 |
| commit | 3950f5b9ce07cdcf398f7043034908648945753a (patch) | |
| tree | 5eef1e44d0c555f1a3f5a29422f109261a3d97b5 /lib/ansible/module_utils/facts | |
| parent | 6a75c1a1385a5b7de9cef28d2db57225be5cd234 (diff) | |
| download | ansible-3950f5b9ce07cdcf398f7043034908648945753a.tar.gz | |
facts: Get virtual facts about VMware (#34132)
This fixes fact gathering of VMware guest machines with
older Linux Kernel versions. These older Kernels do not support /sys
filesystem which is used to gather virtualization related facts.
'dmidecode' is the safest option to find out virtualization related facts.
Fixes: #21573
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Diffstat (limited to 'lib/ansible/module_utils/facts')
| -rw-r--r-- | lib/ansible/module_utils/facts/virtual/linux.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/lib/ansible/module_utils/facts/virtual/linux.py b/lib/ansible/module_utils/facts/virtual/linux.py index 68f3cfe0a2..fcb1568f3e 100644 --- a/lib/ansible/module_utils/facts/virtual/linux.py +++ b/lib/ansible/module_utils/facts/virtual/linux.py @@ -220,6 +220,17 @@ class LinuxVirtual(Virtual): virtual_facts['virtualization_role'] = 'guest' return virtual_facts + # In older Linux Kernel versions, /sys filesystem is not available + # dmidecode is the safest option to parse virtualization related values + dmi_bin = self.module.get_bin_path('dmidecode') + (rc, out, err) = self.module.run_command('%s -s system-product-name' % dmi_bin) + if rc == 0: + # Strip out commented lines (specific dmidecode output) + vendor_name = ''.join([line.strip() for line in out.splitlines() if not line.startswith('#')]) + if vendor_name in ['VMware Virtual Platform', 'VMware7,1']: + virtual_facts['virtualization_type'] = 'VMware' + virtual_facts['virtualization_role'] = 'guest' + # If none of the above matches, return 'NA' for virtualization_type # and virtualization_role. This allows for proper grouping. virtual_facts['virtualization_type'] = 'NA' |