This is an automated email from the ASF dual-hosted git repository. rusackas pushed a commit to branch chore/helm-actions-devendor in repository https://gitbox.apache.org/repos/asf/superset.git
commit 0155e329ff2308d2ba33ccc27b9ee2a5a04051df Author: Claude Code <[email protected]> AuthorDate: Mon Jul 27 22:21:59 2026 -0700 ci: de-vendor helm/chart-releaser-action helm/chart-releaser-action is now allowlisted upstream (ASF Actions allowlist, apache/infrastructure-actions/actions.yml) at v1.7.0 (cae68fefc6b5f367a0275617c9f83181ba54714f), so the git-submodule fork under .github/actions/chart-releaser-action — added as a workaround when it wasn't allowlisted — is no longer needed. Verified with a real CI run (install_only, no release side effects) before landing this. Also drops the now-dead debug step that cat'd the vendored action's action.yml. helm/chart-testing-action stays vendored for now: it's allowlisted at v2.8.0 (6ec842c01de15ebb84c8627d2744a0c2f2755c9f), but that release depends internally on astral-sh/[email protected], which isn't itself on the allowlist (only v8.1.0+ are). De-vendoring it needs an INFRA request to add that setup-uv SHA first — confirmed by testing it directly, not just reading the config. --- .github/actions/chart-releaser-action | 1 - .github/workflows/superset-helm-lint-test.yml | 5 +++++ .github/workflows/superset-helm-release.yml | 7 +------ .gitmodules | 3 --- 4 files changed, 6 insertions(+), 10 deletions(-) diff --git a/.github/actions/chart-releaser-action b/.github/actions/chart-releaser-action deleted file mode 160000 index a917fd15b20..00000000000 --- a/.github/actions/chart-releaser-action +++ /dev/null @@ -1 +0,0 @@ -Subproject commit a917fd15b20e8b64b94d9158ad54cd6345335584 diff --git a/.github/workflows/superset-helm-lint-test.yml b/.github/workflows/superset-helm-lint-test.yml index db59101bfa9..18d18ff4ef2 100644 --- a/.github/workflows/superset-helm-lint-test.yml +++ b/.github/workflows/superset-helm-lint-test.yml @@ -38,6 +38,11 @@ jobs: with: install-superset: "false" + # Still vendored (not de-vendored like chart-releaser-action below): the + # allowlisted helm/[email protected] depends internally on + # astral-sh/[email protected], which isn't itself on the ASF Actions + # allowlist (only v8.1.0+ are, at apache/infrastructure-actions' + # actions.yml). Needs an INFRA request before this can de-vendor too. - name: Set up chart-testing uses: ./.github/actions/chart-testing-action diff --git a/.github/workflows/superset-helm-release.yml b/.github/workflows/superset-helm-release.yml index 0b07a26c3da..8ba1cc63227 100644 --- a/.github/workflows/superset-helm-release.yml +++ b/.github/workflows/superset-helm-release.yml @@ -86,13 +86,8 @@ jobs: # Return to the original branch git checkout local_gha_temp - - name: Fetch/list all tags - run: | - git submodule update - cat .github/actions/chart-releaser-action/action.yml - - name: Run chart-releaser - uses: ./.github/actions/chart-releaser-action + uses: helm/chart-releaser-action@cae68fefc6b5f367a0275617c9f83181ba54714f # v1.7.0 with: version: v1.6.0 charts_dir: helm diff --git a/.gitmodules b/.gitmodules index 54a0fb6b8f4..39e51236da9 100644 --- a/.gitmodules +++ b/.gitmodules @@ -30,9 +30,6 @@ [submodule ".github/actions/chart-testing-action"] path = .github/actions/chart-testing-action url = https://github.com/helm/chart-testing-action -[submodule ".github/actions/chart-releaser-action"] - path = .github/actions/chart-releaser-action - url = https://github.com/helm/chart-releaser-action [submodule ".github/actions/github-action-push-to-another-repository"] path = .github/actions/github-action-push-to-another-repository url = https://github.com/cpina/github-action-push-to-another-repository
