Skip to content

Go/No-Go Analysis: metrics-server Chart 3.13.0 with Image v0.8.1 - #203

Draft
Azahorscak wants to merge 2 commits into
mainfrom
cursor/gonogo-metrics-server-3.13.0-3.13.1-c22e
Draft

Go/No-Go Analysis: metrics-server Chart 3.13.0 with Image v0.8.1#203
Azahorscak wants to merge 2 commits into
mainfrom
cursor/gonogo-metrics-server-3.13.0-3.13.1-c22e

Conversation

@Azahorscak

@Azahorscak Azahorscak commented May 1, 2026

Copy link
Copy Markdown
Member

Checklist

  • I have signed the CLA
  • I have updated/added any relevant documentation

Description

What's the goal of this PR?

This PR provides a comprehensive go/no-go analysis for upgrading metrics-server using Helm chart 3.13.0 with an image override to v0.8.1.

What changes did you make?

Updated the analysis report (gonogo-metrics-server-3.13.0-to-3.13.1.md) to reflect the image override approach:

Key Changes

Status: ⚠️ CONDITIONAL GO (upgraded from NO-GO)

The analysis now recommends using Helm chart 3.13.0 with image.tag override to v0.8.1 since chart version 3.13.1 does not exist.

Comprehensive Analysis Includes

  1. Security Assessment

    • ✅ Golang 1.24.12 fixes 6 CVEs (archive/zip DoS, net/http exhaustion, crypto/tls issues)
    • ✅ Kubernetes dependencies upgraded to v0.33.7
    • ⚠️ New dependency CVEs present in v0.8.1 (CVE-2026-33186 Critical, CVE-2026-24051 High)
  2. Risk Assessment Matrix

    • Compatibility: Low risk
    • Testing: Medium risk (not officially bundled)
    • Security: Medium risk (trades old CVEs for new ones)
    • Rollback: Low risk
  3. Implementation Guide

    • Helm commands for image override
    • Step-by-step deployment procedure
    • Validation checks
    • Rollback instructions
  4. CVE Comparison Table

    • Tracks 8 CVEs across both versions
    • Shows what gets fixed vs. introduced
  5. Three Alternative Options

    • Option 1: Image override to v0.8.1 (recommended)
    • Option 2: Stay on v0.8.0
    • Option 3: Wait for official chart release

Key Findings

Upgrade Components

  • Helm Chart: 3.13.0 (no change - latest available)
  • Application Image: v0.8.0 → v0.8.1 (via override)
  • Golang: 1.24.11 → 1.24.12
  • K8s deps: v0.33.x → v0.33.7

Pros ✅

  • Fixes 6 Golang CVEs from previous versions
  • Better Kubernetes 1.33+ compatibility
  • Official image from kubernetes-sigs
  • Easy rollback if issues arise

Cons ⚠️

  • Not officially bundled/tested combination
  • Introduces 2 new dependency CVEs (may not be exploitable)
  • Security scanners will flag new CVEs
  • Future official chart may require another upgrade

What alternative solution should we consider, if any?

As documented in the report, three options:

  1. Image Override (Recommended): Deploy chart 3.13.0 with image v0.8.1 override

    • Best for: Need security updates, can test first, tolerate scanner alerts
  2. Stay on v0.8.0: Keep current configuration

    • Best for: Risk-averse, prefer officially bundled versions
  3. Wait for Official Release: Delay until chart 3.13.1/3.14.0 released

    • Best for: Can accept delay, want fully tested solution

Recommendations

CONDITIONAL GO with testing:

  1. Test in non-production environment first
  2. Run security scans to assess CVE exploitability
  3. Validate metrics collection and HPA functionality
  4. Deploy to production if validation passes

The security improvements from Golang 1.24.12 likely outweigh the risk from dependency CVEs, which may not be exploitable in metrics-server's specific usage context.

Slack Thread

Open in Web Open in Cursor 

- Document that version 3.13.1 does not exist
- Latest available version is 3.13.0 (released Jul 22, 2025)
- Provide detailed investigation findings from official sources
- Recommend NO-GO with alternative actions

Co-authored-by: Adam Zahorscak <Azahorscak@users.noreply.github.com>
@fairwinds-insights

Copy link
Copy Markdown

Fairwinds Insights Scan Results

View the full report at insights.fairwinds.com.

✅ No new Action Items detected!

- Change from NO-GO to CONDITIONAL GO with image override approach
- Detail security improvements: Golang 1.24.12 fixes 6 CVEs
- Document new CVEs in v0.8.1 dependencies (CVE-2026-33186, CVE-2026-24051)
- Add comprehensive risk assessment and pros/cons
- Provide implementation guide with Helm commands
- Include validation and rollback procedures
- Add testing recommendations and monitoring guidance

Co-authored-by: Adam Zahorscak <Azahorscak@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@cursor cursor Bot changed the title Go/No-Go Analysis: metrics-server 3.13.0 → 3.13.1 Go/No-Go Analysis: metrics-server Chart 3.13.0 with Image v0.8.1 May 1, 2026
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