-
Notifications
You must be signed in to change notification settings - Fork 190
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
Refactor(eos_designs): Using common method for raising duplicate detection error in python_modules #3033
Refactor(eos_designs): Using common method for raising duplicate detection error in python_modules #3033
Conversation
…ction error in python_modules
f2907f9
to
e095379
Compare
e095379
to
8a93c99
Compare
list_of_dicts=ethernet_interfaces, | ||
primary_key="name", | ||
new_dict=ethernet_interface, | ||
context="Ethernet Interfaces defined under underlay", |
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.
There is not really a concept of "under underlay" for a user of AVD so we need to reword this
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.
fixed
f"Duplicate port-channel name {candidate_port_channel_config['name']} with conflicting configurations found while generating port-channels for" | ||
" connected-endpoints or network-ports" | ||
) | ||
append_if_not_duplicate( |
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.
do you still need the wrapper function _add_port_channel_if_not_duplicate
? it looks like we could remove it and just call append_if_not_duplicate
in the main code?
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.
fixed
list_of_dicts=neighbors, | ||
primary_key="ip_address", | ||
new_dict=neighbor, | ||
context="Ip address defined under BGP neighbor for underlay", |
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.
context="Ip address defined under BGP neighbor for underlay", | |
context="IP address defined under BGP neighbor for underlay", |
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.
fixed
2b5bf51
to
86aa6f6
Compare
f9723b4
to
fff8bc3
Compare
fff8bc3
to
77dfa75
Compare
Change Summary
Using common_method for raising duplicate detection error in python_modules
Related Issue(s)
Fixes #2749
Component(s) name
arista.avd.eos_designs
Checklist
User Checklist
Repository Checklist