-
Notifications
You must be signed in to change notification settings - Fork 232
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 event-handler trigger on-maintenance. #3629
Feat(eos_cli_config_gen): Add event-handler trigger on-maintenance. #3629
Conversation
77f79b6
to
cab7bba
Compare
...tions/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/event_handlers.schema.yml
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/event-handlers.j2
Outdated
Show resolved
Hide resolved
...tions/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/event_handlers.schema.yml
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/event-handlers.j2
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/event-handlers.j2
Outdated
Show resolved
Hide resolved
464e4e4
to
8ece604
Compare
...tions/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/event_handlers.schema.yml
Outdated
Show resolved
Hide resolved
598f7fc
to
1a6b879
Compare
ansible_collections/arista/avd/molecule/eos_cli_config_gen/intended/configs/event-handlers.cfg
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/molecule/eos_cli_config_gen/intended/configs/event-handlers.cfg
Outdated
Show resolved
Hide resolved
...le_collections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/event-handlers.yml
Outdated
Show resolved
Hide resolved
...tions/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/event_handlers.schema.yml
Outdated
Show resolved
Hide resolved
...tions/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/event_handlers.schema.yml
Show resolved
Hide resolved
1a6b879
to
f16bc73
Compare
8e98358
to
a78245b
Compare
ansible_collections/arista/avd/roles/eos_cli_config_gen/docs/tables/event-handlers.md
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/docs/tables/event-handlers.md
Outdated
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.
LGMT, tested on lab
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!
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/event-handlers.j2
Outdated
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.
.
de07932
to
6ffa1c2
Compare
...tions/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/event_handlers.schema.yml
Outdated
Show resolved
Hide resolved
...e_collections/arista/avd/molecule/eos_cli_config_gen/documentation/devices/event-handlers.md
Show resolved
Hide resolved
…s/schema_fragments/event_handlers.schema.yml
3523d75
to
cf5cdda
Compare
@ClausHolbechArista Thank you very much for implementing this. |
I have tested this addition to the event-handlers but it does not fix my warnings
|
Hi @kmueller68, According to the data-model we have, you need to give the input in below format:
|
Hi @Shivani-gslab I have tested it right now with the new model again and it is working well. Thank you! |
Change Summary
Add event-handler trigger "on-maintenance".
Related Issue(s)
Fixes #2951
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
Adding an option for "on-maintenance" in event-handlers.trigger and updated the data-model for the same.
How to test
Add input trigger on-maintenance:
Check the output config:
Test the config on EOS CLI
Checklist
Repository Checklist