Fixes loginBanner plucking over eapi transport when loginBanner isn't…

… set. (ansible#23363)
rajasaur · Apr 6, 2017 · 394cb34 · 394cb34
1 parent 990ba90
commit 394cb34
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/ansible/modules/network/eos/eos_banner.py b/lib/ansible/modules/network/eos/eos_banner.py
@@ -121,7 +121,8 @@ def map_config_to_obj(module):
         else:
             # On EAPI we need to extract the banner text from dict key
             # 'loginBanner'
-            obj['text'] = output[0]['loginBanner'].strip('\n')
+            if isinstance(output[0], dict) and 'loginBanner' in output[0].keys():
+                obj['text'] = output[0]['loginBanner'].strip('\n')
         obj['state'] = 'present'
     return obj
 

diff --git a/test/integration/group_vars/eos.yaml b/test/integration/group_vars/eos.yaml
@@ -12,4 +12,5 @@ eapi:
   password: "{{ eos_eapi_pass | default('admin') }}"
   transport: eapi
   use_ssl: no
+  port: 80
   authorize: yes