Added DPC role support for SmartSwitch.

[nnelluri-cisco]

Description of PR

Summary:added support for subtype for smartswicth
Fixes # (issue)

Type of change

• Bug fix
• [X ] Testbed and Framework(new/improvement)
• New Test case
  • Skipped for non-supported platforms
• Test case improvement

Back port request

• 202012
• 202205
• 202305
• 202311
• [X ] 202405
• [ X] 202411

Approach

What is the motivation for this PR?

The SmartSwitch platform introduced new backplane interfaces that require role configuration.

How did you do it?

added new config in device specific template.
ansible/templates/minigraph_device.j2

How did you verify/test it?

Deployed the minigraph and confirmed that the DPU configuration is present for all backplane interfaces.

Any platform specific information?

HwSKU: Cisco-8102-28FH-DPU-O

Supported testbed topology if it's a new test case?

t1-28-lag

Documentation

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[nnelluri-cisco]

@prabhataravind @rameshraghupathy @bhavani
Please review

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[prabhataravind]

lgtm

[prabhataravind]

@congh-nvidia @wen587 please review if there are any concerns as this is generic code..

[prabhataravind]

@kperumalbfn could you help add this to 202411?

[mssonicbld]

Cherry-pick PR to 202411: #19894

[mssonicbld]

Cherry-pick PR to 202505: #19902

[congh-nvidia]

Hi @prabhataravind @nnelluri-cisco , looks like this will cause issue on the hwsku with more than 28 ports like Mellanox-SN4280-O8C40, And I still don't understand why we need this since we can add the Dpc role in the hwsku.json and when deploying mingraph it will be taken to the config_db.json automatically by sonic.
For example: https://github.com/sonic-net/sonic-buildimage/blob/3e0bdc60b22e68e11c8b513c4e63f0eab1529147/device/mellanox/x86_64-nvidia_sn4280-r0/Mellanox-SN4280-O8C40/hwsku.json#L247
It doesn't look like a proper way to handle this in test.

[prabhataravind]

@nnelluri-cisco please check and update on the plan to address this without breaking any other platform.

[nnelluri-cisco]

@prabhataravind
will check and Ramesh and will address the issue.

[rameshraghupathy]

@prabhataravind @nnelluri-cisco You don't have to check the index at all. You can check only the role as shown

{% for index in range(num_of_intf) %}
  <a:EthernetInterface>
    <ElementType>DeviceInterface</ElementType>
    <AlternateSpeeds i:nil="true"/>
    <EnableFlowControl>true</EnableFlowControl>
    <Index>1</Index>
    <InterfaceName>{{ port_alias[index] }}</InterfaceName>
    <InterfaceType i:nil="true"/>
    <MultiPortsInterface>false</MultiPortsInterface>
    <PortName>0</PortName>
    <Priority>0</Priority>
{% set intf = port_alias[index] %}
{% if subtype is defined and subtype == 'SmartSwitch' and intf[8:]|int >= 224 %}
    <role>Dpc</role>
{% endif %}
  </a:EthernetInterface>
{% endfor %}