Skip to content

Update test_memory_checker to support gnmi and telemetry containers#10748

Merged
zbud-msft merged 7 commits intosonic-net:masterfrom
zbud-msft:update_memory_checker_test
Nov 21, 2023
Merged

Update test_memory_checker to support gnmi and telemetry containers#10748
zbud-msft merged 7 commits intosonic-net:masterfrom
zbud-msft:update_memory_checker_test

Conversation

@zbud-msft
Copy link
Contributor

@zbud-msft zbud-msft commented Nov 15, 2023

Description of PR

Summary:
Fixes # (issue)25833282

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305

Approach

What is the motivation for this PR?

Update test_memory_checker to dynamically check for gnmi or telemetry container

How did you do it?

Add logic to create correct container name fixture

How did you verify/test it?

Pipeline

Any platform specific information?

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

Documentation

@sonic-net sonic-net deleted a comment from mssonicbld Nov 15, 2023
@zbud-msft zbud-msft marked this pull request as ready for review November 15, 2023 19:35
@zbud-msft zbud-msft requested a review from ganglyu November 16, 2023 17:23
@mssonicbld
Copy link
Collaborator

@zbud-msft PR conflicts with 202311 branch

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #11463

StormLiangMS pushed a commit that referenced this pull request Feb 1, 2024
…10748) (#11463)

What is the motivation for this PR?
Update test_memory_checker to dynamically check for gnmi or telemetry container

How did you do it?
Add logic to create correct container name fixture

How did you verify/test it?
Pipeline

Any platform specific information?
Supported testbed topology if it's a new test case?
@zbud-msft
Copy link
Contributor Author

Already included in 202311

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants