Repository navigation
chore: update manifest commit SHAs - #3921
Conversation
The CSV containerImage is a managed placeholder that gets replaced during release; the kuberay RELATED_IMAGE will be addressed by opendatahub-io#3921.
The CSV containerImage is a managed placeholder that gets replaced during release; the kuberay RELATED_IMAGE will be addressed by opendatahub-io#3921.
…roring (#3922) * fix(disconnected): use image digests instead of tags for reliable mirroring Tags are mutable and cannot be reliably mirrored in disconnected/air-gapped environments. Replace tag references with content-addressable digests in the RHOAI manager kuberay RELATED_IMAGE and the CSV containerImage annotation. * chore(tooling): bump golangci-lint from v2.5.0 to v2.12.2 The previous version was built with Go 1.24 and fails on projects targeting Go 1.26.0. * revert(disconnected): drop CSV and kuberay digest changes per review The CSV containerImage is a managed placeholder that gets replaced during release; the kuberay RELATED_IMAGE will be addressed by #3921. * revert(disconnected): drop CSV and kuberay digest changes per review The CSV containerImage is a managed placeholder that gets replaced during release; the kuberay RELATED_IMAGE will be addressed by #3921. * fix(disconnected): use digest for kuberay operator image reference Replace the tag-based image reference with a sha256 digest to ensure reliable mirroring in disconnected/air-gapped environments.
0483cc1 to
8047770
Compare
|
/hold Revision 8047770 was retested 3 times: holding |
7bbbd4f to
71dd0fc
Compare
|
/unhold |
c0d0547 to
bbb36f9
Compare
bbb36f9 to
3787037
Compare
|
/lgtm |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: asanzgom, carlkyrillos, grdryn, MarianMacik 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 |
|
/lgtm |
1 similar comment
|
/lgtm |
|
/retest |
|
Retriggered another SHA update after #3984 has been merged. |
|
/lgtm |
|
New changes are detected. LGTM label has been removed. |
|
Just FYI, in feast module operator PR is open to fix issues with mlflowoperator permission - #3921 , once merged the SHA needs to be updated in this PR. |
| odh: | ||
| repo: opendatahub-io/feast-module-operator | ||
| ref: main@dcd8099577492bf1869c7ea1122420c71b31c780 | ||
| ref: main@d80713b6278b838e1d3b5b8cd172ae8f0a0ba75c |
There was a problem hiding this comment.
Could we please update this to latest 1da508a8d2d294a54173d9b1f73eba4aaf4a735e ?
Dashboard team needs fix for https://redhat.atlassian.net/browse/RHOAIENG-83128
There was a problem hiding this comment.
I shall also provide the cherrypick SHA of RHDS feast module oprtr.
There was a problem hiding this comment.
@jyejare We have an automation that updates the SHAs to whatever is latest. I just ran the action and it updated the odh ref for opendatahub-io/feast-module-operator to 1da508a8d2d294a54173d9b1f73eba4aaf4a735e.
Can you confirm if that SHA does or doesn't have the needed fix?
There was a problem hiding this comment.
@carlkyrillos Yes it does has my comment confirms the same. Also, for downstream feast module operator it should be 7f958fb69bd2f9aca9d8e966ece399627fde7f03. I just merged the cherrypick of the ODH PR.
E2E Failure Triage
|
7 participants
This PR updates the manifest commit SHAs to the latest available commits.