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): Add testcases for ipaddressing and move it to plugin utils #2410
Refactor(eos_designs): Add testcases for ipaddressing and move it to plugin utils #2410
Conversation
ea46682
to
ac5b195
Compare
ansible_collections/arista/avd/plugins/plugin_utils/ip_addressing_utils.py
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/tests/unit/plugins/plugin_utils/test_ipaddressing.py
Outdated
Show resolved
Hide resolved
90c5535
to
37fc0d2
Compare
de90b0e
to
0a398bd
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
9c7305d
to
8eaea21
Compare
ansible_collections/arista/avd/tests/unit/plugins/plugin_utils/test_ipaddressing.py
Outdated
Show resolved
Hide resolved
..._collections/arista/avd/roles/eos_designs/python_modules/network_services/vxlan_interface.py
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/plugins/plugin_utils/ip_addressing_utils.py
Outdated
Show resolved
Hide resolved
8eaea21
to
0150985
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
ansible_collections/arista/avd/plugins/plugin_utils/utils/get_ip_from_pool.py
Outdated
Show resolved
Hide resolved
...ble_collections/arista/avd/roles/eos_designs/python_modules/ip_addressing/avdipaddressing.py
Show resolved
Hide resolved
...ble_collections/arista/avd/roles/eos_designs/python_modules/ip_addressing/avdipaddressing.py
Show resolved
Hide resolved
..._collections/arista/avd/roles/eos_designs/python_modules/network_services/vxlan_interface.py
Show resolved
Hide resolved
ansible_collections/arista/avd/tests/unit/plugins/plugin_utils/test_ipaddressing.py
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/tests/unit/plugins/plugin_utils/test_ipaddressing.py
Outdated
Show resolved
Hide resolved
…/test_ipaddressing.py Co-authored-by: Guillaume Mulocher <gmulocher@arista.com>
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.
LGTM - once CI passes
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.
LGTM
Change Summary
Writing unit testcase for get_ip_from_pool and move it to plugin_utils.
Proposed changes
_ip()
toget_ip_from_pool()
and move toplugin_utils.utils
eos_designs
Python modules toip_addressing
How to test
Checklist
User Checklist
Repository Checklist