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

fix: add proper labels to the Pooler deployment #3350

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

armru
Copy link
Member

@armru armru commented Nov 15, 2023

This patch ensures that we add the proper labels to the deployment. In a previous patch we ensured that the labels were present on the pods but not on the actual deployment.

@armru armru added the no-issue label Nov 15, 2023
@armru armru changed the title fix: add proper labels to Pooler deployment fix: add proper labels to the Pooler deployment Nov 15, 2023
@github-actions github-actions bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.20 release-1.21 labels Nov 15, 2023
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

@armru armru force-pushed the dev/add-label-to-pooler-deployment branch from 2d4e492 to a5f0979 Compare November 15, 2023 13:19
@armru
Copy link
Member Author

armru commented Nov 15, 2023

/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/6877861716

@baurmatt
Copy link
Contributor

This sounds like it fixes the problem described in #3339. There is also #3346 which introduces dedicated app labels for PostgreSQL and PGBouncer.

This patch ensures that we add the proper labels to the deployment.
In a previous patch we ensured that the labels were present on the pods but not on the actual deployment.

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
@sxd sxd force-pushed the dev/add-label-to-pooler-deployment branch from a5f0979 to cfb89d3 Compare November 17, 2023 20:34
@sxd
Copy link
Member

sxd commented Nov 17, 2023

/ok-to-merge just a dangling test

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Nov 17, 2023
@sxd sxd merged commit c295a7f into main Nov 17, 2023
31 of 32 checks passed
@sxd sxd deleted the dev/add-label-to-pooler-deployment branch November 17, 2023 20:49
cnpg-bot pushed a commit that referenced this pull request Nov 17, 2023
This patch ensures that we add the proper labels to the deployment. In a
previous patch we ensured that the labels were present on the pods but
not on the actual deployment.

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit c295a7f)
cnpg-bot pushed a commit that referenced this pull request Nov 17, 2023
This patch ensures that we add the proper labels to the deployment. In a
previous patch we ensured that the labels were present on the pods but
not on the actual deployment.

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit c295a7f)
ceriath pushed a commit to ceriath/cloudnative-pg that referenced this pull request Nov 23, 2023
This patch ensures that we add the proper labels to the deployment. In a
previous patch we ensured that the labels were present on the pods but
not on the actual deployment.

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
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 no-issue ok to merge 👌 This PR can be merged release-1.20 release-1.21
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants