Skip to content

Update Go version to 1.26.8 - #488

Open
anispate wants to merge 1 commit into
openshift:masterfrom
anispate:ROSAENG-66308
Open

Update Go version to 1.26.8#488
anispate wants to merge 1 commit into
openshift:masterfrom
anispate:ROSAENG-66308

Conversation

@anispate

@anispate anispate commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

https://redhat.atlassian.net/browse/ROSAENG-66308 - Update minimum Go version from 1.26.0 to 1.26.8.

Summary by CodeRabbit

  • Chores
    • Updated the required Go version to 1.26.8.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Advanced

Run ID: 1271afaa-917b-406b-8a6b-aae56a196529

📥 Commits

Reviewing files that changed from the base of the PR and between 3d8a1d8 and 2f42026.

📒 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 module’s declared Go version changes from 1.26.0 to 1.26.8.

Changes

Go version update

Layer / File(s) Summary
Update module Go version
go.mod
The module now requires Go 1.26.8 instead of 1.26.0.

Priority: ⬇️ Low

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

Merge Risk: ⚪ Minimal · up to 2f420

This updates the declared Go patch version only. No current merge-blocking risk is identified.

Suggested reviewers: pshickeydev

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
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 PASS. The pull request changes only go.mod, replacing go 1.26.0 with go 1.26.8. It does not add or modify Ginkgo test titles, so it introduces no unstable test names.
Test Structure And Quality ✅ Passed PASS: The pull request changes only go.mod, updating go 1.26.0 to go 1.26.8. It does not add or modify Ginkgo test code, so the listed test-structure requirements are not applicable.
Microshift Test Compatibility ✅ Passed The pull request changes only go.mod, updating go 1.26.0 to go 1.26.8. The diff adds no Ginkgo tests and introduces no MicroShift-incompatible API, namespace, or feature usage.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PASS: The pull request changes only the go.mod Go version declaration from 1.26.0 to 1.26.8. The commit diff contains no added or modified Ginkgo e2e tests, so it introduces no SNO multi-node or…
Topology-Aware Scheduling Compatibility ✅ Passed PASS. The pull request changes only go.mod, updating go 1.26.0 to go 1.26.8. It does not add or modify deployment manifests, operators, controllers, or scheduling constraints. The topology-aware…
Ote Binary Stdout Contract ✅ Passed PASS. The pull request changes only go.mod, from go 1.26.0 to go 1.26.8. It does not change Go source or process-level code, so it introduces no OTE stdout write or JSON stdout contract violatio…
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PASS: The pull request changes only go.mod, updating the Go version from 1.26.0 to 1.26.8. It adds no Ginkgo e2e tests or networking behavior, so the IPv4 and external-connectivity conditions do…
No-Weak-Crypto ✅ Passed The pull request changes only the go.mod language version from go 1.26.0 to go 1.26.8. The diff adds no MD5, SHA1, DES, RC4, 3DES, Blowfish, ECB, custom crypto, or secret-comparison code. The we…
Container-Privileges ✅ Passed PASS. The pull request changes only go.mod, from go 1.26.0 to go 1.26.8. It does not change a container or Kubernetes manifest. Existing allowPrivilegeEscalation: true and runAsUser: 0 entri…
No-Sensitive-Data-In-Logs ✅ Passed PASS. The commit changes only go.mod, replacing go 1.26.0 with go 1.26.8. It adds no logging code or sensitive data output.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: updating the Go version to 1.26.8.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@openshift-ci
openshift-ci Bot requested review from Mhodesty and tnierman September 8, 2026 18:33
@openshift-ci

openshift-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anispate

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

The pull request process is described 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 added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 8, 2026
@openshift-ci

openshift-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

@anispate: all tests passed!

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.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.06%. Comparing base (d12dd78) to head (2f42026).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #488   +/-   ##
=======================================
  Coverage   74.06%   74.06%           
=======================================
  Files          11       11           
  Lines         752      752           
=======================================
  Hits          557      557           
  Misses        173      173           
  Partials       22       22           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@anispate

anispate commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

/hold

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants