diff options
author | Adrian Likins <alikins@redhat.com> | 2018-01-20 15:05:53 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-01-20 15:05:53 -0500 |
commit | ec9769c82fc1818b31fd3c4a31de4cba9a5c2ab8 (patch) | |
tree | 5b227c94861c127952c46734216f2d76b6f48d14 | |
parent | fc180a378ad6582c4c4a0d1b972113fd3afb4ae9 (diff) | |
download | ansible-ec9769c82fc1818b31fd3c4a31de4cba9a5c2ab8.tar.gz |
Facts distribution clear linux 31501 (#32453)
The search string used to look for Clear Linux
was changed in 45a9f967749ec68e2077fe1d1d32dd37660ab376 to
be more specific, but was too specific. Now finding
a substring match for 'Clear Linux' in /usr/lib/os-release
is enough to consider a match.
Since the details of the full name in os-release varies
('Clear Linux Software for Intel Architecture',
'Clear Linux OS for Intel Architecture', etc) the
search string match was failing and would fall back to the
'first word in the release file' method resulting in
ansible_distribution='NAME="Clear'
Also add a meta fact indicating which search string
was matched.
Test case info from:
https://github.com/ansible/ansible/issues/31501#issuecomment-340861535
Fixes #31501
-rwxr-xr-x | hacking/tests/gen_distribution_version_testcase.py | 1 | ||||
-rw-r--r-- | lib/ansible/module_utils/facts/system/distribution.py | 3 | ||||
-rw-r--r-- | test/units/module_utils/test_distribution_version.py | 31 |
3 files changed, 34 insertions, 1 deletions
diff --git a/hacking/tests/gen_distribution_version_testcase.py b/hacking/tests/gen_distribution_version_testcase.py index bc2ab3c09c..fdd0a058bc 100755 --- a/hacking/tests/gen_distribution_version_testcase.py +++ b/hacking/tests/gen_distribution_version_testcase.py @@ -32,6 +32,7 @@ filelist = [ '/etc/altlinux-release', '/etc/os-release', '/etc/coreos/update.conf', + '/usr/lib/os-release', ] fcont = {} diff --git a/lib/ansible/module_utils/facts/system/distribution.py b/lib/ansible/module_utils/facts/system/distribution.py index 7e3cbcf6c1..992aca13c9 100644 --- a/lib/ansible/module_utils/facts/system/distribution.py +++ b/lib/ansible/module_utils/facts/system/distribution.py @@ -82,7 +82,7 @@ class DistributionFiles: 'OracleLinux': 'Oracle Linux', 'RedHat': 'Red Hat', 'Altlinux': 'ALT Linux', - 'ClearLinux': 'Clear Linux Software for Intel Architecture', + 'ClearLinux': 'Clear Linux', 'SMGL': 'Source Mage GNU/Linux', } @@ -115,6 +115,7 @@ class DistributionFiles: if self.SEARCH_STRING[name] in dist_file_content: # this sets distribution=RedHat if 'Red Hat' shows up in data dist_file_dict['distribution'] = name + dist_file_dict['distribution_file_search_string'] = self.SEARCH_STRING[name] else: # this sets distribution to what's in the data, e.g. CentOS, Scientific, ... dist_file_dict['distribution'] = dist_file_content.split()[0] diff --git a/test/units/module_utils/test_distribution_version.py b/test/units/module_utils/test_distribution_version.py index 06d9162000..b806099a88 100644 --- a/test/units/module_utils/test_distribution_version.py +++ b/test/units/module_utils/test_distribution_version.py @@ -839,6 +839,37 @@ DISTRIB_DESCRIPTION="CoreOS 976.0.0 (Coeur Rouge)" } }, + # ClearLinux https://github.com/ansible/ansible/issues/31501#issuecomment-340861535 + { + "platform.dist": [ + "Clear Linux OS for Intel Architecture", + "18450", + "clear-linux-os" + ], + "input": { + "/usr/lib/os-release": ''' +NAME="Clear Linux OS for Intel Architecture" +VERSION=1 +ID=clear-linux-os +VERSION_ID=18450 +PRETTY_NAME="Clear Linux OS for Intel Architecture" +ANSI_COLOR="1;35" +HOME_URL="https://clearlinux.org" +SUPPORT_URL="https://clearlinux.org" +BUG_REPORT_URL="mailto:dev@lists.clearlinux.org" +PRIVACY_POLICY_URL="http://www.intel.com/privacy" +''' + }, + "name": "Clear Linux OS for Intel Architecture 1", + "result": { + "distribution_release": "clear-linux-os", + "distribution": "ClearLinux", + "distribution_major_version": "18450", + "os_family": "ClearLinux", + "distribution_version": "18450" + } + }, + # ArchLinux with no /etc/arch-release but with a /etc/os-release with NAME=Arch Linux # The fact needs to map 'Arch Linux' to 'Archlinux' for compat with 2.3 and earlier facts { |