Skip to content

Fix Helm chart workflow action policy - #28

Merged
abhishekrb19 merged 1 commit into
apache:masterfrom
razinbouzar:fix-helm-chart-workflow-action-policy
Jun 15, 2026
Merged

abhishekrb19 merged 1 commit into
apache:masterfrom
razinbouzar:fix-helm-chart-workflow-action-policy

Conversation

@razinbouzar

Copy link
Copy Markdown
Contributor

This PR replaces the Helm chart publishing workflow added in PR #26 with a publishing mechanism that works with Apache repository constraints.

The first workflow failed at startup because Apache's GitHub Actions policy blocks azure/setup-helm@v4. No chart artifacts were published.

We should keep the same public Helm repo contract:

https://apache.github.io/druid-operator

Expected gh-pages layout:

index.yaml
helm-releases/druid-operator-<version>.tgz

PR #27 already cleaned gh-pages, so this PR should only update how charts are published going forward.

Validation before publishing:

make helm-lint
make helm-template
make helm-package
make rat

Validation after publishing:

curl -I https://apache.github.io/druid-operator/index.yaml
helm repo add apache-druid https://apache.github.io/druid-operator
helm repo update
helm search repo apache-druid/druid-operator

Fixes #XXXX.

Description


This PR has:

  • been tested on a real K8S cluster to ensure creation of a brand new Druid cluster works.
  • been tested for backward compatibility on a real K*S cluster by applying the changes introduced here on an existing Druid cluster. If there are any backward incompatible changes then they have been noted in the PR description.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

Key changed/added files in this PR
  • MyFoo
  • OurBar
  • TheirBaz

@abhishekrb19
abhishekrb19 merged commit 9c5d9f2 into apache:master Jun 15, 2026
2 checks passed
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.

2 participants