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: prevent fenced instances from using all the CPUs #4625

Merged
merged 1 commit into from
May 22, 2024

Conversation

leonardoce
Copy link
Contributor

After receiving the exit status, this patch clears up the postmaster error channel.
This avoids entering into a continuous spin loop.

Closes: #4623

This patch clears up the postmaster error channel after
having received the exit status. This avoids entering
in a continuous spin loop.

Closes: cloudnative-pg#4623

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
@leonardoce leonardoce requested a review from a team as a code owner May 21, 2024 15:11
@github-actions github-actions bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.21 release-1.22 release-1.23 labels May 21, 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

@jsilvela
Copy link
Collaborator

/test limit=local

Copy link
Contributor

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

@jsilvela
Copy link
Collaborator

/ok-to-merge

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label May 22, 2024
@leonardoce leonardoce merged commit 4826a96 into cloudnative-pg:main May 22, 2024
29 of 31 checks passed
cnpg-bot pushed a commit that referenced this pull request May 22, 2024
After receiving the exit status, this patch clears up the postmaster
error channel.
This avoids entering into a continuous spin loop.

Closes: #4623

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
(cherry picked from commit 4826a96)
cnpg-bot pushed a commit that referenced this pull request May 22, 2024
After receiving the exit status, this patch clears up the postmaster
error channel.
This avoids entering into a continuous spin loop.

Closes: #4623

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
(cherry picked from commit 4826a96)
cnpg-bot pushed a commit that referenced this pull request May 22, 2024
After receiving the exit status, this patch clears up the postmaster
error channel.
This avoids entering into a continuous spin loop.

Closes: #4623

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
(cherry picked from commit 4826a96)
dougkirkley pushed a commit to dougkirkley/cloudnative-pg that referenced this pull request Jun 11, 2024
…#4625)

After receiving the exit status, this patch clears up the postmaster
error channel.
This avoids entering into a continuous spin loop.

Closes: cloudnative-pg#4623

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
Signed-off-by: Douglass Kirkley <dkirkley@eitccorp.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 ok to merge 👌 This PR can be merged ready to test 🐘 release-1.21 release-1.22 release-1.23
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: instance manager is continuously spinning, using 100% of the CPU when fencing is on
5 participants