diff --git a/lib/ansible/modules/network/ios/ios_logging.py b/lib/ansible/modules/network/ios/ios_logging.py index 698c6c4021..b9019b0415 100644 --- a/lib/ansible/modules/network/ios/ios_logging.py +++ b/lib/ansible/modules/network/ios/ios_logging.py @@ -232,17 +232,17 @@ def map_config_to_obj(module): for line in data.split('\n'): match = re.search(r'logging (\S+)', line, re.M) + if match: + if match.group(1) in dest_group: + dest = match.group(1) - if match.group(1) in dest_group: - dest = match.group(1) - else: - pass - - obj.append({'dest': dest, + obj.append({ + 'dest': dest, 'name': parse_name(line, dest), 'size': parse_size(line, dest), 'facility': parse_facility(line), - 'level': parse_level(line, dest)}) + 'level': parse_level(line, dest) + }) return obj diff --git a/test/integration/targets/ios_logging/tests/cli/basic.yaml b/test/integration/targets/ios_logging/tests/cli/basic.yaml index 468d3611dd..be365d9a17 100644 --- a/test/integration/targets/ios_logging/tests/cli/basic.yaml +++ b/test/integration/targets/ios_logging/tests/cli/basic.yaml @@ -1,4 +1,30 @@ --- +# ensure logging configs are empty +- name: Remove host logging + ios_logging: + dest: host + name: 172.16.0.1 + state: absent + authorize: yes + provider: "{{ cli }}" + +- name: Remove console + ios_logging: + dest: console + level: warnings + state: absent + authorize: yes + provider: "{{ cli }}" + +- name: Remove buffer + ios_logging: + dest: buffered + size: 8000 + authorize: yes + state: absent + provider: "{{ cli }}" + +# start tests - name: Set up host logging ios_logging: dest: host