Skip to content

fix: Use correct chart versions for 1.33.1#1594

Merged
viccuad merged 6 commits intokubewarden:mainfrom
viccuad:fix/chart-versions
Mar 19, 2026
Merged

fix: Use correct chart versions for 1.33.1#1594
viccuad merged 6 commits intokubewarden:mainfrom
viccuad:fix/chart-versions

Conversation

@viccuad
Copy link
Member

@viccuad viccuad commented Mar 19, 2026

Description

We transiently bumped the chart versions a minor version, instead of a patch version.

Test

CI

Additional Information

Tradeoff

Potential improvement

Checklist

viccuad added 6 commits March 19, 2026 15:52
Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
@viccuad viccuad self-assigned this Mar 19, 2026
@viccuad viccuad requested a review from a team as a code owner March 19, 2026 14:55
Copilot AI review requested due to automatic review settings March 19, 2026 14:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR corrects the Helm chart version pins associated with Kubewarden stack v1.33.1, reverting an accidental minor-version bump to the intended patch-version bump so downstream packaging/installation references the correct chart releases.

Changes:

  • Adjust chart version fields for kubewarden-crds, kubewarden-controller, and kubewarden-defaults to the correct patch versions for v1.33.1.
  • Update Rancher catalog.cattle.io/auto-install annotations to reference kubewarden-crds=1.25.1.
  • Update charts/hauler_manifest.yaml to pin the corrected chart versions for airgap imports.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
charts/kubewarden-defaults/Chart.yaml Corrects chart version and updates auto-install CRD chart pin to 1.25.1.
charts/kubewarden-crds/Chart.yaml Corrects CRD chart version to 1.25.1 for appVersion: v1.33.1.
charts/kubewarden-controller/Chart.yaml Corrects chart version and updates auto-install CRD chart pin to 1.25.1.
charts/hauler_manifest.yaml Aligns Hauler chart pins with the corrected chart versions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@codecov
Copy link

codecov bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.98%. Comparing base (bea5733) to head (d9ff0a5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1594      +/-   ##
==========================================
- Coverage   80.00%   79.98%   -0.02%     
==========================================
  Files         127      127              
  Lines       16573    16573              
==========================================
- Hits        13259    13256       -3     
- Misses       3314     3317       +3     
Flag Coverage Δ
rust-tests 79.98% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@viccuad viccuad merged commit 9972a11 into kubewarden:main Mar 19, 2026
32 checks passed
@github-project-automation github-project-automation bot moved this from Pending review to Done in Kubewarden Admission Controller Mar 19, 2026
@viccuad viccuad deleted the fix/chart-versions branch March 19, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants