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

High resource usage for docker containers #2779

Closed
evan-forbes opened this issue Oct 27, 2023 · 2 comments
Closed

High resource usage for docker containers #2779

evan-forbes opened this issue Oct 27, 2023 · 2 comments
Labels
bug Something isn't working investigation item tracks efforts related to an investigation. does not always require a PR to close.

Comments

@evan-forbes
Copy link
Member

It has been observed by one of the community members that docker containers can result in using large amounts of ram. We should investigate this.

@evan-forbes evan-forbes added bug Something isn't working docker investigation item tracks efforts related to an investigation. does not always require a PR to close. labels Oct 27, 2023
@evan-forbes
Copy link
Member Author

evan-forbes commented Nov 6, 2023

after further discussion with users and some experiments a large portion of this seems to be very related to the spam that was done on mocha and only for nodes that have the tx-indexer on.

This was found by comparing one node that was not experiencing the issue, and another that was in pyroscope to examine what was eating up so much ram. The node that had the tx-indexer enabled and synced from scratch (therefore including the spam in the tx-index). The node that did not experience the issue synced from scratch, but didn't enable the tx-index until after the period where the chain was spammed.

after this analysis I feel comfortable shutting down this issue. I do however think this issue emphasizes the need to switch away from something like the default tx-indexer, or alter the schema of tx storage

@rootulp
Copy link
Collaborator

rootulp commented Nov 6, 2023

Note: celestia-app overrides the default config so that the tx indexer is disabled. Therefore, we only expect this issue for nodes that explicitly enable the tx indexer.

Ref: https://github.com/celestiaorg/celestia-app/blame/e5e98d8f3fd5a1a8ee2b6b1751a12bb2af1eaf5f/app/default_overrides.go#L252

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working investigation item tracks efforts related to an investigation. does not always require a PR to close.
Projects
None yet
Development

No branches or pull requests

2 participants