CI(eos_cli_config_gen): Restructure molecule host vars part6 #4712
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-4712
# Activate the virtual environment
source test-avd-pr-4712/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/laxmikantchintakindi/avd.git@refactor/molecule-2#subdirectory=python-avd" --force
# Point Ansible collections path to the Python virtual environment
export ANSIBLE_COLLECTIONS_PATH=$VIRTUAL_ENV/ansible_collections
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/laxmikantchintakindi/avd.git#/ansible_collections/arista/avd/,refactor/molecule-2 --force
# Optional: Install AVD examples
cd test-avd-pr-4712
ansible-playbook arista.avd.install_examples |
github-actions
bot
added
the
state: CI Updated
laxmikantchintakindi
force-pushed
the
refactor/molecule-2
branch
4 times, most recently
from
8c75ff7
to
404ee49
Compare
| vrf: MGMT | ||
| ip_address: 10.73.254.11/24 | ||
| gateway: 10.73.254.253 | ||
|
|
||
| ### VLAN Interfaces ### | ||
| vlan_interfaces: |
There was a problem hiding this comment.
please move vlan interfaces from this host
There was a problem hiding this comment.
Also in ospf host there are ethernet, port-channel and vlan interfaces keys are there. Please move to specific hosts
There was a problem hiding this comment.
Moved.
| @@ -1,3 +1,4 @@ | |||
| --- | |||
| ### Spanning-Tree (MST settings) | |||
There was a problem hiding this comment.
i dont see any file changes for host3.cfg
There was a problem hiding this comment.
Added.
| @@ -163,14 +163,6 @@ loopback_interfaces: | |||
| ipv4_index: 10 | |||
| ipv6_index: 1000 | |||
|
|
|||
| ### Management Interfaces ### | |||
There was a problem hiding this comment.
Can you please remove the duplicate config for isis with respect to all 4 hosts. Add the host only in case config is not supported with each other
There was a problem hiding this comment.
Yes I added more host as they were required for different isis_authentication under router_isis
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
laxmikantchintakindi
force-pushed
the
refactor/molecule-2
branch
from
7e50aa5
to
554b228
Compare
|
Conflicts have been resolved. A maintainer will review the pull request shortly. |
laxmikantchintakindi
force-pushed
the
refactor/molecule-2
branch
from
077db20
to
e5d89f9
Compare
|
Change Summary
Restructure molecule host vars for below hosts.
Related Issue(s)
Fixes #
Component(s) name
arista.avd.eos_cli_config_genProposed changes
Restructure molecule host vars part6
How to test
Checklist
User Checklist
Repository Checklist