Skip to content

chore: add webhook validation for shared_buffers - #2840

Merged
litaocdl merged 5 commits into
mainfrom
dev/2790
Sep 26, 2023
Merged

litaocdl merged 5 commits into
mainfrom
dev/2790

Conversation

@armru

@armru armru commented Sep 21, 2023 •

Copy link
Copy Markdown
Member

Partially Closes #2664

@github-actions github-actions Bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.19 labels Sep 21, 2023
@github-actions

Copy link
Copy Markdown
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

@armru
armru force-pushed the dev/2790 branch 2 times, most recently from c8acc68 to 4329d12 Compare September 21, 2023 14:16
@armru

armru commented Sep 21, 2023

Copy link
Copy Markdown
Member Author

/test limit=local

@github-actions

Copy link
Copy Markdown
Contributor

Comment thread api/v1/cluster_webhook.go Resolved
@litaocdl

Copy link
Copy Markdown
Collaborator

/test limit=local

@github-actions

Copy link
Copy Markdown
Contributor

Comment thread api/v1/cluster_webhook.go Resolved
Comment thread api/v1/cluster_webhook_test.go Resolved
@litaocdl

Copy link
Copy Markdown
Collaborator

/ok-to-merge

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Sep 25, 2023
armru and others added 4 commits September 26, 2023 10:10
Partially Closes #2790

Signed-off-by: Armando Ruocco <[email protected]>
Signed-off-by: Armando Ruocco <[email protected]>
Signed-off-by: Gabriele Bartolini <[email protected]>
@litaocdl

Copy link
Copy Markdown
Collaborator

/ok-to-merge

Signed-off-by: Tao Li <[email protected]>
@litaocdl
litaocdl merged commit 791d48a into main Sep 26, 2023
@litaocdl
litaocdl deleted the dev/2790 branch September 26, 2023 03:48
cnpg-bot pushed a commit that referenced this pull request Sep 26, 2023
add webhook validation for `shared_buffers`

Signed-off-by: Armando Ruocco <[email protected]>
Signed-off-by: Gabriele Bartolini <[email protected]>
Signed-off-by: Tao Li <[email protected]>
Co-authored-by: Gabriele Bartolini <[email protected]>
Co-authored-by: Tao Li <[email protected]>
(cherry picked from commit 791d48a)
cnpg-bot pushed a commit that referenced this pull request Sep 26, 2023
add webhook validation for `shared_buffers`

Signed-off-by: Armando Ruocco <[email protected]>
Signed-off-by: Gabriele Bartolini <[email protected]>
Signed-off-by: Tao Li <[email protected]>
Co-authored-by: Gabriele Bartolini <[email protected]>
Co-authored-by: Tao Li <[email protected]>
(cherry picked from commit 791d48a)
Lennart01 pushed a commit to Lennart01/cloudnative-pg that referenced this pull request Sep 30, 2023
add webhook validation for `shared_buffers`

Signed-off-by: Armando Ruocco <[email protected]>
Signed-off-by: Gabriele Bartolini <[email protected]>
Signed-off-by: Tao Li <[email protected]>
Co-authored-by: Gabriele Bartolini <[email protected]>
Co-authored-by: Tao Li <[email protected]>
Signed-off-by: Lennart Krauch <[email protected]>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CNPG should validate memory resources settings against the shared_memory Postgres parameter

5 participants