-
Notifications
You must be signed in to change notification settings - Fork 51
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 functionality to pass templates #84
base: devel
Are you sure you want to change the base?
Conversation
…g of cisco ios Signed-off-by: jwoogee <jswoods1@gmail.com>
Signed-off-by: jwoogee <jswoods1@gmail.com>
Signed-off-by: z003gn3 <jonathon.woods@target.com>
Signed-off-by: z003gn3 <jonathon.woods@target.com>
…config_manager role. Also removed extra space from run_cli when setting name parameter for textfsm templates. Signed-off-by: z003gn3 <jonathon.woods@target.com>
Signed-off-by: z003gn3 <jonathon.woods@target.com>
Signed-off-by: z003gn3 <jonathon.woods@target.com>
Build failed.
|
Can someone take a look at this one? |
@trishnaguha Sorry to tag you, is there someone or some group I should tag when submitting a PR? I just want to make sure I'm doing it right. |
@jwoogee it looks like you need to add a new line to the end of the file, per the CI linter failure: |
Signed-off-by: Jonathon Woods <jonathon.woods@target.com>
Thank you, I fixed that error. |
Build succeeded.
|
@justjais can you take a look at this to review? |
this PR should not be merged until ansible-network/config_manager#19 is resolved and merged |
I have a PR on config_manager to add this functionality. This allows user to pass through list of templates to load. I also fixed spacing issue in run_cli task.