[konflux-agent] Fix: run go mod tidy to update go.sum (fixes PR #1669 Konflux build)#1671
[konflux-agent] Fix: run go mod tidy to update go.sum (fixes PR #1669 Konflux build)#1671raptorsun wants to merge 2 commits into
Conversation
PR openshift#1669 bumped two dependencies in go.mod: - github.com/openshift/api v0.0.0-20260424174501-4f63a40a2970 - google.golang.org/protobuf v1.36.12-0.20260120151049-f2248ac996af But go.sum was not updated, causing the Konflux hermetic build to fail when go mod download tried to verify checksums by contacting sum.golang.org (unreachable in a hermetic environment). Running go mod tidy adds the missing checksum entries. konflux-agent/retried-from: lightspeed-operator-on-pull-request-hl46t
|
Warning Review limit reached
More reviews will be available in 56 minutes. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
✨ 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 |
|
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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
|
@raptorsun: The following tests 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. |
Auto-fix by Konflux Agent
Original PR: #1669
Failed PipelineRun:
lightspeed-operator-on-pull-request-hl46t(namespace:crt-nshift-lightspeed-tenant)Root Cause
PR #1669 bumped two dependencies in
go.modbut did not updatego.sum:github.com/openshift/apiv0.0.0-20260420151639-34e60874783ev0.0.0-20260424174501-4f63a40a2970google.golang.org/protobufv1.36.11v1.36.12-0.20260120151049-f2248ac996afThe Konflux hermetic build ran
go mod downloadinside a network-isolated container. Without the checksum entries ingo.sum, Go attempted to verify the new module versions by contactingsum.golang.org, which is unreachable in a hermetic environment — causing the build to fail with:Changes
go mod tidyon the PR branch to add the 4 missing checksum lines togo.sumgo.sumwas modified (1 file, 4 insertions / 4 deletions)Validation
go mod verify→ all modules verified ✅go build -tags exclude_graphdriver_btrfs ./...→ clean build ✅Auto-generated by konflux-agent. Please review carefully.
konflux-agent/retried-from: lightspeed-operator-on-pull-request-hl46t