ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

Return correct version on installed VyOS (#39115)

Browse source 
* Return correct version on installed VyOS

Previously existing regexp will shows only "VyOS" without numeric output of router version.
For example: from  "Version:      VyOS 1.1.6" only VyOS will be written in ansible_net_version variable
For more informative output numeric value should be returned as well

* Fixed unittests
This commit is contained in:
Maugli 2018-09-14 19:21:30 +02:00 committed by Nilashish Chakraborty
parent 829f48e3f5
commit 235b11f681
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/ansible/modules/network/vyos/vyos_facts.py
View file

@ -135,7 +135,7 @@ class Default(FactsBase):
self.facts['hostname'] = self.responses[1] self.facts['hostname'] = self.responses[1]
def parse_version(self, data): def parse_version(self, data):
match = re.search(r'Version:\s*(\S+)', data) match = re.search(r'Version:\s*(.*)', data)
if match: if match:
return match.group(1) return match.group(1)

6
test/units/modules/network/vyos/test_vyos_facts.py
View file

@ -63,7 +63,7 @@ class TestVyosFactsModule(TestVyosModule):
facts = result.get('ansible_facts') facts = result.get('ansible_facts')
self.assertEqual(len(facts), 5) self.assertEqual(len(facts), 5)
self.assertEqual(facts['ansible_net_hostname'].strip(), 'vyos01') self.assertEqual(facts['ansible_net_hostname'].strip(), 'vyos01')
self.assertEqual(facts['ansible_net_version'], 'VyOS') self.assertEqual(facts['ansible_net_version'], 'VyOS 1.1.7')
def test_vyos_facts_not_all(self): def test_vyos_facts_not_all(self):
set_module_args(dict(gather_subset='!all')) set_module_args(dict(gather_subset='!all'))
@ -71,7 +71,7 @@ class TestVyosFactsModule(TestVyosModule):
facts = result.get('ansible_facts') facts = result.get('ansible_facts')
self.assertEqual(len(facts), 5) self.assertEqual(len(facts), 5)
self.assertEqual(facts['ansible_net_hostname'].strip(), 'vyos01') self.assertEqual(facts['ansible_net_hostname'].strip(), 'vyos01')
self.assertEqual(facts['ansible_net_version'], 'VyOS') self.assertEqual(facts['ansible_net_version'], 'VyOS 1.1.7')
def test_vyos_facts_exclude_most(self): def test_vyos_facts_exclude_most(self):
set_module_args(dict(gather_subset=['!neighbors', '!config'])) set_module_args(dict(gather_subset=['!neighbors', '!config']))
@ -79,7 +79,7 @@ class TestVyosFactsModule(TestVyosModule):
facts = result.get('ansible_facts') facts = result.get('ansible_facts')
self.assertEqual(len(facts), 5) self.assertEqual(len(facts), 5)
self.assertEqual(facts['ansible_net_hostname'].strip(), 'vyos01') self.assertEqual(facts['ansible_net_hostname'].strip(), 'vyos01')
self.assertEqual(facts['ansible_net_version'], 'VyOS') self.assertEqual(facts['ansible_net_version'], 'VyOS 1.1.7')
def test_vyos_facts_invalid_subset(self): def test_vyos_facts_invalid_subset(self):
set_module_args(dict(gather_subset='cereal')) set_module_args(dict(gather_subset='cereal'))