[BUG]: Setting graceful_shutdown on an interface using sonic_lag_interfaces does not work.

[TafkaMax]

Bug Description

I cannot set graceful_shutdown on an interface: https://galaxy.ansible.com/ui/repo/published/dellemc/enterprise_sonic/content/module/sonic_lag_interfaces/

It says it is unchanged in config when running Ansible Check Mode. When I manually add no graceful-shutdown to an interface via sonic-cli it works.

            "fallback": true,
            "fast_rate": true,
            "graceful_shutdown": false,
            "members": {
                "interfaces": [
                    {
                        "member": "Ethernet24"
                    },
                    {
                        "member": "Ethernet28"
                    }
                ]
            },
            "mode": "lacp",
            "name": "PortChannel100",

Product Name

Dell Enteprise Sonic

Component or Module Name

httpapi

DellEMC Enterprise SONiC Ansible Collection Version

dellemc.enterprise_sonic 3.2.0

SONiC Software Version

4.5.1

Configuration

ANSIBLE_PIPELINING(/home/tansper/ansible/ansible.cfg) = True
CONFIG_FILE() = /home/tansper/ansible/ansible.cfg
DEFAULT_HOST_LIST(/home/tansper/ansible/ansible.cfg) = ['/home/tansper/ansible/hosts']
DEFAULT_LOOKUP_PLUGIN_PATH(/home/tansper/ansible/ansible.cfg) = ['/home/tansper/ansible/plugins/lookup']
DEFAULT_VAULT_PASSWORD_FILE(/home/tansper/ansible/ansible.cfg) = /home/tansper/ansible/scripts/vault_password.sh
EDITOR(env: EDITOR) = vim
HOST_KEY_CHECKING(/home/tansper/ansible/ansible.cfg) = False
PAGER(env: PAGER) = less
PERSISTENT_COMMAND_TIMEOUT(/home/tansper/ansible/ansible.cfg) = 180
PERSISTENT_CONNECT_RETRY_TIMEOUT(/home/tansper/ansible/ansible.cfg) = 30
PERSISTENT_CONNECT_TIMEOUT(/home/tansper/ansible/ansible.cfg) = 180

Steps to Reproduce

Use sonic_lag_interfaces module and set graceful_shutdown option to false.

Expected Behavior

It actually sets it to no graceful-shutdown

Actual Behavior

It doesn't change it.

interface PortChannel100 mode active
 switchport trunk allowed Vlan X,Y,Z
 fast_rate
 fallback
 no shutdown

Logs

none

Screenshots

No response

Additional Information

No response

[stalabi1]

I am not able reproduce your issue.
Playbook:

---
- name: LAG
  hosts: datacenter
  connection: httpapi
  collections:
    - dellemc.enterprise_sonic

  tasks:
    - name: Update LAG
      sonic_lag_interfaces:
        config:
          - name: PortChannel100
            mode: lacp
            members:
              interfaces:
                - member: Ethernet24
                - member: Ethernet28
            graceful_shutdown: false
            fast_rate: true
            fallback: true
        state: merged

Playbook execution output:

changed: [sonic1] => {
    "after": [
        {
            "fallback": true,
            "fast_rate": true,
            "graceful_shutdown": false,
            "members": {
                "interfaces": [
                    {
                        "member": "Ethernet24"
                    },
                    {
                        "member": "Ethernet28"
                    }
                ]
            },
            "mode": "lacp",
            "name": "PortChannel100"
        }
    ],
    "before": [],
    "changed": true,
    "commands": [
        {
            "fallback": true,
            "fast_rate": true,
            "graceful_shutdown": false,
            "members": {
                "interfaces": [
                    {
                        "member": "Ethernet24"
                    },
                    {
                        "member": "Ethernet28"
                    }
                ]
            },
            "mode": "lacp",
            "name": "PortChannel100",
            "state": "merged"
        }
    ],
    "invocation": {
        "module_args": {
            "config": [
                {
                    "adv_speed": null,
                    "ethernet_segment": null,
                    "fallback": true,
                    "fast_rate": true,
                    "graceful_shutdown": false,
                    "lacp_individual": null,
                    "members": {
                        "interfaces": [
                            {
                                "member": "Ethernet24"
                            },
                            {
                                "member": "Ethernet28"
                            }
                        ]
                    },
                    "min_links": null,
                    "mode": "lacp",
                    "name": "PortChannel100",
                    "speed": null,
                    "system_mac": null
                }
            ],
            "state": "merged"
        }
    }
}

show running configuration output on device:

sonic# show running-configuration interface PortChannel 100
!
interface PortChannel100
 fast_rate
 fallback
 no graceful-shutdown
 no shutdown

[TafkaMax]

Could you also please try replaced, instead of merged?