[konflux-agent] Fix: update go.sum for bumped dependency versions in PR #1669#1672
[konflux-agent] Fix: update go.sum for bumped dependency versions in PR #1669#1672raptorsun wants to merge 2 commits into
Conversation
go.sum was missing checksum entries for the two dependency versions introduced in go.mod: - github.com/openshift/api v0.0.0-20260424174501-4f63a40a2970 - google.golang.org/protobuf v1.36.12-0.20260120151049-f2248ac996af Without these entries the hermetic build's 'go mod download' step fails with 'missing go.sum entry', causing both build-images matrix tasks to fail and the entire Konflux PipelineRun to fail. Fix: run 'go mod tidy' to populate go.sum with the correct checksums. konflux-agent/retried-from: lightspeed-operator-on-pull-request-hl46t
|
Warning Review limit reached
More reviews will be available in 50 minutes and 4 seconds. 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 (and retry lightspeed-operator-on-pull-request-tpnjt)
Root Cause
PR #1669 bumped two dependencies in
go.modbut did not updatego.sum:github.com/openshift/api:v0.0.0-20260420151639-34e60874783e→v0.0.0-20260424174501-4f63a40a2970google.golang.org/protobuf:v1.36.11→v1.36.12-0.20260120151049-f2248ac996afBecause
go.sumlacked checksum entries for the new versions, the hermetic Konflux build's Dockerfile step:```
RUN . /cachi2/cachi2.env && go mod download
```
failed with:
```
missing go.sum entry for module providing package github.com/openshift/api/config/v1
```
Both
build-imagesmatrix tasks (linux/x86_64 and linux/arm64) failed, causing 13 downstream tasks to be skipped.Changes
Ran
go mod tidyto updatego.sumwith correct checksums for the two new dependency versions.go.sum)Validation
go mod tidycompleted with no errorsgo.sumnow contains entries for both new versionsmissing go.sum entryerror; fix branch does notAuto-generated by konflux-agent. Please review carefully.