Skip to content

Conversation

@dmaizel
Copy link
Contributor

@dmaizel dmaizel commented Jan 19, 2026

Summary

  • Removes argo-rollouts as a chart dependency
  • Removes all argo-rollouts configuration from values files
  • Removes argo-rollouts helper functions and env vars from templates
  • Updates documentation to remove argo-rollouts references
  • Removes rollout-reporter related configuration

Background

Argo Rollouts was deprecated and disabled by default in a previous release (see RELEASE_NOTES_0.26.0.md). This PR completes the removal as planned.

Breaking Change

argo-rollouts is no longer included in the chart. Users requiring Argo Rollouts must install it separately.

Test plan

  • helm dependency update - Successful
  • helm lint - Passed
  • helm template - Renders successfully
  • helm unittest - All 168 tests pass
  • Verified no remaining argo-rollouts references in chart

Argo Rollouts was deprecated and disabled by default in a previous release.
This commit completes the removal by:

- Removing argo-rollouts dependency from Chart.yaml
- Removing external-argo-rollouts and argo-rollouts configuration from values.yaml
- Removing argo-rollouts helper function and component-list entry from _helpers.tpl
- Removing ROLLOUTS_HELM_REPOSITORY/VERSION env vars from app-proxy
- Removing argo-rollouts from all CI values files
- Removing argo-rollouts from multi-runtime values files
- Removing argo-rollouts tests and rollout-reporter references
- Updating README documentation to remove argo-rollouts sections
- Removing rollout-reporter from values-ha.yaml

BREAKING CHANGE: argo-rollouts is no longer included in the chart.
Users requiring Argo Rollouts must install it separately.
@dmaizel dmaizel requested review from a team as code owners January 19, 2026 09:42
{{- range $d := .Chart.Dependencies }}
{{- if eq $d.Name "argo-rollouts" }}
ROLLOUTS_HELM_REPOSITORY: {{ $d.Repository | quote }}
ROLLOUTS_HELM_VERSION: {{ $d.Version | quote }}
Copy link
Contributor

Choose a reason for hiding this comment

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

is the app-proxy code using these values already gone?

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.

5 participants