Fix ios_system lookup enable issue (#30263)
Fixes #29974
Add `None` check while comparing module parameter values (want) with the actual
configuration present on device (have).
(cherry picked from commit d8371cec91
)
This commit is contained in:
parent
ba30a8e4c7
commit
aca0d1a2f8
2 changed files with 25 additions and 1 deletions
|
@ -146,7 +146,7 @@ def map_obj_to_commands(want, have, module):
|
||||||
commands = list()
|
commands = list()
|
||||||
state = module.params['state']
|
state = module.params['state']
|
||||||
|
|
||||||
needs_update = lambda x: want.get(x) and (want.get(x) != have.get(x))
|
needs_update = lambda x: want.get(x) is not None and (want.get(x) != have.get(x))
|
||||||
|
|
||||||
if state == 'absent':
|
if state == 'absent':
|
||||||
if have['hostname'] != 'Router':
|
if have['hostname'] != 'Router':
|
||||||
|
|
|
@ -33,6 +33,30 @@
|
||||||
that:
|
that:
|
||||||
- result.changed == false
|
- result.changed == false
|
||||||
|
|
||||||
|
- name: Disable lookup_source
|
||||||
|
ios_system:
|
||||||
|
lookup_enabled: False
|
||||||
|
provider: "{{ cli }}"
|
||||||
|
authorize: yes
|
||||||
|
register: result
|
||||||
|
|
||||||
|
- assert:
|
||||||
|
that:
|
||||||
|
- result.changed == true
|
||||||
|
- "'no ip domain lookup' in result.commands"
|
||||||
|
|
||||||
|
- name: Disable lookup_source
|
||||||
|
ios_system:
|
||||||
|
lookup_enabled: True
|
||||||
|
provider: "{{ cli }}"
|
||||||
|
authorize: yes
|
||||||
|
register: result
|
||||||
|
|
||||||
|
- assert:
|
||||||
|
that:
|
||||||
|
- result.changed == true
|
||||||
|
- "'ip domain lookup' in result.commands"
|
||||||
|
|
||||||
#- name: change to vrf
|
#- name: change to vrf
|
||||||
# ios_system:
|
# ios_system:
|
||||||
# lookup_source:
|
# lookup_source:
|
||||||
|
|
Loading…
Reference in a new issue