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

feat: annotation to disable WAL archiving #4055

Merged
merged 8 commits into from Mar 11, 2024

Conversation

leonardoce
Copy link
Contributor

@leonardoce leonardoce commented Mar 10, 2024

Introduce the cnpg.io/skipWalArchiving annotation to disable WAL archiving when set to enabled.

Closes: #3774

@github-actions github-actions bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.21 release-1.22 labels Mar 10, 2024
Copy link
Contributor

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@leonardoce leonardoce marked this pull request as ready for review March 10, 2024 16:00
@leonardoce
Copy link
Contributor Author

/test limit=local

Copy link
Contributor

@leonardoce, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/8223295982

@gbartolini gbartolini changed the title feat: allow setting archive_mode=off using an annotation feat: allow disabling WAL archiving via annotation Mar 10, 2024
@gbartolini
Copy link
Contributor

I have been able to disable WAL archiving and reduce WAL footprint with the following example:

apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: cluster-example
  annotations:
    cnpg.io/skipWalArchiving: "enabled"
spec:
  instances: 1

  storage:
    size: 1Gi
  postgresql:
    parameters:
      wal_level: minimal
      max_wal_senders: "0"

@armru
Copy link
Member

armru commented Mar 11, 2024

/test limit=local

Copy link
Contributor

@armru, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/8233800341

@armru
Copy link
Member

armru commented Mar 11, 2024

/ok-to-merge

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Mar 11, 2024
@gbartolini gbartolini changed the title feat: allow disabling WAL archiving via annotation feat: annotation to disable WAL archiving Mar 11, 2024
Closes: cloudnative-pg#3774

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
leonardoce and others added 6 commits March 11, 2024 16:56
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enteprisedb.com>
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
@gbartolini gbartolini merged commit 852ee34 into cloudnative-pg:main Mar 11, 2024
6 checks passed
cnpg-bot pushed a commit that referenced this pull request Mar 11, 2024
Introduce the `cnpg.io/skipWalArchiving` annotation to disable WAL
archiving when set to `enabled`.

Closes: #3774

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enteprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit 852ee34)
mnencia pushed a commit that referenced this pull request Mar 11, 2024
Introduce the `cnpg.io/skipWalArchiving` annotation to disable WAL
archiving when set to `enabled`.

Closes: #3774

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@gmail.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enteprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit 852ee34)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-requested ◀️ This pull request should be backported to all supported releases ok to merge 👌 This PR can be merged release-1.21 release-1.22
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: Enable configuration of archive_mode
4 participants