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

chore: Clean up data configuration #9973

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

chore: Clean up data configuration #9973

wants to merge 32 commits into from

Conversation

PhilWindle
Copy link
Collaborator

@PhilWindle PhilWindle commented Nov 14, 2024

This PR changes the following related to data storage configuration

  • PXE_DATA_DIRECTORY has been removed. The PXE uses DATA_DIRECTORY along with everything else.
  • DATA_DIRECTORY is deemed to be the root of all state directories. This was already partly implemented. Each component stores it's state in one or more sub-directories of this root.
  • DATA_STORE_MAP_SIZE_KB is a new environment variable and represented the mmap size provided to all environments of LMDB.
  • WS_DATA_DIRECTORY is a new optional environment variable. If provided, represents an alternative root directory for the world state only.
  • WS_DB_MAP_SIZE_KB is a new optional environment variable. If provided, represent the mmap size provided to all LMDB environment used in the world state.

@PhilWindle PhilWindle added the e2e-all CI: Enables this CI job. label Nov 14, 2024
@PhilWindle PhilWindle changed the base branch from master to pw/world-state-cleanup November 15, 2024 09:28
@PhilWindle PhilWindle marked this pull request as ready for review November 15, 2024 09:47
Base automatically changed from pw/world-state-cleanup to master November 15, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e-all CI: Enables this CI job.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants