Skip to content

[regression] fix regression in test_memory_exhaustion.py #9320

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

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

wenyiz2021
Copy link
Contributor

@wenyiz2021 wenyiz2021 commented Aug 7, 2023

Description of PR

#9075 introduced new function which has 3 arguments, caller only give 2.
Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

verified on T0 device:

PASSED platform_tests/test_memory_exhaustion.py::TestMemoryExhaustion::test_memory_exhaustion[xxxacs-10]
================== 1 passed, 218 warnings in 1033.24 seconds ===================

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

@wenyiz2021 PR conflicts with 202305 branch

mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 8, 2023
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #9330

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #9339

@lizhijianrd
Copy link
Contributor

@wenyiz2021 PR conflicts with 202305 branch

Solved this conflict.

@wenyiz2021 wenyiz2021 deleted the memory_checker branch August 8, 2023 18:14
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.

5 participants