Skip to content

[conflict resolve]test_snmp_queue_counters.py/test_telemetry.py config_reload and snmpwwalk output time delay fix, test_snmp_queue_counters.py multi-asic KeyError fix#15735

Merged
yejianquan merged 2 commits intosonic-net:202405from
harjotsinghpawra:202405_snmp_resolve
Nov 27, 2024
Merged

[conflict resolve]test_snmp_queue_counters.py/test_telemetry.py config_reload and snmpwwalk output time delay fix, test_snmp_queue_counters.py multi-asic KeyError fix#15735
yejianquan merged 2 commits intosonic-net:202405from
harjotsinghpawra:202405_snmp_resolve

Conversation

@harjotsinghpawra
Copy link
Copy Markdown
Contributor

Description of PR

Cherry pick PR #15688 for 202405
Resolve conflit issues -

This PR also resolves following issues along with ones mentioned in above PR -
Fixed multi-asic issues where right buffer queues were not selected
asic namespace should not be checked in queue name

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

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

@harjotsinghpawra harjotsinghpawra marked this pull request as ready for review November 26, 2024 03:21
Copy link
Copy Markdown
Collaborator

@vperumal vperumal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vperumal
Copy link
Copy Markdown
Collaborator

FYI @abdosi @yejianquan - Kindly help review and merge

@yejianquan yejianquan merged commit 1990a5f into sonic-net:202405 Nov 27, 2024
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

Successfully merging this pull request may close these issues.

4 participants