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

chore: avoid hitting the API server when updating plugins status #4425

Merged
merged 1 commit into from
May 19, 2024

Conversation

leonardoce
Copy link
Contributor

When not needed, this patch avoids hitting the API server with a PATCH request.

@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 2, 2024
Copy link
Contributor

github-actions bot commented May 2, 2024

❗ 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

@sxd sxd added the no-issue label May 18, 2024
When not needed, this patch avoids hitting the API server with a
PATCH request.

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
@sxd sxd requested a review from a team as a code owner May 18, 2024 19:56
@mnencia
Copy link
Member

mnencia commented May 18, 2024

/test limit=local tl=4

Copy link
Contributor

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

@sxd
Copy link
Member

sxd commented May 19, 2024

/ok-to-merge some test failed but it's not related at all

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label May 19, 2024
@sxd sxd merged commit cc62f8f into cloudnative-pg:main May 19, 2024
27 of 28 checks passed
cnpg-bot pushed a commit that referenced this pull request May 19, 2024
When not needed, this patch avoids hitting the API server with a PATCH
request.

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
(cherry picked from commit cc62f8f)
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.23
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants