-
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: fix vlan connection modification Fixes #42322 #42415
Conversation
@dangoscomb Thanks for the PR. Could you please add unit test here ? Thanks, |
fix syntax of vlan modification command
@Akasurde amended the 2 tests as requested. They run fine locally |
bot_status |
1 similar comment
bot_status |
Componentslib/ansible/modules/net_tools/nmcli.py test/units/modules/net_tools/test_nmcli.py Metadatawaiting_on: ansible |
shipit |
rebuild_merge |
…42415) * ensure optional items are set to empty strings rather than not presented fix syntax of vlan modification command * extended tests for nmcli
fix syntax of vlan modification command
SUMMARY
fix command syntax for nmcli con mod command
set None values on params to '' in order to remove previously set values (and result in syntactically correct commands)
Fixes #42322
ISSUE TYPE
COMPONENT NAME
nmcli
ANSIBLE VERSION
current version below, however nmcli taken from devel branch
ADDITIONAL INFORMATION