[release-4.16] OCPBUGS-79950: Bump google.golang.org/grpc to v1.79.3#1504
Conversation
|
@ocp-sustaining-admins: This pull request references Jira Issue OCPBUGS-79950, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. 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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Review Summary by QodoBump google.golang.org/grpc and dependencies for CVE-2026-33186
WalkthroughsDescription• Bump google.golang.org/grpc to v1.65.0 with security patch • Update OpenTelemetry dependencies to v1.28.0 • Update golang.org/x packages to latest versions • Add replace directive for grpc-go security fork Diagramflowchart LR
grpc["google.golang.org/grpc<br/>v1.63.2 → v1.65.0"]
otel["OpenTelemetry packages<br/>v1.23.0 → v1.28.0"]
golang["golang.org/x packages<br/>updated to latest"]
replace["Add grpc-go fork<br/>security patch"]
grpc --> replace
otel --> grpc
golang --> grpc
File Changes1. go.mod
|
Code Review by Qodo
1. gRPC version mismatch
|
|
Hi @ocp-sustaining-admins. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ocp-sustaining-admins The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
|
||
| replace google.golang.org/grpc => github.com/openshift-sustaining/grpc-go v1.67.3-sec.1 |
There was a problem hiding this comment.
1. Grpc version mismatch 🐞 Bug ≡ Correctness
go.mod still selects google.golang.org/grpc v1.65.0 and then replaces it with github.com/openshift-sustaining/grpc-go v1.67.3-sec.1, so this change does not move the codebase to gRPC v1.79.3 and will not include any fixes introduced after v1.67.3. If consumers/CI/security tracking expect v1.79.3 semantics or patches, they will not be present with the current replace target.
Agent Prompt
## Issue description
The PR does not actually move the dependency set to gRPC v1.79.3; it pins to a forked gRPC at v1.67.3-sec.1 while go.mod still lists google.golang.org/grpc v1.65.0.
## Issue Context
This can leave the repo without fixes/API changes introduced after v1.67.3 and can diverge from the stated intent of bumping gRPC to v1.79.3.
## Fix Focus Areas
- go.mod[170-174]
- go.mod[243-243]
- go.sum[1326-1327]
## Suggested fix
1. Confirm the intended effective gRPC version (e.g., truly v1.79.3 vs. an OCP-patched fork).
2. Update the `replace google.golang.org/grpc => ...` target to the intended patched version (or update the requirement to reflect the intended base version) so that the effective gRPC code matches the desired release.
3. Re-run `go mod tidy` (and vendoring if this repo vendors) to regenerate go.sum consistently.
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
This is an automated PR from the ARC system.
Go version used:
go1.21.13Patch applied:
github.com/openshift-sustaining/grpc-go@v1.67.3-sec.1Modules affected:
.The commands used to generate this PR were:
A member of the Red Hat Openshift Sustaining Team will review the PR and take appropriate action.