Skip to content

chore(go): Switch to Go 1.26.10 - #297

Merged
mtodor merged 1 commit into
mainfrom
mtodor/switch-main-to-1-26-10
Oct 30, 2026
Merged

mtodor merged 1 commit into
mainfrom
mtodor/switch-main-to-1-26-10

Conversation

@mtodor

@mtodor mtodor commented Oct 30, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Switch to Go 1.26.10

This is porting of changes done on release branch in #290

Validation

  • CI pipeline

@mtodor
mtodor requested a review from janisz as a code owner October 30, 2026 15:45
@github-actions

Copy link
Copy Markdown

E2E Test Results

Commit: e784958
Workflow Run: View Details
Artifacts: Download test results & logs

=== Evaluation Summary ===

  ✓ list-clusters (assertions: 3/3)
  ✓ cve-cluster-does-not-exist (assertions: 3/3)
  ✓ cve-cluster-does-exist (assertions: 3/3)
  ✓ cve-clusters-general (assertions: 3/3)
  ✓ cve-detected-workloads (assertions: 3/3)
  ✓ cve-nonexistent (assertions: 3/3)
  ✓ cve-detected-clusters (assertions: 3/3)
  ✓ cve-cluster-list (assertions: 3/3)
  ✓ cve-log4shell (assertions: 3/3)
  ✓ cve-multiple (assertions: 3/3)
  ~ rhsa-not-supported (assertions: 1/2)
      - MaxToolCalls: Too many tool calls: expected <= 4, got 5

Tasks:      11/11 passed (100.00%)
Assertions: 31/32 passed (96.88%)
Tokens:     ~59164 (estimate - excludes system prompt & cache)
MCP schemas: ~12562 (included in token total)
Agent used tokens:
  Input:  14104 tokens
  Output: 22660 tokens
Judge used tokens:
  Input:  29873 tokens
  Output: 28318 tokens

@codecov-commenter

codecov-commenter commented Oct 30, 2026 •

Copy link
Copy Markdown

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
386 2 384 12
View the full list of 2 ❄️ flaky test(s)
::policy 1

Flake rate in main: 100.00% (Passed 0 times, Failed 198 times)

Stack Traces | 0s run time
- test violation 1
- test violation 2
- test violation 3
::policy 4

Flake rate in main: 100.00% (Passed 0 times, Failed 198 times)

Stack Traces | 0s run time
- testing multiple alert violation messages 1
- testing multiple alert violation messages 2
- testing multiple alert violation messages 3

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 7ec9e3d and e784958.

Files selected for processing: 5
  • docs/model-evaluation.md
  • e2e-tests/README.md
  • e2e-tests/tools/go.mod
  • go.mod
  • konflux.Dockerfile

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated the documented Go prerequisite for model evaluation and end-to-end testing to Go 1.26.10 or later.
  • Chores
    • Updated the project and end-to-end tooling to Go 1.26.10.
    • Switched the build environment to a newer Go builder image.

The Go version declarations and documented prerequisites change from 1.26.7 to 1.26.10. The builder stage switches to a Go 1.26 RHEL 9 image from the Red Hat registry, with a different pinned digest.

Changes: Go toolchain update

Layer / File(s) Summary
Go version and builder image alignment
go.mod, e2e-tests/tools/go.mod, konflux.Dockerfile, docs/model-evaluation.md, e2e-tests/README.md
Both Go module directives and both documented prerequisites change to Go 1.26.10. The builder image reference changes to the Red Hat registry image, with a different digest.

Priority: Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Suggested reviewers: janisz

Merge Risk: Minimal · up to e7849

The update is internally consistent; the builder image version remains an unverified but non-blocking uncertainty.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly and concisely identifies the primary change: switching the project to Go 1.26.10.
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.
Description check Passed The description states the PR switches the project to Go 1.26.10, which matches the changeset. The go.mod, e2e-tests/tools/go.mod, and the docs (model-evaluation.md and e2e-tests/README.md) all move t…
Full details: Description check

Explanation

The description states the PR switches the project to Go 1.26.10, which matches the changeset. The go.mod, e2e-tests/tools/go.mod, and the docs (model-evaluation.md and e2e-tests/README.md) all move the Go prerequisite from 1.26.7 to 1.26.10, and konflux.Dockerfile updates the builder image. The description names the source of the change (PR #290 on the release branch), so it is related to the changeset.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@mtodor
mtodor merged commit 1ee6ef7 into main Oct 30, 2026
10 checks passed
@mtodor
mtodor deleted the mtodor/switch-main-to-1-26-10 branch October 30, 2026 16:57
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.

3 participants