sonic-mgmt icon indicating copy to clipboard operation
sonic-mgmt copied to clipboard

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

Open echuawu opened this issue 1 year ago • 2 comments

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 avatar Jan 26 '24 02:01 echuawu

@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

liat-grozovik avatar Apr 16 '24 16:04 liat-grozovik

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

bingwang-ms avatar Apr 16 '24 17:04 bingwang-ms

@ZhaohuiS Kindly ping on this issue.

bingwang-ms avatar Jul 23 '24 15:07 bingwang-ms

Waiting for https://github.com/sonic-net/sonic-mgmt/pull/13698#pullrequestreview-2185315241 merged, then will use this fixture

ZhaohuiS avatar Jul 24 '24 00:07 ZhaohuiS