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

Define resources for common components: NodeSet, Blockchain, PostgreSQL #1590

Merged
merged 6 commits into from
Jan 21, 2025

Conversation

skudasov
Copy link
Collaborator

@skudasov skudasov commented Jan 21, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes introduce a new section in the documentation for Components Resources and implement resource limits for containers in various blockchain components, Chainlink nodes, and PostgreSQL components. This ensures efficient resource usage and prevents any single component from monopolizing system resources, improving overall system stability and performance.

What

  • Added Components Resources section to the documentation.
  • Set cgroups resource for containers in the changeset v0.4.7.
  • Added resource limits for containers in blockchain components (anvil.go, aptos.go, besu.go, geth.go, solana.go, sui.go, tron.go), Chainlink nodes (clnode.go), and PostgreSQL components (postgres.go).
  • Introduced ContainerResources structure to define CPU and memory limits for containers.
  • Implemented ResourceLimitsFunc to apply resource limits based on ContainerResources.
  • Added resource limits configuration in examples (smoke.toml, smoke_limited_resources.toml).

@skudasov skudasov requested review from sebawo and a team as code owners January 21, 2025 13:28
@skudasov skudasov changed the title control resources for common components: NodeSet, Blockchain, PostgreSQL Define resources for common components: NodeSet, Blockchain, PostgreSQL Jan 21, 2025
framework/docker.go Outdated Show resolved Hide resolved
framework/docker.go Outdated Show resolved Hide resolved
@cl-sonarqube-production
Copy link

@skudasov skudasov merged commit 9c5de77 into main Jan 21, 2025
58 of 59 checks passed
@skudasov skudasov deleted the chaos-resources branch January 21, 2025 16:54
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.

2 participants