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 4b6bd217d254dc539e9e1cc41c96216f7226918d Author: Claude Code <[email protected]> AuthorDate: Mon Jul 27 22:12:43 2026 -0700 ci: de-vendor helm/chart-testing-action and helm/chart-releaser-action Both are now allowlisted upstream at pinned SHAs, so the git-submodule forks under .github/actions/ (added as a workaround when they weren't allowlisted) are no longer needed. Pins match what the submodules were already pinned to (v2.3.1 / v1.6.0) — this is a mechanical de-vendor, not a version bump. Also drops the now-dead debug step in superset-helm-release.yml that cat'd the vendored chart-releaser-action's action.yml. --- .github/actions/chart-releaser-action | 1 - .github/actions/chart-testing-action | 1 - .github/workflows/superset-helm-lint-test.yml | 6 +++++- .github/workflows/superset-helm-release.yml | 7 +------ .gitmodules | 6 ------ 5 files changed, 6 insertions(+), 15 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/actions/chart-testing-action b/.github/actions/chart-testing-action deleted file mode 160000 index afea100a513..00000000000 --- a/.github/actions/chart-testing-action +++ /dev/null @@ -1 +0,0 @@ -Subproject commit afea100a513515fbd68b0e72a7bb0ae34cb62aec diff --git a/.github/workflows/superset-helm-lint-test.yml b/.github/workflows/superset-helm-lint-test.yml index db59101bfa9..0d137098d84 100644 --- a/.github/workflows/superset-helm-lint-test.yml +++ b/.github/workflows/superset-helm-lint-test.yml @@ -5,6 +5,10 @@ on: types: [opened, reopened, synchronize] paths: - "helm/**" + # Spike-only: forces a run on demand without a helm/** diff, to validate the + # de-vendored chart-testing-action against the org's action allowlist. + # Remove once validated. + workflow_dispatch: permissions: contents: read @@ -39,7 +43,7 @@ jobs: install-superset: "false" - name: Set up chart-testing - uses: ./.github/actions/chart-testing-action + uses: helm/chart-testing-action@afea100a513515fbd68b0e72a7bb0ae34cb62aec # v2.3.1 - name: Run chart-testing (list-changed) id: list-changed diff --git a/.github/workflows/superset-helm-release.yml b/.github/workflows/superset-helm-release.yml index 0b07a26c3da..bb77526020c 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@a917fd15b20e8b64b94d9158ad54cd6345335584 # v1.6.0 with: version: v1.6.0 charts_dir: helm diff --git a/.gitmodules b/.gitmodules index 54a0fb6b8f4..436dfa5a8f1 100644 --- a/.gitmodules +++ b/.gitmodules @@ -27,12 +27,6 @@ [submodule ".github/actions/comment-on-pr"] path = .github/actions/comment-on-pr url = https://github.com/unsplash/comment-on-pr -[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
