NO-ISSUE: Update github.com/openshift/cluster-api-provider-agent/api digest to 7e9b75f#1015
Conversation
|
@red-hat-konflux[bot]: This pull request explicitly references no jira issue. DetailsIn response to this:
Instructions 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 openshift-eng/jira-lifecycle-plugin repository. |
WalkthroughUpdated the ChangesDependency Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 13 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (13 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 NOT APPROVED This pull-request has been approved by: red-hat-konflux[bot] 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[bot]. Thanks for your PR. I'm waiting for a openshift 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. |
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 16: The go.mod shows a require for
github.com/openshift/cluster-api-provider-agent/api at
v0.0.0-20260528181349-f46caad1d728 but that is shadowed by the replace directive
"replace github.com/openshift/cluster-api-provider-agent/api => ./api"; confirm
intended supply-chain behavior by either (A) removing or gating the local
replace so CI/release builds resolve the upstream version, (B) updating the
dependency inside the ./api module itself so the bumped digest is reflected
where consumed, or (C) documenting and enforcing that the local replace is only
for local dev and that SBOM/provenance/CVE tooling target the resolved upstream
artifact for production; update go.mod (or CI/release config) and add a short
note in repository docs explaining which approach you chose.
🪄 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: Repository: openshift/coderabbit/.coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 02e71a37-8306-4c79-bcd7-f24c7d95f6c5
⛔ Files ignored due to path filters (1)
vendor/modules.txtis excluded by!**/vendor/**,!vendor/**
📒 Files selected for processing (1)
go.mod
| github.com/onsi/ginkgo v1.16.5 | ||
| github.com/onsi/gomega v1.38.2 | ||
| github.com/openshift/cluster-api-provider-agent/api v0.0.0-20251202202927-3ad4558809a2 | ||
| github.com/openshift/cluster-api-provider-agent/api v0.0.0-20260528181349-f46caad1d728 |
There was a problem hiding this comment.
go.mod digest bump is likely shadowed by local replace (supply-chain impact uncertain).
In go.mod, the require github.com/openshift/cluster-api-provider-agent/api v0.0.0-20260528181349-f46caad1d728 (line 16) is overridden by replace github.com/openshift/cluster-api-provider-agent/api => ./api (line 137). With a local replace, this upstream version update often won’t change the resolved/downloaded module inputs for this module’s builds—so SBOM/provenance and CVE remediation may not reflect the bumped digest.
Confirm the intended supply-chain outcome for release builds: update the dependency graph where it’s actually consumed (e.g., inside ./api) and ensure SBOM/provenance attestations/CVE checks target the resolved artifacts, or remove/conditionally apply the replace for production and document why the upstream digest bump must still be tracked.
🤖 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 16, The go.mod shows a require for
github.com/openshift/cluster-api-provider-agent/api at
v0.0.0-20260528181349-f46caad1d728 but that is shadowed by the replace directive
"replace github.com/openshift/cluster-api-provider-agent/api => ./api"; confirm
intended supply-chain behavior by either (A) removing or gating the local
replace so CI/release builds resolve the upstream version, (B) updating the
dependency inside the ./api module itself so the bumped digest is reflected
where consumed, or (C) documenting and enforcing that the local replace is only
for local dev and that SBOM/provenance/CVE tooling target the resolved upstream
artifact for production; update go.mod (or CI/release config) and add a short
note in repository docs explaining which approach you chose.
b4ee6f5 to
bc09ad2
Compare
…digest to 7e9b75f Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
bc09ad2 to
04de10c
Compare
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 18: The dependency digest bump for
github.com/openshift/cluster-api-provider-agent/api at line 18 is being
overridden by the local replace directive (replace
github.com/openshift/cluster-api-provider-agent/api => ./api) which shadows the
updated upstream version and creates a supply-chain integrity issue. Choose one
of the recommended approaches: (A) remove or conditionally gate the replace
directive for production builds, (B) update the dependency version inside the
./api module itself, or (C) document the dev-only nature of the replace and
ensure SBOM/provenance tooling targets the resolved upstream artifact. After
selecting an approach, update the go.mod file accordingly and document the
chosen resolution in the repository to ensure the digest bump is actually
reflected in production builds.
🪄 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: Repository: openshift/coderabbit/.coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 8fb6eace-3954-49e5-b3af-4a8652511cd7
⛔ Files ignored due to path filters (1)
vendor/modules.txtis excluded by!**/vendor/**,!vendor/**
📒 Files selected for processing (1)
go.mod
| github.com/onsi/ginkgo v1.16.5 | ||
| github.com/onsi/gomega v1.38.2 | ||
| github.com/openshift/cluster-api-provider-agent/api v0.0.0-20251202202927-3ad4558809a2 | ||
| github.com/openshift/cluster-api-provider-agent/api v0.0.0-20260616215033-7e9b75f7e592 |
There was a problem hiding this comment.
replace directive—supply-chain integrity issue remains unresolved.
The bumped digest at line 18 will not be used in production builds because the replace github.com/openshift/cluster-api-provider-agent/api => ./api directive at line 139 overrides it. This means:
- SBOM/provenance attestations won't reflect the updated upstream dependency version (violates supply-chain security guideline).
- CVE remediation claimed by this digest bump won't be reflected in production artifacts.
- The digest update may create a false sense of patched dependencies.
The previous review flagged this as a critical issue. To resolve it, choose one of these approaches:
Option A (recommended): Remove or conditionally gate the local replace directive for production/CI builds.
Option B: Update the dependency inside the ./api module itself so the bumped digest is reflected where consumed.
Option C: Document that the local replace is dev-only, and ensure SBOM/provenance/CVE tooling target the resolved upstream artifact for production.
Then update go.mod and document the chosen approach in the repository.
🤖 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 18, The dependency digest bump for
github.com/openshift/cluster-api-provider-agent/api at line 18 is being
overridden by the local replace directive (replace
github.com/openshift/cluster-api-provider-agent/api => ./api) which shadows the
updated upstream version and creates a supply-chain integrity issue. Choose one
of the recommended approaches: (A) remove or conditionally gate the replace
directive for production builds, (B) update the dependency version inside the
./api module itself, or (C) document the dev-only nature of the replace and
ensure SBOM/provenance tooling targets the resolved upstream artifact. After
selecting an approach, update the go.mod file accordingly and document the
chosen resolution in the repository to ensure the digest bump is actually
reflected in production builds.
Source: Coding guidelines
This PR contains the following updates:
3ad4558→7e9b75fConfiguration
📅 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