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 ethernet_interface with dot1x eapol authentication_failure_falback #2482
Conversation
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/documentation/dot1x.j2
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/documentation/dot1x.j2
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/ethernet-interfaces.j2
Outdated
Show resolved
Hide resolved
…tes/eos/ethernet-interfaces.j2 Co-authored-by: Guillaume Mulocher <gmulocher@arista.com>
…tes/documentation/dot1x.j2 Co-authored-by: Guillaume Mulocher <gmulocher@arista.com>
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/documentation/dot1x.j2
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/ethernet-interfaces.j2
Outdated
Show resolved
Hide resolved
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
...llections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/ethernet-interfaces.yml
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.
Looks like there is no test for the new feature? (all the Eapol values are -
in all the tests)
dot1x eapol authentication failure fallback mba timeout 600
|
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 for the updates @dmilyt - will merge this once the CI finishes
Pipeline failing is unrelated to the PR but to a new version of ansible-lint - merging the PR and the pipeline issue will be resolved elsewhere. |
Change Summary
extend ethernet_interface with dot1x eapol authentication_failure_falback
Related Issue(s)
Fixes #2344
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
How to test
Molecule
Checklist
User Checklist
Repository Checklist