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
New options to Interface #60499
New options to Interface #60499
Conversation
@thedoubl3j how do we force ansibot to return this? |
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.
docs look clear, just need CI fixed.
if parameters.get('dns_domain_name') is not None: | ||
options['dns-domain-name'] = parameters['dns_domain_name'] | ||
if parameters.get('listen_for_dns_query') is not None: | ||
options['listen-for-dns-query'] = str(parameters['listen_for_dns_query']) |
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.
Is it enough to use str, or do we need to convert to lowercase?
Look also at line 296. Should we use the same construct?
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.
I confirmed the ZAPI accepts both False and false, or True and true, so this is OK as is.
It's only when making comparisons with returned values (get), that we need to be case sensitive.
@@ -237,6 +263,14 @@ def get_interface(self): | |||
return_value['netmask'] = interface_attributes['netmask'] | |||
if interface_attributes.get_child_by_name('firewall-policy'): | |||
return_value['firewall_policy'] = interface_attributes['firewall-policy'] | |||
if interface_attributes.get_child_by_name('dns-domain-name') is not 'none': |
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.
should be != 'none'
is not None is correct. is not 'none' is not correct.
@carchi8py @lonico still waiting on fixes for this 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.
shipit
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, tests updated with new features. shipit
* update to interface * force ansibot to run again * fixes
* update to interface * force ansibot to run again * fixes
SUMMARY
New Options for Interface
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION