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_cli_config_gen)!: Require queue_monitor_length.enabled to be set #2429
Refactor(eos_cli_config_gen)!: Require queue_monitor_length.enabled to be set #2429
Conversation
6e415a0
to
b1cc595
Compare
...ons/arista/avd/molecule/eos_cli_config_gen_v4.0/inventory/host_vars/queue_monitor_length.yml
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/README_v4.0.md
Outdated
Show resolved
Hide resolved
...arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/queue_monitor_length.schema.yml
Outdated
Show resolved
Hide resolved
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
c2751e2
to
7a140cc
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
e6ab13b
to
b12c0c2
Compare
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. |
228fb5d
to
178fdf8
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
178fdf8
to
47bdba5
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. |
bcf1346
to
368b7e8
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
368b7e8
to
37fc4bc
Compare
c001a99
to
3a01346
Compare
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
For queue_monitor_length data-model "enabled: true" will be required in AVD4.0.
Requires #2616
Related Issue(s)
Fixes #85
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
How to test
Checklist
User Checklist
Repository Checklist