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

Implement a Memory Leak detection Test #2260

Open
AlfredoG87 opened this issue Mar 28, 2024 · 0 comments
Open

Implement a Memory Leak detection Test #2260

AlfredoG87 opened this issue Mar 28, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@AlfredoG87
Copy link
Collaborator

Problem

Sometimes changes to the codebase of the relay, as well as upgrades of libraries that are used within the relay, could provoke a memory leak issue. that is not apparent on the tests until we move the version to production environments.
That makes it harder to identify, since the release contains many changes, all of wich could be the source of the problem.

Solution

Ideally we need a Test or set of tests that are designed to find and detect memory issues or leaks faster, that we can run on every PR, or at least on every release.

Alternatives

We have worked on a POC that uses node-memwatch to log the differential of the memory heap after each request is made.
see here: #2253

maybe we can extend this solution to perform many requests, simulating load, and then take the diff on heap. compare those results with a baseline and raise a flag in case there is something that changes too drastically.

@AlfredoG87 AlfredoG87 added the enhancement New feature or request label Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Sprint Backlog
Development

No branches or pull requests

1 participant