From 1917906dd6c279b8b1281e5d6c99fa4a886b6dfe Mon Sep 17 00:00:00 2001 From: Marcin Praczko Date: Mon, 13 Oct 2014 22:57:03 +0200 Subject: [PATCH] Fix detecting distribution release on OpenSuSE Ansible raised exception during parsering /etc/SuSE-release file. Regular expresion should use string instead of list. Fix tested on OpenSuse 13.1 --- lib/ansible/module_utils/facts.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/ansible/module_utils/facts.py b/lib/ansible/module_utils/facts.py index 04a25ab9d66..8ec1b4f7c7b 100644 --- a/lib/ansible/module_utils/facts.py +++ b/lib/ansible/module_utils/facts.py @@ -328,10 +328,11 @@ class Facts(object): break elif path == '/etc/SuSE-release': data = data.splitlines() - release = re.search('CODENAME *= *([^\n]+)\n', data) - if release: - self.facts['distribution_release'] = release.groups()[0].strip() - break + for line in data: + release = re.search('CODENAME *= *([^\n]+)', line) + if release: + self.facts['distribution_release'] = release.groups()[0].strip() + break elif name == 'Debian': data = get_file_content(path) if 'Debian' in data: