OPRUN-4202: Fix issue to sync faced due replace forcing old dependency #1125
Conversation
openshift-ci-robot
added
the
jira/valid-reference
|
@camilamacedo86: This pull request references OPRUN-4202 which is a valid jira issue. 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/label qe-approved |
openshift-ci
bot
added
the
qe-approved
|
@camilamacedo86: This pull request references OPRUN-4202 which is a valid jira issue. 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 openshift-eng/jira-lifecycle-plugin repository. |
openshift-ci-robot
added
the
verified
|
@bandrade: This PR has been marked as verified by 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86, grokspawn 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 |
openshift-ci
bot
added
the
approved
|
/test e2e-gcp-olm |
|
/test e2e-gcp-olm |
|
/retest-required |
|
/test e2e-gcp-olm |
3 similar comments
|
/test e2e-gcp-olm |
|
/test e2e-gcp-olm |
|
/test e2e-gcp-olm |
|
/retest-required |
|
/test e2e-gcp-olm |
|
/retest-required |
|
/test e2e-gcp-olm |
|
/retest-required |
|
/test ci/prow/verify-deps |
|
@camilamacedo86: The specified target(s) for The following commands are available to trigger optional jobs: Use 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. |
|
/retest-required |
|
/test pull-ci-openshift-operator-framework-olm-main-verify-deps |
|
@camilamacedo86: The specified target(s) for The following commands are available to trigger optional jobs: Use 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. |
|
@camilamacedo86: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
PR needs rebase. DetailsInstructions 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. |
openshift-merge-robot
added
the
needs-rebase
|
Get merged with #1129 |
8 participants
Description
When we tried to sync the build was failing due to:
grpc.SupportPackageIsVersion9
which is generated by
protoc-gen-go-grpc v1.5.1for code that requires gRPC-Go ≥ v1.64.0.However, the
go.modcurrently includes:replace google.golang.org/grpc => google.golang.org/grpc v1.63.2
This pins the project to an older gRPC version that is incompatible with the generated code.
Fix
Removed the
replacedirective forgoogle.golang.org/grpcingo.mod.The replacement was originally added due to limitations with Go 1.23, but we are now using Go 1.24.4, so it is no longer needed. See: operator-framework/operator-lifecycle-manager#3284
This change aligns the project with the upstream OLM (v0) behavior.