Fix ios_vlan issue CP in 2.5 (#42668)

* Make ios_vlan identify vlans starting with 9 (#42247)

* Make ios_vlan identify vlans starting with 9

* Add unit test for vlan id start with 9

(cherry picked from commit 70e33ef92c)

* Added changelog for ios_vlan fix
pull/43174/head
Nilashish Chakraborty 6 years ago committed by Matt Davis
parent 3a481c4548
commit 85122a6c38

@ -0,0 +1,2 @@
- bugfixes:
- ios_vlan - fix unable to identify correct vlan issue (https://github.com/ansible/ansible/pull/42247)

@ -218,7 +218,7 @@ def parse_to_logical_rows(out):
if not l:
"""Skip empty lines."""
continue
if '0' < l[0] < '9':
if '0' < l[0] <= '9':
"""Line starting with a number."""
if started_yielding:
yield cur_row

@ -4,5 +4,6 @@ VLAN Name Status Ports
Gi1/0/52
Gi1/0/54
2 vlan2 active Gi1/0/6, Gi1/0/7
9 vlan9 active Gi1/0/6
1002 fddi-default act/unsup
1003 fddo-default act/unsup

@ -59,6 +59,12 @@ class TestIosVlanModule(TestIosModule):
]
self.assertEqual(result['commands'], expected_commands)
def test_ios_vlan_id_startwith_9(self):
set_module_args({'vlan_id': '9', 'name': 'vlan9', 'state': 'present'})
result = self.execute_module(changed=False)
expected_commands = []
self.assertEqual(result['commands'], expected_commands)
def test_ios_vlan_rename(self):
set_module_args({'vlan_id': '2', 'name': 'test', 'state': 'present'})
result = self.execute_module(changed=True)
@ -121,6 +127,14 @@ class TestIosVlanModule(TestIosModule):
'state': 'active',
'vlan_id': '2',
},
{
'name': 'vlan9',
'interfaces': [
'GigabitEthernet1/0/6',
],
'state': 'active',
'vlan_id': '9',
},
{
'name': 'fddi-default',
'interfaces': [],

Loading…
Cancel
Save