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

[test_tunnel_memory_leak] The memory leak check mechanism need to be updated #11403

Open
echuawu opened this issue Jan 26, 2024 · 2 comments
Open

Comments

@echuawu
Copy link
Contributor

echuawu commented Jan 26, 2024

Description
The test logic is flowed. The test tries to trigger tunnel packet handler and check the memory increase. However it checks swss docker using "docker state swss --no-stream". This considers the memory of swss docker + all processes. Since the while docker is considered there might be some temporal memory increased due to usage by docker subsystem as you can see in the running log.
Overall, the test needs to be more deterministic in figuring out the memory leak and not use very generic mechanism as used today.

Steps to reproduce the issue:

  1. Run the script at dualtor-64 topology.

Describe the results you received:

  1. Memory usage exceeds the threshold

Describe the results you expected:

  1. The test needs to be more deterministic in figuring out the memory leak and not use very generic mechanism as used today.

Additional information you deem important:

**Output of `show version`:**

```
(paste your output here)
```

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
echuawu added a commit to echuawu/sonic-mgmt that referenced this issue Feb 7, 2024
Skip it by github issue sonic-net#11403 on dualtor-64 topology

Change-Id: I767b31b8c62533fae216e9e8d67ddf8cd4bfca51
bingwang-ms pushed a commit that referenced this issue Feb 12, 2024
Skip it by github issue #11403 on dualtor-64 topology

Change-Id: I767b31b8c62533fae216e9e8d67ddf8cd4bfca51
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this issue Mar 22, 2024
Skip it by github issue sonic-net#11403 on dualtor-64 topology

Change-Id: I767b31b8c62533fae216e9e8d67ddf8cd4bfca51
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this issue Mar 22, 2024
Skip it by github issue sonic-net#11403 on dualtor-64 topology

Change-Id: I767b31b8c62533fae216e9e8d67ddf8cd4bfca51
mssonicbld pushed a commit that referenced this issue Mar 22, 2024
Skip it by github issue #11403 on dualtor-64 topology

Change-Id: I767b31b8c62533fae216e9e8d67ddf8cd4bfca51
echuawu added a commit to echuawu/sonic-mgmt that referenced this issue Mar 22, 2024
Skip it by github issue sonic-net#11403 on dualtor-64 topology

Change-Id: Iaba470f9f0c8dd0ac295d3a81118d443345bcbe5
@liat-grozovik
Copy link
Collaborator

@yxieca are you aware of someone going to fix this issue? no assignee.
if not, i believe we should go a head and merge to skip based on github issue

@bingwang-ms
Copy link
Collaborator

@ZhaohuiS Can you please help take a look at this issue?

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

No branches or pull requests

3 participants