We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I think it would be a good idea to update the README.md of ntc-nautobot-plugin-nornir to replace the sample config:
PLUGINS_CONFIG = { "nautobot_plugin_nornir": { # ... "dispatcher_mapping": None, "username": "ntc", "password": "password123", "secret": "password123", } }
with
PLUGINS_CONFIG = { "nautobot_plugin_nornir": { "nornir_settings": { "credentials": "nautobot_plugin_nornir.plugins.credentials.settings_vars.CredentialsSettingsVars # ... }, # ... "dispatcher_mapping": None, "username": "ntc", "password": "password123", "secret": "password123", } }
It would be much more explicit and avoid some mistake
The text was updated successfully, but these errors were encountered:
And you can also fix a typo in the previous bloc of code:
"use_config_context": {"secrets", False, "connection_options": True},
should be:
"use_config_context": {"secrets": False, "connection_options": True},
comma after secrets to be replaced by colons
Sorry, something went wrong.
@jeffkala This issue reflects the changes that could be made to simplify or reduce errors. Thank you
fixes #59
e4c081f
Doc cleanup and better error handling (#60)
53e89de
* fixes #59, #56
Successfully merging a pull request may close this issue.
I think it would be a good idea to update the README.md of ntc-nautobot-plugin-nornir to replace the sample config:
with
It would be much more explicit and avoid some mistake
The text was updated successfully, but these errors were encountered: