Feat(eos_cli_config_gen): Add support for configuring dhcp server ipv4 and dhcp server ipv6 for Port-Channel interfaces

[Shivani-gslab]

Change Summary

Add support for configuring dhcp server ipv4 and dhcp server ipv6 for Port-Channel interfaces

Related Issue(s)

Fixes #4875

Component(s) name

arista.avd.eos_cli_config_gen

Proposed changes

Adding new key dhcp_server_ipv4 under port-channel-interfaces.

port_channel_interfaces:
     - dhcp_server_ipv4: <bool>

How to test

Test added in molecule, verify the EOS CLI

Checklist

User Checklist

• N/A

Repository Checklist

• My code has been rebased from devel before I start
• I have read the CONTRIBUTING document.
• My change requires a change to the documentation and documentation have been updated accordingly.
• I have updated molecule CI testing accordingly. (check the box if not applicable)

[github-actions]

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-4885
# Activate the virtual environment
source test-avd-pr-4885/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/Shivani-gslab/avd.git@dhcp_server_ipv4_#4875#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/Shivani-gslab/avd.git#/ansible_collections/arista/avd/,dhcp_server_ipv4_#4875 --force
# Optional: Install AVD examples
cd test-avd-pr-4885
ansible-playbook arista.avd.install_examples

[laxmikantchintakindi]

Need to correct PR title and also I think it would be better to add dhcp_server_ipv6 also.

[gmuloc]

@Shivani-gslab - if this merges after #4752 we will need to update the configuration there

[Shivani-gslab]

@Shivani-gslab - if this merges after #4752 we will need to update the configuration there

@gmuloc shall I add dhcp_server_ipv6 also ? and make this open, we can merge this earlier?

[Vibhu-gslab]

LGTM!

[ClausHolbechArista]

This if is not needed. The for loop would just not loop if the list is empty. Same for v6