elastic / elastic-agent-system-metrics

Apache License 2.0
0 stars 22 forks source link

add PssFreeSnapshot #115

Closed leehinman closed 7 months ago

leehinman commented 7 months ago

What does this PR do?

Frees the memory that is allocated when PssCaptureSnapshot is called.

Why is it important?

memory leak

Checklist

Author's Checklist

Related issues

Testing

nimarezainia commented 7 months ago

Thank you @leehinman

amolnater-qasource commented 7 months ago

Hi @pierrehilbert We have revalidated this PR on latest 8.11.0 BC1 kibana cloud environment and found it working fine :

Observations:

Build details: VERSION: 8.11.2 BC1 BUILD: 68299 COMMIT: 92746356b61c3e3ac62b6d7045727f8d737fa4b5 Artifact link: https://staging.elastic.co/8.11.2-c2650cf6/downloads/beats/elastic-agent/elastic-agent-8.11.2-windows-x86_64.zip

Screen Recording:

https://github.com/elastic/elastic-agent-system-metrics/assets/77374876/fce155e0-a4db-4d66-9362-37e1e7bf1c32

https://github.com/elastic/elastic-agent-system-metrics/assets/77374876/37bf9077-7a61-446e-9039-a19f1f4d3ea5

https://github.com/elastic/elastic-agent-system-metrics/assets/77374876/a6a15e4c-9290-4e87-9f1c-e180b53f75d6

Hence we are marking this as [QA:Validated].

Please let us know if we are missing anything here. Thanks!