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.
@Techassi Techassi self-assigned this May 26, 2026
@Techassi Techassi moved this to Development: Waiting for Review in Stackable Engineering May 26, 2026
@NickLarsenNZ NickLarsenNZ moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering May 26, 2026
@NickLarsenNZ NickLarsenNZ self-requested a review May 26, 2026 08:12
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

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 queue May 26, 2026
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering May 26, 2026
Merged via the queue into main with commit 5683e84 May 26, 2026
2 checks passed
@Techassi Techassi deleted the revert/pr-584 branch May 26, 2026 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

2 participants