-
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
Add supported connection in junos module documentation #38813
Conversation
Add supported connection type in junos module doucmentation.
@@ -81,6 +81,7 @@ | |||
- This module requires the netconf system service be enabled on | |||
the remote device being managed. | |||
- Tested against vSRX JUNOS version 15.1X49-D15.4, vqfx-10000 JUNOS Version 15.1X53-D60.4. | |||
- Works with local connection type only. |
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.
What's the plan for this, since it's connection local only
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.
Since these modules will use junos-eznc
library and not the persistent connection framework it will continue to be supported with connection=local
In future, we might want to move these modules to a role within ansible-network.
@@ -54,6 +54,7 @@ | |||
- This module requires the netconf system service be enabled on | |||
the remote device being managed. | |||
- Tested against vMX JUNOS version 17.3R1.10. | |||
- Works with local connection type only. |
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.
What's the plan for this, since it's connection local only
The test
The test
The test
The test
The test
The test
The test
The test
|
When there are lots of errors I'd suggest starting with the errors from |
The test
The test
The test
The test
The test
The test
The test
The test
|
The test
The test
The test
The test
The test
The test
The test
The test
|
Phew. Some day I need to figure out how to break lines within the YAML, but for now two lines will be fine. Sorry about all the error output. |
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.
That took longer than I'd hoped, but I think this looks good - it clearly suggests that people should use the netconf connection whenever they can, but also tells them which other connection types work.
* Add supported connection in junos module documentation Add supported connection type in junos module doucmentation. * adds link to platform docs * makes recommended connection a code snippet * does this fix banner and command? * does this fix the rest of the errors? (cherry picked from commit 67d561b)
* Add supported connection in junos module documentation Add supported connection type in junos module doucmentation. * adds link to platform docs * makes recommended connection a code snippet * does this fix banner and command? * does this fix the rest of the errors? (cherry picked from commit 67d561b)
Cherry-pick to 2.5 8f2d46e |
* Add supported connection in junos module documentation Add supported connection type in junos module doucmentation. * adds link to platform docs * makes recommended connection a code snippet * does this fix banner and command? * does this fix the rest of the errors?
SUMMARY
Add supported connection type in junos module doucmentation.
ISSUE TYPE
COMPONENT NAME
junos_*
ANSIBLE VERSION
ADDITIONAL INFORMATION