From 66e29f0a7c4c9507e7c9c2d576765ba17bda75ec Mon Sep 17 00:00:00 2001 From: Ricardo Carrillo Cruz Date: Wed, 29 Mar 2017 22:38:15 +0200 Subject: [PATCH] Do not re.search show configuration stdout if empty (#23087) If the banner is not set, the stdout of 'show configuration | begin banner ' returns empty string thus the re.search raises an exception. Fixes #22216 --- lib/ansible/modules/network/ios/ios_banner.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/ansible/modules/network/ios/ios_banner.py b/lib/ansible/modules/network/ios/ios_banner.py index 6187a6f4a78..a3e136be934 100644 --- a/lib/ansible/modules/network/ios/ios_banner.py +++ b/lib/ansible/modules/network/ios/ios_banner.py @@ -113,7 +113,10 @@ def map_config_to_obj(module): rc, out, err = exec_command(module, 'show running-config | begin banner %s' % module.params['banner']) - output = re.search('\^C(.*)\^C', out, re.S).group(1).strip() + if out: + output = re.search('\^C(.*)\^C', out, re.S).group(1).strip() + else: + output = None obj = {'banner': module.params['banner'], 'state': 'absent'} if output: obj['text'] = output