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

Memory leak after upgrade to 1.38.0 #47905

Open
vad opened this issue Feb 5, 2024 · 8 comments
Open

Memory leak after upgrade to 1.38.0 #47905

vad opened this issue Feb 5, 2024 · 8 comments

Comments

@vad
Copy link

vad commented Feb 5, 2024

Hi, we're experiencing memory issues after migrating from pushpin 1.37.0 to 1.38.0. The only other change we had was removing zurl from the configuration, pushpin was not starting otherwise.

image

This is the memory usage of the development setup, but we see the same pattern in production. You can clearly see when we upgraded to latest pushpin version. Dotted lines are k8s memory request/limit, we had to increase them.

Apparently memory increase is not determined by traffic: we have low traffic during the night and over the weekend, and very low traffic in the development setup.

@jkarneges
Copy link
Member

Thanks for the report. Will look into this and report back.

Regarding zurl: is it possible you didn't have zurl installed? It's deprecated, but it should have worked if you had it installed and was referenced by your configuration, so I'm wondering what happened there. Maybe you used our updated Dockerfile (which no longer installs zurl) but with your own pushpin.conf?

@vad
Copy link
Author

vad commented Feb 6, 2024

zurl: exactly. I think I took pushpin.conf from the 1.37 docker a few months ago in order to customize the prometheus_port option, and zurl was there.

@vad
Copy link
Author

vad commented Feb 6, 2024

BTW I'm aware the report is short and probably incomplete. Let me know if I can provide more information

@vad
Copy link
Author

vad commented Feb 7, 2024

image

From left to right:

  • orange: pushpin 1.38.0
  • blue: pushpin 1.37.0 with zurl
  • red: pushpin 1.37.0 without zurl

@jkarneges
Copy link
Member

1.39.0 docker image was posted. It includes a memory leak fix, which was hopefully this.

@vad
Copy link
Author

vad commented Jun 27, 2024

I just updated to pushpin 1.39.1 the development instance:

image

Baseline usage increased compared to 1.37.0 but that was expected (RIP zurl). Memory leak looks solved.

Thank you

@vad vad closed this as completed Jun 27, 2024
@vad
Copy link
Author

vad commented Jul 1, 2024

apparently there still is a leak:

image

@vad vad reopened this Jul 1, 2024
@vad
Copy link
Author

vad commented Sep 17, 2024

image

I confirm there's still a leak.

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

No branches or pull requests

2 participants