Skip to content

revert(template): Publish to quay.io (#584) - #593

Merged
Techassi merged 1 commit into
mainfrom
revert/pr-584
May 26, 2026
Merged

revert(template): Publish to quay.io (#584)#593
Techassi merged 1 commit into
mainfrom
revert/pr-584

Conversation

@Techassi

Copy link
Copy Markdown
Member

Part of stackabletech/issues#716.

This reverts commit 96368df.

This change is reverted because the current default Helm version used by the publish-helm-chart action errors out when trying to push Helm Charts to quay.io. This issue is decribed in the upstream issue: helm/helm#32144.

The only option we currently would have had was to downgrade to Helm 3.17.x. We decided to wait a little longer for either a) a new, fixed Helm version, or b) to downgrade until a fixed version is released.

This revert is needed to unblock other changes in this repo to be rolled out without breaking CI.

This reverts commit 96368df.
This change is reverted because the current default Helm version
used by the publish-helm-chart action errors out when trying to
push Helm Charts to quay.io. This issue is decribed in the upstream
issue: helm/helm#32144.
The only option we currently would have had was to downgrade to
Helm 3.17.x. We decided to wait a little longer for either a) a
new, fixed Helm version, or b) to downgrade until a fixed version
is released.
This revert is needed to unblock other changes in this repo to be
rolled out without breaking CI.
@TechassiTechassi self-assigned this May 26, 2026
@TechassiTechassi moved this to Development: Waiting for Review in Stackable EngineeringMay 26, 2026
@NickLarsenNZNickLarsenNZ moved this from Development: Waiting for Review to Development: In Review in Stackable EngineeringMay 26, 2026
@NickLarsenNZ
NickLarsenNZ self-requested a review May 26, 2026 08:12

@NickLarsenNZNickLarsenNZ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Techassi
Techassi added this pull request to the merge queueMay 26, 2026
@TechassiTechassi moved this from Development: In Review to Development: Done in Stackable EngineeringMay 26, 2026
Merged via the queue into main with commit 5683e84May 26, 2026
2 checks passed
@Techassi
Techassi deleted the revert/pr-584 branch May 26, 2026 09:04
@lfranckelfrancke moved this from Development: Done to Done in Stackable EngineeringMay 28, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

@Techassi@NickLarsenNZ@lfrancke