-
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
Vyos interface desc fix28 #57228
Vyos interface desc fix28 #57228
Conversation
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com> (cherry picked from commit be1820d)
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590, just so you are aware we have a dedicated Working Group for network. |
Build succeeded (third-party-check pipeline).
|
@@ -204,6 +204,8 @@ def map_obj_to_commands(updates): | |||
value = w.get(item) | |||
|
|||
if value and value != obj_in_have.get(item): | |||
if item == 'description': | |||
value = "\'" + str(value) + "\'" |
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.
Just a note, str is not safe to use like this if value can contain non-ascii characters. let me know if that's the case and I can help you test that case and fix it.
Merged for the 2.8.1 release. |
SUMMARY
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/network/vyos/vyos_interface.py
ADDITIONAL INFORMATION