[18-stable] Resolve InstanceHa container image from ConfigMap before fallback - #641
Conversation
The webhook was always defaulting spec.containerImage, which meant the controller never reached the ConfigMap lookup path. This made the infra-instanceha-config ConfigMap (managed by openstack-operator via OpenStackVersion) effectively dead code. Remove the ContainerImage webhook defaulting and reorder the resolution in GetContainerImage() so the priority is: 1. spec.containerImage (explicit user override) 2. infra-instanceha-config ConfigMap (openstack-operator managed) 3. RELATED_IMAGE env var / hardcoded fallback (standalone deployment) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lmiccini, openshift-cherrypick-robot The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/cherry-pick 18.0-fr6 |
|
@lmiccini: once the present PR merges, I will cherry-pick it on top of DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
bc6069e
into
openstack-k8s-operators:18-stable
|
@lmiccini: new pull request created: #642 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
2 participants
This is an automated cherry-pick of #634
/assign lmiccini