-
Notifications
You must be signed in to change notification settings - Fork 184
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
Feat(eos_cli_config_gen): Add capability to hide passwords and keys in generated doc and conf #2806
Conversation
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 found a few more:
- daemon_terminattr the ingest auth key
- radius_server keys (seen in deprecated vars so maybe old variant?)
- snmp community with vrf and version
- documentation tables with snmp communities. That becomes a bit hard...
ansible_collections/arista/avd/molecule/eos_cli_config_gen/documentation/devices/aaa.md
Show resolved
Hide resolved
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
…n generated doc and conf
Conflicts have been resolved. A maintainer will review the pull request shortly. |
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
Change Summary
Introduce a new filter
hide_passwords
which allow to replace a value in a Jinja Template by<removed>
.By default the values identified in Jinja are hidden in the documentation but kept in the generated config (cf
eos_cil_config_gen/default/main/main.yml
)Related Issue(s)
Fixes #2493
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
true
) and config (false
)How to test
molecule documentation updated.
Checklist
User Checklist
Repository Checklist