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
Refactor(eos_designs): Move default variables to python instead of role defaults. #2760
Refactor(eos_designs): Move default variables to python instead of role defaults. #2760
Conversation
4c6a959
to
892975f
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
e984993
to
d951589
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
f4876be
to
525c35e
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
525c35e
to
326fbaa
Compare
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
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
Move default variables to python instead of role defaults
Related Issue(s)
When reusing AVD code outside of Ansible (pyavd) the default values are lost.
Component(s) name
arista.avd.eos_designs
Proposed changes
yaml_templates_to_facts
to load defaultnode_type_keys
etc. since those are required for validating input vars according to schema usingdynamic_keys
.Note: Still missing
default_templates
which involves other considerations and will be handled separately.How to test
No change to molecule.
Checklist
User Checklist
Repository Checklist