-
Notifications
You must be signed in to change notification settings - Fork 23.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Eos subnet overlap fix #43023
Eos subnet overlap fix #43023
Conversation
@@ -341,7 +341,11 @@ def load_config(self, config, commit=False, replace=False): | |||
commands = ['configure session %s' % session, 'abort'] | |||
self.send_request(commands) | |||
err = response['error'] | |||
self._module.fail_json(msg=err['message'], code=err['code']) | |||
error_text = [] | |||
for data in err['data']: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nate, here err['data'] will always return a List right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell, data has one entry per command, which should mean a minimum of two, as we start with configure session ...
.
* Detect overlapping subnet & provide better errors (cherry picked from commit dba9754)
* Detect overlapping subnet & provide better errors (cherry picked from commit dba9754)
* Detect overlapping subnet & provide better errors (cherry picked from commit dba9754)
SUMMARY
Fixes #40187
Plus an unrelated fix for the check mode check
ISSUE TYPE
COMPONENT NAME
eos
ANSIBLE VERSION