Feat(eos_cli_config_gen): add options tagged and untagged phone to switchport phone trunk for ethernet interfaces

[kmueller68]

Change Summary

From EOS 4.30.0F CLI offers two more options for switchport phone trunk:

nac-710p(config-if-Et2)#switchport phone trunk ?
  tagged    Configure the port to send tagged packets
  untagged  Configure the port to send untagged packets
nac-710p(config-if-Et2)#switchport phone trunk tagged ?
  phone  Tagged packets on phone VLAN
  <cr>
nac-710p(config-if-Et2)#switchport phone trunk untagged ?
  phone  Untagged packets on phone VLAN
  <cr>

Related Issue(s)

Fixes #2479

Component(s) name

arista.avd.eos_cli_config_gen

Proposed changes

# Switched Interfaces
ethernet_interfaces:
  <Ethernet_interface_2 >:
    phone:
      trunk: < tagged | tagged phone | untagged | untagged phone >

No changes in eos_cli_config_gen/templates/eos/ethernet-interfaces.j2necessary, since it just takes the pure string value.
Just an extention of the Schema.

How to test

I kept the existing molecule test unchanged.

  - name: Ethernet13
    logging:
      event:
        link_status: false
        congestion_drops: false
        spanning_tree: false
        storm_control: false
    description: interface_in_mode_access_with_voice
    type: switched
    native_vlan: 100
    mode: "trunk phone"
    phone:
      trunk: untagged
      vlan: 70

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)

[chetryan]

Ideally the eos_cli_config_gen molecule test should be update to have an example of each of the options. You can refer to the existing example here:

https://github.com/aristanetworks/ansible-avd/blob/58bacf3f7a8b0746221b1a24c87bc81777fe35c1/ansible_collections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/ethernet-interfaces.yml#LL278C22-L278C22

[kmueller68]

@chetryan done... hope this meets your expectation.

[chetryan]

LGTM!