-
Notifications
You must be signed in to change notification settings - Fork 230
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): Extend router_isis data-model #3539
Conversation
3630d55
to
49f8b22
Compare
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/router-isis.j2
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/router-isis.j2
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/router-isis.j2
Outdated
Show resolved
Hide resolved
...lections/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/router_isis.schema.yml
Outdated
Show resolved
Hide resolved
...lections/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/router_isis.schema.yml
Outdated
Show resolved
Hide resolved
65ad0dc
to
f4af9ab
Compare
ansible_collections/arista/avd/molecule/eos_cli_config_gen/intended/configs/router-isis-new.cfg
Outdated
Show resolved
Hide resolved
...lections/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/router_isis.schema.yml
Outdated
Show resolved
Hide resolved
...lections/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/router_isis.schema.yml
Outdated
Show resolved
Hide resolved
- str | ||
min: 5 | ||
max: 300 | ||
description: Number of seconds |
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.
description: Number of seconds | |
description: Number of seconds. |
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.
and other descriptions
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.
Please add dots to every description you are adding in this file
Co-authored-by: Claus Holbech <[email protected]>
75808df
to
c31ee11
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 - thanks!
tested new generated config in CLI in cEOS 4.31
…ks#3539) Co-authored-by: Claus Holbech <[email protected]>
Change Summary
Add support in the router_isis data model for:
spf-interval
graceful-restart
bfd all-interfaces under address-family
Related Issue(s)
Fixes #3390
Component(s) name
arista.avd.eos_cli_config_gen
Checklist
User Checklist
Repository Checklist