[action] [PR:19310] Fix buffer queue cnt test to account for different BUFFER_QUEUE configs by mssonicbld · Pull Request #19466 · sonic-net/sonic-mgmt

mssonicbld · 2025-07-08T07:50:00Z

Description of PR

Summary:
Fixes #19201
MSADO: 33479698

Type of change

Back port request

Approach

What is the motivation for this PR?

Currently this test does not account for BUFFER_QUEUE configs to have only entry for an interface.

In some SKU's and config

BUFFER_QUEUE may have config such as

"BUFFER_QUEUE": {
"Ethernet0|0-2": {
"profile": "egress_lossy_profile"
},
"Ethernet0|3-4": {
"profile": "egress_lossless_profile"
},
"Ethernet0|5-7": {
"profile": "egress_lossy_profile"
},

or even

"BUFFER_QUEUE": {
"Ethernet0|0": {
"profile": "egress_lossy_profile"
},
"Ethernet0|1": {
"profile": "egress_lossless_profile"
},
"Ethernet0|2": {
"profile": "egress_lossy_profile"
}, ... etc

The test works fine for the above configs.

However if the BUFFER_QUEUE was configured such as

"BUFFER_QUEUE": {
"Ethernet0|0-9": {
"profile": "egress_lossy_profile"
},
"Ethernet100|0-9": {
"profile": "egress_lossless_profile"
},
"Ethernet200|0-9": {
"profile": "egress_lossy_profile"
},

When we delete Ethernet0|0-9 as part of del data['BUFFER_QUEUE'][ interface_buffer_queues[0] ] we end up deleting all queues for that interface from COUNTERS_QUEUE_NAME_MAP which causes check_buffer_queues_cnt_cmd_output to always return false.

This test checks for such edge case and we split the BUFFER_QUEUE config such that we can delete an entry without deleting all entries in the map.

How did you do it?

Code change

How did you verify/test it?

Test on 20241110

01/07/2025 19:00:48 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"failed": true, "changed": true, "end": "2025-07-01 19:00:48.806079", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:0\"\n}\n]\nGRPC error\n redis: nil", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:0 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "delta": "0:00:00.199525", "stderr": "", "rc": 1, "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:0 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "start": "2025-07-01 19:00:48.606554", "msg": "non-zero return code", "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:0\"", "}", "]", "GRPC error", " redis: nil"], "stderr_lines": [], "_ansible_no_log": false}
01/07/2025 19:00:48 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:1 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:49 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:49.227872", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:1\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x15000000000679", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:1 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:48.981566", "stderr": "", "delta": "0:00:00.246306", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:1 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:1\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x15000000000679"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}
01/07/2025 19:00:49 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:2 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:49 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:49.614484", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:2\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067a", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:2 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:49.411463", "stderr": "", "delta": "0:00:00.203021", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:2 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:2\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067a"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}
01/07/2025 19:00:49 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:3 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:50 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:49.997748", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:3\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067b", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:3 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:49.798159", "stderr": "", "delta": "0:00:00.199589", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:3 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:3\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067b"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}
01/07/2025 19:00:50 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:4 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:50 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:50.381489", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:4\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067c", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:4 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:50.177327", "stderr": "", "delta": "0:00:00.204162", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:4 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:4\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067c"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}
01/07/2025 19:00:50 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:5 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:50 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:50.771962", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:5\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067d", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:5 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:50.566323", "stderr": "", "delta": "0:00:00.205639", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:5 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:5\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067d"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}
01/07/2025 19:00:50 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:6 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:51 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:51.236571", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:6\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067e", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:6 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:50.951268", "stderr": "", "delta": "0:00:00.285303", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:6 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "choir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:6\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067e"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}

Any platform specific information?

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

Documentation

…gs (sonic-net#19310) Approach What is the motivation for this PR? Currently this test does not account for BUFFER_QUEUE configs to have only entry for an interface. In some SKU's and config BUFFER_QUEUE may have config such as "BUFFER_QUEUE": { "Ethernet0|0-2": { "profile": "egress_lossy_profile" }, "Ethernet0|3-4": { "profile": "egress_lossless_profile" }, "Ethernet0|5-7": { "profile": "egress_lossy_profile" }, or even "BUFFER_QUEUE": { "Ethernet0|0": { "profile": "egress_lossy_profile" }, "Ethernet0|1": { "profile": "egress_lossless_profile" }, "Ethernet0|2": { "profile": "egress_lossy_profile" }, ... etc The test works fine for the above configs. However if the BUFFER_QUEUE was configured such as "BUFFER_QUEUE": { "Ethernet0|0-9": { "profile": "egress_lossy_profile" }, "Ethernet100|0-9": { "profile": "egress_lossless_profile" }, "Ethernet200|0-9": { "profile": "egress_lossy_profile" }, When we delete Ethernet0|0-9 as part of del data['BUFFER_QUEUE'][ interface_buffer_queues[0] ] we end up deleting all queues for that interface from COUNTERS_QUEUE_NAME_MAP which causes check_buffer_queues_cnt_cmd_output to always return false. This test checks for such edge case and we split the BUFFER_QUEUE config such that we can delete an entry without deleting all entries in the map. How did you do it? Code change How did you verify/test it? Test on 20241110 01/07/2025 19:00:48 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"failed": true, "changed": true, "end": "2025-07-01 19:00:48.806079", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:0\"\n}\n]\nGRPC error\n redis: nil", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:0 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "delta": "0:00:00.199525", "stderr": "", "rc": 1, "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:0 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "start": "2025-07-01 19:00:48.606554", "msg": "non-zero return code", "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:0\"", "}", "]", "GRPC error", " redis: nil"], "stderr_lines": [], "_ansible_no_log": false} 01/07/2025 19:00:48 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:1 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:49 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:49.227872", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:1\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x15000000000679", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:1 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:48.981566", "stderr": "", "delta": "0:00:00.246306", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:1 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:1\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x15000000000679"], "stderr_lines": [], "_ansible_no_log": false, "failed": false} 01/07/2025 19:00:49 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:2 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:49 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:49.614484", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:2\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067a", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:2 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:49.411463", "stderr": "", "delta": "0:00:00.203021", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:2 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:2\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067a"], "stderr_lines": [], "_ansible_no_log": false, "failed": false} 01/07/2025 19:00:49 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:3 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:50 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:49.997748", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:3\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067b", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:3 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:49.798159", "stderr": "", "delta": "0:00:00.199589", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:3 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:3\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067b"], "stderr_lines": [], "_ansible_no_log": false, "failed": false} 01/07/2025 19:00:50 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:4 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:50 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:50.381489", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:4\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067c", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:4 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:50.177327", "stderr": "", "delta": "0:00:00.204162", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:4 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:4\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067c"], "stderr_lines": [], "_ansible_no_log": false, "failed": false} 01/07/2025 19:00:50 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:5 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:50 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:50.771962", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:5\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067d", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:5 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:50.566323", "stderr": "", "delta": "0:00:00.205639", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:5 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "chdir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:5\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067d"], "stderr_lines": [], "_ansible_no_log": false, "failed": false} 01/07/2025 19:00:50 base._run L0071 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell, args=["python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:6 -xt COUNTERS_DB -o \"xxxxxxxxx\" "], kwargs={"module_ignore_errors": true} 01/07/2025 19:00:51 base._run L0108 DEBUG | /var/src/sonic-mgmt-int/tests/telemetry/test_telemetry.py::get_buffer_queues_cnt#47: [xxxxx-x] AnsibleModule::shell Result => {"changed": true, "end": "2025-07-01 19:00:51.236571", "stdout": "Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path\n ------------------------- \n [elem {\n name: \"COUNTERS_QUEUE_NAME_MAP\"\n}\nelem {\n name: \"Ethernet0:6\"\n}\n]\nThe GetResponse is below\n-------------------------\n\noid:0x1500000000067e", "cmd": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:6 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "rc": 0, "start": "2025-07-01 19:00:50.951268", "stderr": "", "delta": "0:00:00.285303", "invocation": {"module_args": {"creates": null, "executable": null, "_uses_shell": true, "strip_empty_ends": true, "_raw_params": "python /root/gnxi/gnmi_cli_py/py_gnmicli.py -g -t xx.xx.xxx.xxx -p 50051 -m get -x COUNTERS_QUEUE_NAME_MAP/Ethernet0:6 -xt COUNTERS_DB -o \"xxxxxxxxx\" ", "removes": null, "argv": null, "warn": true, "choir": null, "stdin_add_newline": true, "stdin": null}}, "stdout_lines": ["Performing GetRequest, encoding=JSON_IETF to xx.xx.xxx.xxx with the following gNMI Path", " ------------------------- ", " [elem {", " name: \"COUNTERS_QUEUE_NAME_MAP\"", "}", "elem {", " name: \"Ethernet0:6\"", "}", "]", "The GetResponse is below", "-------------------------", "", "oid:0x1500000000067e"], "stderr_lines": [], "_ansible_no_log": false, "failed": false}

mssonicbld · 2025-07-08T07:50:06Z

Original PR: #19310

mssonicbld · 2025-07-08T07:50:10Z

/azp run

azure-pipelines · 2025-07-08T07:50:27Z

Azure Pipelines successfully started running 1 pipeline(s).

Code sync sonic-net/sonic-mgmt:202411 => 202412 ``` * c3fd4c0 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250802:1704 - Merge remote-tracking branch 'base/202411' into code-sync-202412 |\ | * a7a638a (base/202411) nnelluri-cisco 250722:1131 - cherry pick for the PR#17101 (sonic-net#19748) | * fd4ae21 siva-prasad-cisco 250731:0002 - Ignore log analyzer error message related to source port GID mapping in kernel (sonic-net#19879) | * f986cfa Xu Chen 250730:2120 - Fix saithrift URL generation for Mellanox on internal-202411 branch sonic-net#19899 | * a07fb04 nnelluri-cisco 250604:1347 - filterd backplane ports while checking 'sudo sfputil show error-status' (sonic-net#17859) | * d707fcf nnelluri-cisco 250729:1208 - Added DPC role support for SmartSwitch. (sonic-net#18157) | * 80ea05a dypet 250414:1059 - Fix BFD status check and ipv6 PTF intermittent issue. (sonic-net#17819) | * fc623da Xu Chen 250725:1036 - support dynamic saithrift URL generation (sonic-net#19528) | * 0e5fc8a Mai Bui 250724:1807 - [202411] Ignore auditd error in loganalyzer (sonic-net#19767) | * d03770d Hua Liu 250218:1409 - [TACACS] Increase tacacs server reachability check timeout (sonic-net#16971) | * e8111ac nnelluri-cisco 250717:1033 - Generate new npu-dpu ports Ethernet224 To Ethernet280 for cisco smartswicth "Cisco-8102-28FH-DPU-O" (sonic-net#19257) | * e0060fa Zain Budhwani 250716:1109 - Remove yang_validate option from 202411 (sonic-net#19674) | * 89c4281 AharonMalkin 250715:2042 - Add Nvidia to supported platforms on intf_fec test (sonic-net#19398) | * f35fc20 mssonicbld 250716:0129 - Fix buffer queue cnt test to account for different BUFFER_QUEUE configs (sonic-net#19310) (sonic-net#19466) | * 3e0cabd Chuan Wu 250704:0327 - Update vlan ping test to override the affection of secondary vlan ip (sonic-net#18867) | * ff9d112 Longxiang Lyu 250714:1508 - [dualtor-io] Fix `test_active_link_admin_down_config_reload_link_up_downstream_standby` on Cisco/MLNX (sonic-net#18829) (sonic-net#19511) | * e870d66 Yawen 250714:1246 - [manual cherry-pick] add topo dualtor-aa-64-breakout to qos_sai_base.py (sonic-net#19543) | * 8369780 prabhataravind 250711:0929 - Enable skipped BFD and VxLAN tests on smartswitch T1 (sonic-net#19498) ```

…ISTRY if it is not null (sonic-net#19466) (sonic-net#19565) Why I did it DEFAULT_CONTAINER_REGISTRY didn't work as expected in some scenario. How I did it When check for docker arch, use DEFAULT_CONTAINER_REGISTRY if it is not null. Co-authored-by: Liu Shilong <[email protected]>

mssonicbld added the automerge label Jul 8, 2025

mssonicbld mentioned this pull request Jul 8, 2025

Fix buffer queue cnt test to account for different BUFFER_QUEUE configs #19310

Merged

11 tasks

kperumalbfn approved these changes Jul 15, 2025

View reviewed changes

kperumalbfn merged commit f35fc20 into sonic-net:202411 Jul 15, 2025
11 of 14 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[action] [PR:19310] Fix buffer queue cnt test to account for different BUFFER_QUEUE configs#19466

[action] [PR:19310] Fix buffer queue cnt test to account for different BUFFER_QUEUE configs#19466
kperumalbfn merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/19310

mssonicbld commented Jul 8, 2025

Uh oh!

mssonicbld commented Jul 8, 2025

Uh oh!

mssonicbld commented Jul 8, 2025

Uh oh!

azure-pipelines bot commented Jul 8, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

mssonicbld commented Jul 8, 2025

Description of PR

Type of change

Back port request

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

Uh oh!

mssonicbld commented Jul 8, 2025

Uh oh!

mssonicbld commented Jul 8, 2025

Uh oh!

azure-pipelines bot commented Jul 8, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants