fix(deps): update github.com/openshift/lightspeed-agentic-operator/api digest to 3fbf0e7#59
Merged
openshift-merge-bot[bot] merged 1 commit intoJun 4, 2026
Conversation
…i digest to 3fbf0e7 Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
red-hat-konflux
Bot
added
approved
📝 WalkthroughWalkthroughThe PR updates the ChangesDependency Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
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: go.mod pins github.com/openshift/lightspeed-agentic-operator/api at
commit 3fbf0e79eb05 but the local ./api directory is at a different commit
(0605af66...), so align them by either checking out the local ./api to the exact
pinned commit 3fbf0e79eb05 (so the internal API matches go.mod) or update the
go.mod entry to the current ./api HEAD
(0605af66ba1c2da12858733a6c42faa06c38a30b) and run module housekeeping (e.g.,
tidy/vendor update) to ensure the module graph is consistent; reference the
go.mod entry for github.com/openshift/lightspeed-agentic-operator/api and the
local ./api HEAD when making the change.
🪄 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: ff56760c-1c14-4a19-ba3c-f200fe571616
📒 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-20260603171354-eed7142949c4 | ||
| github.com/openshift/lightspeed-agentic-operator/api v0.0.0-20260603215905-3fbf0e79eb05 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify ./api is at the expected commit and check for potentially breaking changes
# Check current commit of ./api submodule/directory
echo "=== Checking ./api commit ==="
cd api && git log -1 --oneline 2>/dev/null || echo "Note: ./api is not a git submodule"
cd ..
# Check if commit 3fbf0e79eb05 exists
echo -e "\n=== Verifying commit 3fbf0e79eb05 ==="
cd api && git log --oneline -1 3fbf0e79eb05 2>/dev/null || echo "Could not find commit 3fbf0e79eb05 in ./api"
cd ..
# Compare old commit (eed7142949c4) to new commit (3fbf0e79eb05) for breaking changes
echo -e "\n=== Checking for changes between commits ==="
cd api && git diff eed7142949c4..3fbf0e79eb05 -- '*.go' 2>/dev/null | head -100 || echo "Could not diff commits"
cd ..
# Search for usages of key API types in controller/CLI/client code
echo -e "\n=== Checking API type usage in codebase ==="
rg -n --type=go -C2 'DerivePhase|agenticv1alpha1\.Proposal|RemediationOption|ExecutionAction|ExecutionVerification' controller/ cli/ 2>/dev/null | head -50Repository: openshift/lightspeed-agentic-operator
Length of output: 3136
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "=== ./api HEAD vs expected ==="
cd api
git rev-parse HEAD
git rev-parse 3fbf0e79eb05
echo -e "\n=== Is expected an ancestor of HEAD? ==="
if git merge-base --is-ancestor 3fbf0e79eb05 HEAD; then
echo "3fbf0e79eb05 IS an ancestor of HEAD"
else
echo "3fbf0e79eb05 is NOT an ancestor of HEAD"
fi
echo -e "\n=== Commits from expected..HEAD (excluding expected) ==="
git rev-list --count 3fbf0e79eb05..HEAD
echo -e "\n=== Top commits expected..HEAD (if any) ==="
git rev-list 3fbf0e79eb05..HEAD --max-count=5
cd ..Repository: openshift/lightspeed-agentic-operator
Length of output: 422
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "=== ./api HEAD vs expected (commits already known) ==="
cd api
git rev-list --left-right --count 3fbf0e79eb05...HEAD || true
echo -e "\n=== Diff (expected..HEAD) for *.go ==="
git diff --stat 3fbf0e79eb05..HEAD -- '*.go' || true
git diff 3fbf0e79eb05..HEAD -- '*.go' | head -200 || echo "No *.go changes"
echo -e "\n=== Diff for likely API/CRD artifacts ==="
git diff --stat 3fbf0e79eb05..HEAD -- '*.yaml' '*.yml' '*.json' 'config/**' 'deploy/**' || true
git diff 3fbf0e79eb05..HEAD -- '*.yaml' '*.yml' '*.json' | head -200 || echo "No YAML/JSON changes"
cd ..
echo -e "\n=== Repo usages of the key API types (controller + cli) ==="
rg -n --type=go -C1 'agenticv1alpha1\.(Proposal|RemediationOption|ExecutionAction|ExecutionVerification)|DerivePhase' controller cli | head -80 || trueRepository: openshift/lightspeed-agentic-operator
Length of output: 5961
Align ./api with the commit pinned in go.mod (breaking-change risk looks low)
go.modpinsgithub.com/openshift/lightspeed-agentic-operator/apito3fbf0e79eb05..., but the local./apidirectory is at0605af66ba1c2da12858733a6c42faa06c38a30b(the pinned commit is not an ancestor; there’s 1 commit between them).- Between the pinned commit and current
./apiHEAD, there are no detected changes in*.goand no detected changes in YAML/JSON/config/deploy artifacts, so breakage risk for controller/CLI/client types likeProposalandDerivePhaseappears low. - For local/external consistency, ensure
./apiis checked out to the exact pinned commit (or update the pinned version to match./api).
🤖 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, go.mod pins
github.com/openshift/lightspeed-agentic-operator/api at commit 3fbf0e79eb05 but
the local ./api directory is at a different commit (0605af66...), so align them
by either checking out the local ./api to the exact pinned commit 3fbf0e79eb05
(so the internal API matches go.mod) or update the go.mod entry to the current
./api HEAD (0605af66ba1c2da12858733a6c42faa06c38a30b) and run module
housekeeping (e.g., tidy/vendor update) to ensure the module graph is
consistent; reference the go.mod entry for
github.com/openshift/lightspeed-agentic-operator/api and the local ./api HEAD
when making the change.
|
@red-hat-konflux[bot]: 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 file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
eed7142→3fbf0e7Warning
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