-
Notifications
You must be signed in to change notification settings - Fork 197
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_designs): MLAG secondary should use short-esi from MLAG primary #4172
Feat(eos_designs): MLAG secondary should use short-esi from MLAG primary #4172
Conversation
Review docs on Read the Docs To test this pull request: # Create virtual environment for this testing below the current directory
python -m venv test-avd-pr-4172
# Activate the virtual environment
source test-avd-pr-4172/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/Vibhu-gslab/avd.git@short_esi_auto#subdirectory=python-avd" --force
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/Vibhu-gslab/avd.git#/ansible_collections/arista/avd/,short_esi_auto --force
# Optional: Install AVD examples
cd test-avd-pr-4172
ansible-playbook arista.avd.install_examples |
610f1b8
to
ae04547
Compare
...e_collections/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/DC1_FABRIC.yml
Outdated
Show resolved
Hide resolved
a1858bc
to
d574ede
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!
d574ede
to
5b25372
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.
Move the mlag logic to eos_designs_facts/short_esi.py
b80e2a0
to
a76a089
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.
Move the mlag logic to eos_designs_facts/short_esi.py
a76a089
to
a6689fe
Compare
stale review
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 only one minor comment
4ca6b0e
to
f589f2c
Compare
Quality Gate passedIssues Measures |
Change Summary
MLAG secondary should use short-esi from MLAG primary.
Related Issue(s)
Fixes #4124
Component(s) name
arista.avd.eos_designs
How to test
CI will check
Checklist
Repository Checklist