Fix(eos_cli_config_gen): Fixing poe link down power-off action comman…

…d in j2 template
aristanetworks · Oct 10, 2024 · 5d6c476 · 5d6c476
1 parent b302cfa
commit 5d6c476
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/...ta/avd/molecule/eos_cli_config_gen/documentation/devices/ethernet-interfaces.md b/...ta/avd/molecule/eos_cli_config_gen/documentation/devices/ethernet-interfaces.md
@@ -942,7 +942,7 @@ interface Ethernet56
    switchport
    poe priority low
    poe reboot action power-off
-   poe link down action power-off 10
+   poe link down action power-off 10 seconds
    poe shutdown action maintain
    poe limit 30.00 watts
    poe negotiation lldp disabled

diff --git a/...llections/arista/avd/molecule/eos_cli_config_gen/intended/configs/ethernet-interfaces.cfg b/...llections/arista/avd/molecule/eos_cli_config_gen/intended/configs/ethernet-interfaces.cfg
@@ -558,7 +558,7 @@ interface Ethernet56
    switchport
    poe priority low
    poe reboot action power-off
-   poe link down action power-off 10
+   poe link down action power-off 10 seconds
    poe shutdown action maintain
    poe limit 30.00 watts
    poe negotiation lldp disabled

diff --git a/python-avd/pyavd/_eos_cli_config_gen/j2templates/eos/ethernet-interfaces.j2 b/python-avd/pyavd/_eos_cli_config_gen/j2templates/eos/ethernet-interfaces.j2
@@ -805,8 +805,8 @@ interface {{ ethernet_interface.name }}
 {%     endif %}
 {%     if ethernet_interface.poe.link_down.action is arista.avd.defined %}
 {%         set poe_link_down_action_cli = 'poe link down action ' ~ ethernet_interface.poe.link_down.action %}
-{%         if ethernet_interface.poe.link_down.power_off_delay is arista.avd.defined %}
-{%             set poe_link_down_action_cli = poe_link_down_action_cli ~ ' ' ~ ethernet_interface.poe.link_down.power_off_delay %}
+{%         if ethernet_interface.poe.link_down.power_off_delay is arista.avd.defined and ethernet_interface.poe.link_down.action == 'power-off' %}
+{%             set poe_link_down_action_cli = poe_link_down_action_cli ~ ' ' ~ ethernet_interface.poe.link_down.power_off_delay ~ ' seconds' %}
 {%         endif %}
    {{ poe_link_down_action_cli }}
 {%     endif %}