Skip to content

fix(pooler): apply resources to the bootstrap-controller#7922

Merged
gbartolini merged 1 commit intomainfrom
dev/7822
Jul 21, 2025
Merged

fix(pooler): apply resources to the bootstrap-controller#7922
gbartolini merged 1 commit intomainfrom
dev/7822

Conversation

@armru
Copy link
Member

@armru armru commented Jun 27, 2025

The resources specified in the Pooler template now correctly apply to the init bootstrap-controller container.

Closes #7822

@armru armru requested a review from a team as a code owner June 27, 2025 10:23
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 27, 2025
@cnpg-bot cnpg-bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.22 release-1.25 release-1.26 labels Jun 27, 2025
@github-actions
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

@dosubot dosubot bot added the bug 🐛 Something isn't working label Jun 27, 2025
@armru
Copy link
Member Author

armru commented Jun 27, 2025

/test limit=local depth=push tl=4

@github-actions
Copy link
Contributor

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

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Jun 27, 2025
@armru armru added this to the 1.27.0 milestone Jul 10, 2025
@mnencia mnencia force-pushed the dev/7822 branch 2 times, most recently from 3dab1bd to 9ab5bfc Compare July 18, 2025 14:13
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 18, 2025
@gbartolini gbartolini modified the milestones: 1.27.0, 1.26.1 Jul 21, 2025
@gbartolini gbartolini removed this from the 1.26.1 milestone Jul 21, 2025
@gbartolini gbartolini self-requested a review July 21, 2025 15:51
…r` init container

Closes #7822

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
@gbartolini gbartolini merged commit b97531e into main Jul 21, 2025
10 of 13 checks passed
@gbartolini gbartolini deleted the dev/7822 branch July 21, 2025 15:53
mnencia pushed a commit that referenced this pull request Jul 21, 2025
The resources specified in the `Pooler` template now correctly apply to
the init `bootstrap-controller` container.

Closes #7822

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit b97531e)
mnencia pushed a commit that referenced this pull request Jul 21, 2025
The resources specified in the `Pooler` template now correctly apply to
the init `bootstrap-controller` container.

Closes #7822

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit b97531e)
@chris-gooch
Copy link

Hi @armru ive got a strange issue with this. One eks cluster that deployed using manifests is setting bootstrap-controller resources as 0:0.

Another EKS cluster that deployed using helm is setting bootstrap-controller as 1000:2000 even when I try with my earlier workaround to force the container resources.

Any clues how to debug this?

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 bug 🐛 Something isn't working lgtm This PR has been approved by a maintainer ok to merge 👌 This PR can be merged release-1.25 release-1.26 size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Pooler deployments not setting initContainer resources to be the same as pgbouncer

6 participants