Skip to content

🌱 build: update Bingo kube-score to upstream master - #3087

Open
tmshort wants to merge 1 commit into
operator-framework:mainfrom
tmshort:build/update-kube-score-bingo-master
Open

tmshort wants to merge 1 commit into
operator-framework:mainfrom
tmshort:build/update-kube-score-bingo-master

Conversation

@tmshort

@tmshort tmshort commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Update Bingo's kube-score pin to upstream master commit 06dedc9 (v1.20.1-0.20260520132026-06dedc9ccce5), which removes imports of Kubernetes autoscaling beta API packages.
  • Keep kube-score's Bingo build on k8s.io/api and k8s.io/apimachinery v0.37.1, matching operator-controller, with module replacements that persist through Bingo regeneration.
  • Update the generated Bingo variables and checksums. This PR updates the dependency before the separate Bingo to go tool migration.

Validation

  • make verify-bingo
  • go list -m -modfile=.bingo/kube-score.mod k8s.io/api k8s.io/apimachinery confirms both effective modules use v0.37.1.

Summary by CodeRabbit

  • Chores
    • Updated the version of the Kubernetes configuration scoring tool used in project validation. This changes the tool available for checking Kubernetes configurations; no user-facing features or behavior are described in this update.

Pin kube-score to upstream commit 06dedc9, whose parser handles removed autoscaling beta API types. Keep its Bingo build on the same k8s.io/api and k8s.io/apimachinery v0.37.1 versions used by operator-controller through persistent module replacements.

Signed-off-by: Todd Short <tshort@redhat.com>
@openshift-ci
openshift-ci Bot requested review from dtfranz and joelanford October 29, 2026 22:47
@openshift-ci

openshift-ci Bot commented Oct 29, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kevinrizza 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

@netlify

netlify Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for olmv1 ready!

Name Link
🔨 Latest commit 13cc271
🔍 Latest deploy log https://app.netlify.com/projects/olmv1/deploys/6ae3cd047887272014a432ef
😎 Deploy Preview https://deploy-preview-3087--olmv1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Walkthrough

The Bingo configuration now targets kube-score v1.20.1-0.20260520132026-06dedc9ccce5. Its module file declares Go 1.26.0 and replaces k8s.io/api and k8s.io/apimachinery with v0.37.1.

Changes: kube-score update

Layer / File(s) Summary
Update kube-score version and module settings
.bingo/kube-score.mod, .bingo/variables.env, .bingo/Variables.mk
The module file updates the Go version, Kubernetes module replacements, and kube-score requirement. The environment variable and binary target use the new kube-score version.

Priority: Low

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

Change: Other

Possibly related PRs

Suggested reviewers: pedjak

Merge Risk: Minimal · up to 13cc2

No actionable merge-blocking risk was identified in this update.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies the build change: updating Bingo's kube-score dependency to upstream master. It is concise and related to the main changes.
Description check Passed The description explains the dependency update, Kubernetes module replacements, generated file changes, motivation, and validation steps. It omits the template's reviewer checklist, but the required c…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Description check

Explanation

The description explains the dependency update, Kubernetes module replacements, generated file changes, motivation, and validation steps. It omits the template's reviewer checklist, but the required change summary and motivation are complete.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files. (2 skipped: 2 unsupported.)

Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@tmshort tmshort changed the title build: update Bingo kube-score to upstream master 🌱 build: update Bingo kube-score to upstream master Oct 29, 2026
@tmshort

tmshort commented Oct 30, 2026

Copy link
Copy Markdown
Member Author

I admit it's a bit odd to grab a tool based on the latest unreleased code.

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant