Skip to content

OCPBUGS-114428: Update build-machinery-go vendor dependency - #260

Open
redhat-chai-bot wants to merge 1 commit into
openshift:masterfrom
redhat-chai-bot:update-build-machinery-go
Open

OCPBUGS-114428: Update build-machinery-go vendor dependency#260
redhat-chai-bot wants to merge 1 commit into
openshift:masterfrom
redhat-chai-bot:update-build-machinery-go

Conversation

@redhat-chai-bot

@redhat-chai-bot redhat-chai-bot commented Sep 10, 2026

Copy link
Copy Markdown

Summary

Update github.com/openshift/build-machinery-go vendor dependency to v0.0.0-20260902143904-520f675c892b.

Changes

  • Updated go.mod with the latest build-machinery-go version
  • Ran go mod tidy
  • Ran go mod vendor to update the vendored copy

Related

  • OCPBUGS-114428: Bump golang builder to 1.26 and protobuf for 5.1 OCP CI and Nightlies

AI-generated. Review for accuracy.

@Prashanth684 requested in Slack thread

Summary by CodeRabbit

  • Chores
    • Updated the build tooling dependency to a newer version.

@openshift-ci-robot openshift-ci-robot added the jira/severity-critical Referenced Jira bug's severity is critical for the branch this PR is targeting. label Sep 10, 2026
@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 0b2a3a3d-e81d-4ec6-938b-1a774a134333

📥 Commits

Reviewing files that changed from the base of the PR and between 371d998 and 9d5595c.

⛔ Files ignored due to path filters (12)
  • go.sum is excluded by !**/*.sum
  • vendor/github.com/openshift/build-machinery-go/.gitignore is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/AGENTS.md is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/ARCHITECTURE.md is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/CLAUDE.md is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/CONTRIBUTING.md is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/OWNERS is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/OWNERS_ALIASES is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/README.md is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/make/lib/golang.mk is excluded by !**/vendor/**, !vendor/**
  • vendor/github.com/openshift/build-machinery-go/make/targets/golang/version.mk is excluded by !**/vendor/**, !vendor/**
  • vendor/modules.txt is excluded by !**/vendor/**, !vendor/**
📒 Files selected for processing (1)
  • go.mod

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The pull request updates the direct github.com/openshift/build-machinery-go dependency in go.mod from the previous pseudo-version to a newer pseudo-version.

Changes

Build Machinery Dependency

Layer / File(s) Summary
Update module version
go.mod
The direct github.com/openshift/build-machinery-go requirement now uses version v0.0.0-20260902143904-520f675c892b.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: ingvagabund, tchap

Merge Risk: ⚪ Minimal · up to 9d559

This is a scoped dependency update with no identified merge-blocking risk.

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the tracked issue and the main change: updating the build-machinery-go vendor dependency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed The reviewed range changes only dependency metadata and vendored build/documentation files. It contains no Go or test files, and the patch introduces no Ginkgo title calls such as It, Describe, `C…
Test Structure And Quality ✅ Passed PASS: The custom check is not applicable to this pull request. The review-scoped diff changes go.mod, go.sum, vendor/modules.txt, and vendored build-machinery metadata/documentation/make files. It cha…
Microshift Test Compatibility ✅ Passed No new Ginkgo e2e tests were added. The review-scoped diff changes only go.mod, go.sum, vendor metadata, documentation, and build makefiles. The added-line scan found no It, Describe, Context, When, o…
Single Node Openshift (Sno) Test Compatibility ✅ Passed PASS: The pull request adds no Go files or Ginkgo e2e tests. The authoritative diff changes dependency metadata and vendored Make/documentation files only. Therefore, the SNO test compatibility check …
Topology-Aware Scheduling Compatibility ✅ Passed PASS: The authoritative PR diff changes only Go dependency metadata, vendored build-machinery-go documentation/make files, and modules.txt. It adds or modifies no deployment manifests, operator code, …
Ote Binary Stdout Contract ✅ Passed PASS: The pull request changes dependency metadata, vendored documentation, and two Make fragments. The authoritative diff contains no changed Go files, no OTE process-level code, and no stdout writes…
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The pull request adds no Go files or Ginkgo tests. The changed files contain dependency metadata, vendored build-machinery documentation, and Make fragments only. Therefore, it introduces no IPv4 assu…
No-Weak-Crypto ✅ Passed PASS: The reviewed range changes only the build-machinery-go version metadata, vendored documentation, and two Make fragments. The functional changes add a version linker flag and validate Go version …
Container-Privileges ✅ Passed PASS: The pull request changes only go.mod, go.sum, vendor/modules.txt, and files in the vendored build-machinery-go module. No Dockerfile, Kubernetes manifest, or YAML/JSON deployment file is changed…
No-Sensitive-Data-In-Logs ✅ Passed PASS: The pull request does not introduce logging of sensitive data. The authoritative diff changes dependency metadata, vendored documentation, and Make fragments. The new version-check target prints…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Sep 10, 2026
@openshift-ci-robot

Copy link
Copy Markdown
Contributor

@redhat-chai-bot: This pull request references Jira Issue OCPBUGS-114428, which is valid.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (5.1.0) matches configured target version for branch (5.1.0)
  • bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, POST)

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Summary

Update github.com/openshift/build-machinery-go vendor dependency to v0.0.0-20260902143904-520f675c892b.

Changes

  • Updated go.mod with the latest build-machinery-go version
  • Ran go mod tidy
  • Ran go mod vendor to update the vendored copy

Related

  • OCPBUGS-114428: Bump golang builder to 1.26 and protobuf for 5.1 OCP CI and Nightlies

AI-generated. Review for accuracy.

@Prashanth684 requested in Slack thread

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.

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

@openshift-ci
openshift-ci Bot requested review from bertinatto and ibihim September 10, 2026 22:32
@openshift-ci

openshift-ci Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: redhat-chai-bot
Once this PR has been reviewed and has the lgtm label, please assign benluddy for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci

openshift-ci Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

@redhat-chai-bot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/images 9d5595c link true /test images

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/severity-critical Referenced Jira bug's severity is critical for the branch this PR is targeting. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants