Skip to content

fix: eliminate redundant Cluster status updates with image catalog #6277

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.

Sign up for GitHub

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

Jump to bottom
Merged
mnencia merged 1 commit into from
Dec 5, 2024
Merged

fix: eliminate redundant Cluster status updates with image catalog #6277

mnencia merged 1 commit into from
Dec 5, 2024

Conversation

@litaocdl
Copy link
Collaborator

@litaocdl litaocdl commented Dec 5, 2024 •
edited by mnencia
Loading

This patch resolves the issue of redundant Cluster status updates triggered when the image catalog is enabled.

Closes: #6276

Release Notes

  • Resolved an issue causing redundant Cluster status updates when an image catalog is enabled

@litaocdl litaocdl requested a review from a team as a code owner December 5, 2024 08:04
@cnpg-bot cnpg-bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.22 release-1.23 release-1.24 labels Dec 5, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 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

@litaocdl litaocdl changed the title fix: only trigger patch status when image is different fix: only trigger cluster status patch when needed Dec 5, 2024
@litaocdl @mnencia
fix: only trigger patch status when image is different
993ef89 
Signed-off-by: Tao Li <tao.li@enterprisedb.com>
@mnencia
Copy link
Member

mnencia commented Dec 5, 2024

/test d=push tl=4

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2024

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

@mnencia mnencia changed the title fix: only trigger cluster status patch when needed fix(image-catalog): avoid unneeded status patches when image catalog is in use Dec 5, 2024
@mnencia mnencia changed the title fix(image-catalog): avoid unneeded status patches when image catalog is in use fix: eliminate redundant status updates when using image catalog Dec 5, 2024
@mnencia mnencia changed the title fix: eliminate redundant status updates when using image catalog fix: eliminate redundant Cluster status updates with image catalog Dec 5, 2024
@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Dec 5, 2024
@mnencia mnencia merged commit 7dcb199 into main Dec 5, 2024
@mnencia mnencia deleted the dev/6276 branch December 5, 2024 12:19
cnpg-bot pushed a commit that referenced this pull request Dec 5, 2024
@litaocdl
fix: eliminate redundant Cluster status updates with image catalog (#…
2d5bd7e 
…6277)

This patch resolves the issue of redundant Cluster status updates
triggered when the image catalog is enabled.

Closes: #6276

Signed-off-by: Tao Li <tao.li@enterprisedb.com>
(cherry picked from commit 7dcb199)
cnpg-bot pushed a commit that referenced this pull request Dec 5, 2024
@litaocdl
fix: eliminate redundant Cluster status updates with image catalog (#…
6e44498 
…6277)

This patch resolves the issue of redundant Cluster status updates
triggered when the image catalog is enabled.

Closes: #6276

Signed-off-by: Tao Li <tao.li@enterprisedb.com>
(cherry picked from commit 7dcb199)
@ardentperf ardentperf mentioned this pull request Sep 8, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mnencia mnencia mnencia approved these changes

@fcanovai fcanovai fcanovai approved these changes

Assignees

No one assigned

Labels

backport-requested ◀️ This pull request should be backported to all supported releases ok to merge 👌 This PR can be merged release-1.23 release-1.24

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: Empty patch was executed for each reconciling loop for imageCatalog

4 participants

@litaocdl @mnencia @fcanovai @cnpg-bot