-
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
junos_config documentation missing "update" choice #51299
Conversation
The test
The test
|
@crab86, just so you are aware we have a dedicated Working Group for network. |
The test
|
Thanks @crab86 for this - to make the PR pass our CI suite, you need to remove line 618 of this ignore.txt file. Removing that line will get our CI system to run the |
The current junos_config module documentation only lists "merge","override" and "replace" as update parameter choices. Looking into junos_python.py "update" is another option which is really helpful and in my opinion the most needed option. ''' update=dict(default='merge', choices=['merge', 'override', 'replace', 'update']) ''' +label: docsite_pr
The current junos_config module documentation only lists "merge","override" and "replace" as update parameter choices. Looking into junos_python.py "update" is another option which is really helpful and in my opinion the most needed option. ''' update=dict(default='merge', choices=['merge', 'override', 'replace', 'update']) ''' Co-Authored-By: crab86 <sgesenhoff@gmail.com>
SUMMARY
The current junos_config module documentation only lists "merge","override" and "replace" as update parameter choices. Looking into junos_python.py "update" is another option which is really helpful and in my opinion the most needed option.
ISSUE TYPE
COMPONENT NAME
junos_config