fix(deps): update github.com/openshift/lightspeed-agentic-operator/api digest to eed7142#56
Conversation
…i digest to eed7142 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
📝 WalkthroughWalkthroughUpdated the ChangesDependency Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: 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 |
1 similar comment
|
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: 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 |
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 61: Update the go.mod pseudo-version for the module declaration that
currently reads "github.com/openshift/lightspeed-agentic-operator/api
v0.0.0-20260603171354-eed7142949c4" so it matches the actual ./api HEAD commit
(d8e185ae3fe0de7dca8f28bef306dab2a05cee2c); specifically, replace the
pseudo-version suffix (eed7142949c4) with the correct commit-derived
pseudo-version for commit d8e185ae3fe0... (or alternatively revert the ./api
worktree to commit eed7142949c4313 if you intend to keep the existing
pseudo-version). Ensure the module declaration and any reference to the "replace
... => ./api" directive remain consistent so downstream consumers (without
replace) get the same API commit.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 185e3c51-ea2b-481e-8c5f-37d90c4dc595
📒 Files selected for processing (1)
go.mod
| github.com/monochromegane/go-gitignore v0.0.0-20200626010858-205db1a8cc00 // indirect | ||
| github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect | ||
| github.com/openshift/lightspeed-agentic-operator/api v0.0.0-20260603115325-18e98b1ead8f | ||
| github.com/openshift/lightspeed-agentic-operator/api v0.0.0-20260603171354-eed7142949c4 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify that the pseudo-version matches the actual commit in ./api
echo "=== Checking current commit in ./api directory ==="
cd api && git log -1 --format="Commit: %H%nDate: %ci" && cd ..
echo -e "\n=== Extracting commit from go.mod version ==="
echo "go.mod declares: eed7142949c4 (partial commit hash)"
echo "Expected format: v0.0.0-YYYYMMDDHHMMSS-<commit_prefix>"
echo -e "\n=== Checking if ./api is on the expected commit ==="
cd api && git log --oneline | grep -i "eed7142" && cd ..Repository: openshift/lightspeed-agentic-operator
Length of output: 517
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "=== Resolve pseudo-version commit prefix eed7142949c4 ==="
FULL="$(git rev-parse eed7142949c4)"
echo "Resolved: $FULL"
echo "=== Commit details for resolved hash ==="
git show -s --format="Hash: %H%nDate: %ci%nSubject: %s" "$FULL"
echo "=== Compare with current ./api HEAD ==="
API_HEAD="$(cd api && git rev-parse HEAD)"
echo "api HEAD: $API_HEAD"
echo "api HEAD == resolved? $([ "$API_HEAD" = "$FULL" ] && echo yes || echo no)"
echo "=== Pseudo-version timestamp from go.mod: 20260603171354 ==="
echo "Expected date components: 2026-06-03 17:13:54 (UTC-ish per git timestamps)"Repository: openshift/lightspeed-agentic-operator
Length of output: 628
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "=== Checking replace directive(s) in go.mod ==="
rg -n "replace\s+.*=>\s+\./api" go.mod || true
rg -n "replace\s+.*\./api" go.mod || trueRepository: openshift/lightspeed-agentic-operator
Length of output: 275
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "=== Checking replace directive(s) in go.mod ==="
rg -n "replace\s+.*=>\s+\./api" go.mod || true
rg -n "replace\s+.*\./api" go.mod || trueRepository: openshift/lightspeed-agentic-operator
Length of output: 275
Align go.mod API pseudo-version with the actual ./api commit.
go.mod declares github.com/openshift/lightspeed-agentic-operator/api v0.0.0-20260603171354-eed7142949c4 (line 61), which resolves to commit eed7142949c43136362d87902b0fe047eaeb90c4, but the current ./api HEAD is d8e185ae3fe0de7dca8f28bef306dab2a05cee2c—they don’t match. Since replace ... => ./api is used (line 95), local builds may work, but downstream consumers won’t get the same API state once replace is ignored. Update the pseudo-version to match ./api HEAD (or adjust ./api back to the eed714... commit).
🤖 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 61, Update the go.mod pseudo-version for the module
declaration that currently reads
"github.com/openshift/lightspeed-agentic-operator/api
v0.0.0-20260603171354-eed7142949c4" so it matches the actual ./api HEAD commit
(d8e185ae3fe0de7dca8f28bef306dab2a05cee2c); specifically, replace the
pseudo-version suffix (eed7142949c4) with the correct commit-derived
pseudo-version for commit d8e185ae3fe0... (or alternatively revert the ./api
worktree to commit eed7142949c4313 if you intend to keep the existing
pseudo-version). Ensure the module declaration and any reference to the "replace
... => ./api" directive remain consistent so downstream consumers (without
replace) get the same API commit.
|
@red-hat-konflux: all tests passed! 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. |
This PR contains the following updates:
18e98b1→eed7142Warning
Some dependencies could not be looked up. Check the warning logs for more information.
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), 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.
Summary by CodeRabbit