fix ansible_os_family fact on openSUSE Leap

ansible_os_family on openSUSE Leap has the wrong value:

        "ansible_os_family": "openSUSE Leap",

It should be:

        "ansible_os_family": "Suse",

This change fixes that by adding the relevant key and ensuring that dict
lookups replace ' ' with '_' so the key does not contain a space.
pull/14899/head
Marcus Furlong 9 years ago committed by James Cammarata
parent 6072dad15f
commit 31ce8bc3a2

@ -274,7 +274,7 @@ class Facts(object):
Archlinux = 'Archlinux', Manjaro = 'Archlinux', Mandriva = 'Mandrake', Mandrake = 'Mandrake', Altlinux = 'Altlinux', Archlinux = 'Archlinux', Manjaro = 'Archlinux', Mandriva = 'Mandrake', Mandrake = 'Mandrake', Altlinux = 'Altlinux',
Solaris = 'Solaris', Nexenta = 'Solaris', OmniOS = 'Solaris', OpenIndiana = 'Solaris', Solaris = 'Solaris', Nexenta = 'Solaris', OmniOS = 'Solaris', OpenIndiana = 'Solaris',
SmartOS = 'Solaris', AIX = 'AIX', Alpine = 'Alpine', MacOSX = 'Darwin', SmartOS = 'Solaris', AIX = 'AIX', Alpine = 'Alpine', MacOSX = 'Darwin',
FreeBSD = 'FreeBSD', HPUX = 'HP-UX' FreeBSD = 'FreeBSD', HPUX = 'HP-UX', openSUSE_Leap = 'Suse'
) )
# TODO: Rewrite this to use the function references in a dict pattern # TODO: Rewrite this to use the function references in a dict pattern
@ -511,8 +511,9 @@ class Facts(object):
machine_id = machine_id.split('\n')[0] machine_id = machine_id.split('\n')[0]
self.facts["machine_id"] = machine_id self.facts["machine_id"] = machine_id
self.facts['os_family'] = self.facts['distribution'] self.facts['os_family'] = self.facts['distribution']
if self.facts['distribution'] in OS_FAMILY: distro = self.facts['distribution'].replace(' ', '_')
self.facts['os_family'] = OS_FAMILY[self.facts['distribution']] if distro in OS_FAMILY:
self.facts['os_family'] = OS_FAMILY[distro]
def get_cmdline(self): def get_cmdline(self):
data = get_file_content('/proc/cmdline') data = get_file_content('/proc/cmdline')

Loading…
Cancel
Save