-
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
Add junos_l3_interfaces #59026
Add junos_l3_interfaces #59026
Conversation
|
||
for config in want: | ||
if config['name'] not in existing_l3_intfs: | ||
continue |
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.
junos_l3_interfaces:
state: deleted
Will this logic handle the ^ scenario that deletes all the l3 interface config is a key option (name in this case) within config is not mentioned?
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.
Yes, as in that case if you check I'm passing the same existing intf lists. What this would do is only skip it in case the interface is not a l3 one.
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.
oh, hold on, my bad, was thinking about another thing
loop_var: test_case_to_run | ||
|
||
- name: run test case (connection=local) | ||
include: "{{ test_case_to_run }} ansible_connection=local" |
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.
connection=local is not supported. This task is not required.
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.
will remove
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.
junos_facts module doc needs to be updated to reflect l3_interfaces
facts gathering
The test
|
Pull-request updated, HEAD is now 1fb2efe2b19e7550b92fd520a8b40d240d5f61be |
c667e62
to
1fb2efe
Compare
The test
|
Build failed (third-party-check pipeline) integration testing with
|
Pull-request updated, HEAD is now ac20e10835fee87a0b8fd4de755e8aa8581e7f0c |
1fb2efe
to
ac20e10
Compare
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. Thanks!
Build failed (third-party-check pipeline) integration testing with
|
Pull-request updated, HEAD is now 49079333f6f0374c0fcfe86b4afca27c7cbf7c10 |
ac20e10
to
4907933
Compare
The test
|
Build failed (third-party-check pipeline) integration testing with
|
recheck |
Build failed (third-party-check pipeline) integration testing with
|
recheck |
Build failed (third-party-check pipeline) integration testing with
|
Pull-request updated, HEAD is now aa4399688e1b3a1db6eaf6cbb7a1e61bb9710d2b |
4907933
to
aa43996
Compare
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.
Please add deprecation for existing junos_l3_interface
module. For example refer: https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/network/junos/_junos_interface.py#L25
Pull-request updated, HEAD is now 8f8c0bb85646694770ad70b5b66cd7dd2eed9de5 |
aa43996
to
8f8c0bb
Compare
Pull-request updated, HEAD is now de736be4a71e73fd2d1b455f7ad9839a8a646301 |
8f8c0bb
to
de736be
Compare
Pull-request updated, HEAD is now 84111d31f296dfa2f4ee5e14318fb957140ddb3c |
de736be
to
84111d3
Compare
Pull-request updated, HEAD is now 2336dd39f8ee14feefb92655362e9d8d46eb1d83 |
84111d3
to
2336dd3
Compare
Build failed (third-party-check pipeline) integration testing with
|
Pull-request updated, HEAD is now f6d9206 |
2336dd3
to
f6d9206
Compare
Build succeeded (third-party-check pipeline).
|
SUMMARY
Add junos_l3_interface
This commit adds junos_l3_interface resource module, allowing operations such
as these for l3 interfaces configuration in junos
The bootstrapping from this module has been done using resource module builder
[1]
[1] https://github.com/ansible-network/resource_module_builder
ISSUE TYPE
COMPONENT NAME
junos_l3_interface
ADDITIONAL INFORMATION
As this module uses netconf, it does require ncclient, as mentioned there.