[debug dump] dump interface module added

[vivekrnv]

What I did

HLD for Dump Utility: HLD.

• Added the Interface Module to the Debug Dump Utility
• Added the Corresponding UT's
• Refactored Common methods required into match_helper.py

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

root@mts-sonic-dut/root/# dump state interface Ethernet16 --key-map
{
    "Ethernet16": {
        "CONFIG_DB": {
            "keys": [
                "INTERFACE|Ethernet16",
                "INTERFACE|Ethernet16|10.0.0.8/31"
            ],
            "tables_not_found": []
        },
        "APPL_DB": {
            "keys": [
                "INTF_TABLE:Ethernet16",
                "INTF_TABLE:Ethernet16:10.0.0.8/31"
            ],
            "tables_not_found": []
        },
        "STATE_DB": {
            "keys": [
                "INTERFACE_TABLE|Ethernet16",
                "INTERFACE_TABLE|Ethernet16|10.0.0.8/31"
            ],
            "tables_not_found": []
        },
        "ASIC_DB": {
            "keys": [
                "ASIC_STATE:SAI_OBJECT_TYPE_ROUTER_INTERFACE:oid:0x60000000005d0"
            ],
            "tables_not_found": [],
            "vidtorid": {
                "oid:0x60000000005d0": "oid:0x34100000006"
            }
        }
    }
}


root@mts-sonic-dut/root/# dump state interface PortChannel1234 --key-map
{
    "PortChannel1234": {
        "CONFIG_DB": {
            "keys": [
                "PORTCHANNEL_INTERFACE|PortChannel1234",
                "PORTCHANNEL_INTERFACE|PortChannel1234|7.7.7.1/24"
            ],
            "tables_not_found": []
        },
        "APPL_DB": {
            "keys": [
                "INTF_TABLE:PortChannel1234",
                "INTF_TABLE:PortChannel1234:7.7.7.1/24"
            ],
            "tables_not_found": []
        },
        "STATE_DB": {
            "keys": [
                "INTERFACE_TABLE|PortChannel1234",
                "INTERFACE_TABLE|PortChannel1234|7.7.7.1/24"
            ],
            "tables_not_found": []
        },
        "ASIC_DB": {
            "keys": [
                "ASIC_STATE:SAI_OBJECT_TYPE_ROUTER_INTERFACE:oid:0x60000000005ec"
            ],
            "tables_not_found": [],
            "vidtorid": {
                "oid:0x60000000005ec": "oid:0x4100000006"
            }
        }
    }
}

root@mts-sonic-dut/root/# dump state interface Eth0.1 --key-map
{
    "Eth0.1": {
        "CONFIG_DB": {
            "keys": [
                "VLAN_SUB_INTERFACE|Eth0.1",
                "VLAN_SUB_INTERFACE|Eth0.1|9.9.9.9/24"
            ],
            "tables_not_found": []
        },
        "APPL_DB": {
            "keys": [
                "INTF_TABLE:Eth0.1",
                "INTF_TABLE:Eth0.1:9.9.9.9/24"
            ],
            "tables_not_found": []
        },
        "STATE_DB": {
            "keys": [
                "INTERFACE_TABLE|Eth0.1",
                "INTERFACE_TABLE|Eth0.1|9.9.9.9/24"
            ],
            "tables_not_found": []
        },
        "ASIC_DB": {
            "keys": [
                "ASIC_STATE:SAI_OBJECT_TYPE_ROUTER_INTERFACE:oid:0x60000000005e9"
            ],
            "tables_not_found": [],
            "vidtorid": {
                "oid:0x60000000005e9": "oid:0x90100000006"
            }
        }
    }
}

[lgtm-com]

This pull request introduces 6 alerts when merging 91a6fab into 84ddbfd - view on LGTM.com

new alerts:

• 6 for Unused import

[vivekrnv]

/azpw run

[mssonicbld]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[vivekrnv]

/azpw run

[mssonicbld]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[liat-grozovik]

@SuvarnaMeenakshi kindly reminder to review and signoff

[liat-grozovik]

/azp run Azure.sonic-utilities

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[SuvarnaMeenakshi]

Why do we need this specific change when we already have "dump state port " and "dump state portchannel"?
#1853
#1667

[dgsudharsan]

Why do we need this specific change when we already have "dump state port " and "dump state portchannel"? #1853 #1667

The interface is a separate table in config_db and it is different from port which is for the physical port attributes and portchannel which is for LAG's properties

[liat-grozovik]

@SuvarnaMeenakshi kindly reminder to review and signoff

[dgsudharsan]

@SuvarnaMeenakshi Can you please review and signoff?

[SuvarnaMeenakshi]

lgtm

[vivekrnv]

Request for 202111