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

store-db add more appropriate db settings for current db hw #37

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

Ivansete-status
Copy link
Contributor

After one week of larger hardware in status.prod's db hosts (8 GB RAM, 4 CPUs) we still don't have an evident performance enhancement.
This PR aims at getting the major profit from the current hardware in status.prod.

We still have high swap level and the RAM is not completely used:
image
image

⚠️ these new settings might not be suitable for current status.staging's hardware.


ℹ️
I attach the stats before and after increasing the hw
stats-prev.csv
stats.csv

Signed-off-by: Ivan Folgueira Bande <ivansete@status.im>
@apentori
Copy link
Contributor

apentori commented Aug 7, 2024

If those change are only for status.prod,we need to load them depending on the env.
I will modify the PR later for that.

@apentori
Copy link
Contributor

apentori commented Aug 7, 2024

How do you want us to apply the change ? Only to one db / all at once ?

@Ivansete-status
Copy link
Contributor Author

How do you want us to apply the change ? Only to one db / all at once ?

All at once. I can compare them with previous performance

Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would make sense to make those settings dependent on available memory on the host rather than hardcoding them.

@apentori
Copy link
Contributor

apentori commented Aug 8, 2024

Applied in status.prod

Signed-off-by: Alexis Pentori <alexis@status.im>
@Ivansete-status
Copy link
Contributor Author

Thanks @apentori 🥳 ! The issue is completed successfully.

The nodes are swapping less now.

Before the change:
image

After the change:
image

@apentori apentori merged commit af9162f into master Aug 8, 2024
2 checks passed
@apentori apentori deleted the adapt-pg-settings-status branch August 8, 2024 12:26
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

Successfully merging this pull request may close these issues.

3 participants