-
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
nmcli: Introduce SIT Tunnel #45937
nmcli: Introduce SIT Tunnel #45937
Conversation
The test
|
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.
Thanks for the contribution. Could you please write a test for this change here - https://github.com/ansible/ansible/blob/devel/test/units/modules/net_tools/test_nmcli.py ? Thanks.
@@ -1066,6 +1089,61 @@ def modify_connection_vlan(self): | |||
|
|||
return cmd | |||
|
|||
def create_connection_sit(self): | |||
cmd = [self.nmcli_bin] |
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.
cmd = [self.nmcli_bin, 'con', 'add', 'type', 'ip-tunnel', 'mode', 'sit', 'con-name']
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 sure . updated.
return cmd | ||
|
||
def modify_connection_sit(self): | ||
cmd = [self.nmcli_bin] |
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.
cmd = [self.nmcli_bin, 'con', 'mod']
I have updated not sure how to run these tests locally ? |
@ssahani You can run tests locally using following command -
|
Thanks apparently I am missing something
Ok I fixed it will fix the test cases |
@Akasurde Thanks for reviewing. Seems all tests passed. |
@Akasurde Ping |
@ssahani Could you please rebase this ? needs_info |
Thanks for looking into it. I have updated #45933 . SInce again be a be conflict once IPIP PR get merged I will rebase it. |
The test
The test
The test
The test
The test
The test
The test
The test
The test
The test
|
This work enables to add sit tunnel via nmcli module Signed-off-by: Susant Sahani susant@redhat.com SUMMARY ISSUE TYPE Feature Pull Request COMPONENT NAME nmcli ANSIBLE VERSION 2.8 ADDITIONAL INFORMATION - nmcli: state: present type: sit conn_name: sit_test1 autoconnect: yes ip_tunnel_dev: enp0s8 ip_tunnel_local: 192.168.1.2 ip_tunnel_remote: 192.168.1.5
@Akasurde Thanks updated. |
rebuild_merge |
This work enables to add sit tunnel via nmcli module Signed-off-by: Susant Sahani susant@redhat.com SUMMARY ISSUE TYPE Feature Pull Request COMPONENT NAME nmcli ANSIBLE VERSION 2.8 ADDITIONAL INFORMATION - nmcli: state: present type: sit conn_name: sit_test1 autoconnect: yes ip_tunnel_dev: enp0s8 ip_tunnel_local: 192.168.1.2 ip_tunnel_remote: 192.168.1.5
This work enables to add sit tunnel via nmcli module
Signed-off-by: Susant Sahani susant@redhat.com
SUMMARY
ISSUE TYPE
COMPONENT NAME
nmcli
ANSIBLE VERSION
ADDITIONAL INFORMATION