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

Add separate PVC for wal #43

Open
wants to merge 11 commits into
base: v0.7.0-dev
Choose a base branch
from
Open

Add separate PVC for wal #43

wants to merge 11 commits into from

Conversation

RafiaSabih
Copy link
Contributor

Define WalPvc filed with the name of the pvc and the volume specifications to create a separate pvc for wal files.

Define WalPvc filed with the name of the pvc and the volume specifications
to create a separate pvc for wal files.
@RafiaSabih RafiaSabih self-assigned this Jun 14, 2024
@RafiaSabih RafiaSabih requested a review from Schmaetz June 14, 2024 12:42
@Schmaetz Schmaetz added this to the v0.8.0 milestone Jun 18, 2024
Copy link
Collaborator

@Schmaetz Schmaetz left a comment

Choose a reason for hiding this comment

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

To ensure that the Containers moving already existing wal-files, we need informations about the old wal-dir-path.
For example, on an existing cluster you modify the manifest to add an additional wal_dir.
old-wal-dir-ENV can be empty
New wal-Dir-ENV ,map to new volume

If you remove the additional wal-volume from a cluster
old-wal-dir-ENV map to the volume
New wal-Dir-ENV , can be empty

We can use this ENVs on the Container-Project to allow complete customized wal-path.

rafia sabih added 7 commits August 1, 2024 10:20
User provides only the volume specifics for the PVC.
The name of the volume is pre-configured as walpvc-cluster-name.
The volume is mounted at constants.PostgresPVCWalMount.
When using separate PVc for wal, USE_WAL_PVC is set otherwise False.
Use the path in PVCWALDIR for moving wal only when USE_WAL_PVC is set.
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