Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Multi-asic support for test_gnmi_configdb TCs #15407

Open
agadia-cisco opened this issue Nov 6, 2024 · 4 comments
Open

[Bug]: Multi-asic support for test_gnmi_configdb TCs #15407

agadia-cisco opened this issue Nov 6, 2024 · 4 comments

Comments

@agadia-cisco
Copy link
Contributor

Issue Description

For Multi-Asic devices, test_gnmi_configdb_full_01 TC fails because sonic-cfggen -d --print-data isn't picking the right config_db; by default its picking the config_db.json instead of asic specific config_db due to which the values of PORT key and show interface status are not in sync.

Therefore right config_db has to be picked which has the PORT keys which are being used in the TC.

Results you see

    "PORT": {
        "Eth0/0/0/0": {
            "admin_status": "up",
            "description": "ASIC0:Eth0-ASIC0",
            "lanes": "0",
            "mtu": "9100",
            "pfc_asym": "off",
            "speed": "400000",
            "tpid": "0x8100"
        },
        "Eth0/0/0/1": {
            "admin_status": "up",
            "description": "ASIC0:Eth8-ASIC0",
            "lanes": "0",
            "mtu": "9100",
            "pfc_asym": "off",
            "speed": "400000",
            "tpid": "0x8100"
        },
....
....
....

Results you expected to see

    "PORT": {
        "Ethernet0": {
            "admin_status": "up",
            "alias": "Eth0/0/0/0",
            "asic_port_name": "Eth0-ASIC0",
            "description": "ARISTA01T3:Ethernet1",
            "fec": "rs",
            "index": "0",
            "lanes": "1288,1289,1290,1291",
            "mtu": "9100",
            "pfc_asym": "off",
            "role": "Ext",
            "speed": "100000",
            "tpid": "0x8100"
        },
        "Ethernet8": {
            "admin_status": "up",
            "alias": "Eth0/0/0/1",
            "asic_port_name": "Eth8-ASIC0",
            "description": "ARISTA01T3:Ethernet2",
            "fec": "rs",
            "index": "1",
            "lanes": "1280,1281,1282,1283",
            "mtu": "9100",
            "pfc_asym": "off",
            "role": "Ext",
            "speed": "100000",
            "tpid": "0x8100"
        },
 ....
....
....

Is it platform specific

generic

Relevant log output

No response

Output of show version

No response

Attach files (if any)

No response

@agadia-cisco
Copy link
Contributor Author

agadia-cisco commented Nov 6, 2024

I am picking this issue & will raise a PR for it soon.

cc - @vperumal

@vperumal
Copy link
Contributor

vperumal commented Nov 6, 2024

FYI @abdosi @yejianquan . @agadia-cisco is going to fix the issue and upstream the PR

@yejianquan
Copy link
Collaborator

@xincunli-sonic for viz

@agadia-cisco
Copy link
Contributor Author

SET TCs fails because Multi-Asic Support is needed for ApplyPatchDb API.
Issue - sonic-net/sonic-gnmi#329

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants