Update google.golang.org/genproto/googleapis/api digest to 3dc84a4#163
Conversation
Signed-off-by: red-hat-konflux-kflux-prd-rh02 <190377777+red-hat-konflux-kflux-prd-rh02[bot]@users.noreply.github.com>
|
|
[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 |
|
Hi @red-hat-konflux-kflux-prd-rh02[bot]. Thanks for your PR. I'm waiting for a openshift-hyperfleet 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. |
📝 WalkthroughSummary by CodeRabbitRelease Notes
WalkthroughThe PR updates a single indirect dependency in Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.12.2)level=error msg="Running error: context loading failed: failed to load packages: failed to load packages: failed to load with go/packages: err: exit status 1: stderr: go: updates to go.mod needed, disabled by -mod=readonly; to update it:\n\tgo mod tidy\n" Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@go.mod`:
- Line 139: The go.mod entry for module
google.golang.org/genproto/googleapis/api uses an invalid pseudo-version
"3dc84a4a5aaa"; replace it with the full Go pseudo-version format
v0.0.0-YYYYMMDDHHMMSS-3dc84a4a5aaa by querying the module proxy or running go
mod tidy to resolve the correct timestamped pseudo-version for commit
3dc84a4a5aaa, update the line referencing
google.golang.org/genproto/googleapis/api accordingly, then run the repository's
standard bingo get and go mod tidy/verify to ensure the corrected version is
accepted.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: af4c78c2-4936-4855-8593-24cce0ce10ac
📒 Files selected for processing (1)
go.mod
| google.golang.org/api v0.266.0 // indirect | ||
| google.golang.org/genproto v0.0.0-20260209200024-4cfbd4190f57 // indirect | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20260401024825-9d38bb4040a9 // indirect | ||
| google.golang.org/genproto/googleapis/api 3dc84a4a5aaa // indirect |
There was a problem hiding this comment.
Invalid Go module version syntax.
Line 139 is missing the v0.0.0- prefix and timestamp required for Go pseudo-versions. Current format 3dc84a4a5aaa will cause go mod commands to fail.
Expected format: v0.0.0-YYYYMMDDHHMMSS-3dc84a4a5aaa
Run this script to validate the syntax and retrieve the correct pseudo-version:
#!/bin/bash
# Validate go.mod syntax and find correct version
# Check if go.mod is valid
go mod verify 2>&1 || true
# Try to tidy and see what version Go resolves
cp go.mod go.mod.backup
go mod tidy 2>&1 || true
if [ $? -eq 0 ]; then
echo "=== Corrected version from go mod tidy ==="
grep "google.golang.org/genproto/googleapis/api" go.mod
fi
mv go.mod.backup go.mod
# Query the module proxy for the commit
echo "=== Querying module proxy for commit 3dc84a4a5aaa ==="
curl -s "https://proxy.golang.org/google.golang.org/genproto/googleapis/api/@v/list" | tail -5As per coding guidelines, ensure bingo get is run before validation per HyperFleet standards for Go-module dependency updates.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@go.mod` at line 139, The go.mod entry for module
google.golang.org/genproto/googleapis/api uses an invalid pseudo-version
"3dc84a4a5aaa"; replace it with the full Go pseudo-version format
v0.0.0-YYYYMMDDHHMMSS-3dc84a4a5aaa by querying the module proxy or running go
mod tidy to resolve the correct timestamped pseudo-version for commit
3dc84a4a5aaa, update the line referencing
google.golang.org/genproto/googleapis/api accordingly, then run the repository's
standard bingo get and go mod tidy/verify to ensure the corrected version is
accepted.
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for the If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This PR contains the following updates:
9d38bb4→3dc84a4Configuration
📅 Schedule: Branch creation - "on monday" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
To execute skipped test pipelines write comment
/ok-to-test.Documentation
Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.