-
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 support for qos map exp #3204
Feat(eos_cli_config_gen): Add support for qos map exp #3204
Conversation
...e_collections/arista/avd/roles/eos_cli_config_gen/schemas/eos_cli_config_gen.jsonschema.json
Outdated
Show resolved
Hide resolved
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
817b077
to
9aa6ccd
Compare
ansible_collections/arista/avd/molecule/eos_cli_config_gen/documentation/devices/qos.md
Show resolved
Hide resolved
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. Tested locally
@philippebureau I have fixed a couple of issues. The things with blank lines are old issues, but since we are touching the template I wanted to get it fixed here. You also had a copy/paste error for the |
Change Summary
add knob in eos_cli_config_gen to configure qos map exp
Related Issue(s)
N/A
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
adding a key under qos.map for exp field
Addition to qos.schema:
How to test
See molecule test
Repository Checklist