Skip to content

use Reader to get EDB configmap#2593

Merged
Daniel-Fan merged 1 commit intoIBM:masterfrom
qpdpQ:patch_EDB_image
Jul 25, 2025
Merged

use Reader to get EDB configmap#2593
Daniel-Fan merged 1 commit intoIBM:masterfrom
qpdpQ:patch_EDB_image

Conversation

@qpdpQ
Copy link
Contributor

@qpdpQ qpdpQ commented Jul 25, 2025

What this PR does / why we need it:
Side effect of cache upgrade, because cloud-native-postgresql-operand-images-config doesn't contain watchByCSoperatorLabel we can't use client to get it from cache, use reader instead
Which issue(s) this PR fixes:
Fixes # https://github.ibm.com/IBMPrivateCloud/roadmap/issues/67248

Signed-off-by: Allen Li <liyuchen223@gmail.com>
@qpdpQ
Copy link
Contributor Author

qpdpQ commented Jul 25, 2025

Able to update image in cluster cr

I0725 16:49:25.708449 1 init.go:2636] Postgres Cluster CR common-service-db image is not updated, waiting for update to the desired image in configmap openshift-operators/cloud-native-postgresql-operand-images-config
W0725 16:49:35.683219 1 init.go:2640] Failed to wait for Postgres Cluster CR common-service-db image update: context deadline exceeded
I0725 16:49:35.683286 1 init.go:2642] Updating Postgres Cluster CR common-service-db image to the desired image icr.io/cpopen/edb/postgresql:16.9-5.16.0@sha256:e9e408a13bd103fb46536f896c51e7a27cba38d9b224231b20328a28efe1e616 in configmap openshift-operators/cloud-native-postgresql-operand-images-config
I0725 16:49:35.723754 1 init.go:2653] Postgres Cluster CR common-service-db image successfully updated to the desired image in configmap openshift-operators/cloud-native-postgresql-operand-images-config

IAM operandrequest succeed
Screenshot 2025-07-25 at 12 56 38 PM

Copy link
Contributor

@Daniel-Fan Daniel-Fan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@ibm-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Daniel-Fan, qpdpQ

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Daniel-Fan Daniel-Fan merged commit 45a4fe3 into IBM:master Jul 25, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants