You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ru...@apache.org on 2026/07/28 05:22:36 UTC
(superset) branch chore/helm-actions-devendor updated (b62bb365086 -> 0155e329ff2)
This is an automated email from the ASF dual-hosted git repository.
rusackas pushed a change to branch chore/helm-actions-devendor
in repository https://gitbox.apache.org/repos/asf/superset.git
discard b62bb365086 diagnostic: isolated job to test chart-releaser-action v1.7.0 allowlist (install_only)
discard 0777a348cdc diagnostic: try latest chart-testing-action SHA (v2.8.0) to test allowlist match
discard 4b6bd217d25 ci: de-vendor helm/chart-testing-action and helm/chart-releaser-action
new 0155e329ff2 ci: de-vendor helm/chart-releaser-action
This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version. This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:
* -- * -- B -- O -- O -- O (b62bb365086)
\
N -- N -- N refs/heads/chore/helm-actions-devendor (0155e329ff2)
You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.
Any revisions marked "omit" are not gone; other references still
refer to them. Any revisions marked "discard" are gone forever.
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.github/actions/chart-testing-action | 1 +
.github/workflows/superset-helm-lint-test.yml | 23 ++++++-----------------
.github/workflows/superset-helm-release.yml | 2 +-
.gitmodules | 3 +++
4 files changed, 11 insertions(+), 18 deletions(-)
create mode 160000 .github/actions/chart-testing-action
(superset) 01/01: ci: de-vendor helm/chart-releaser-action
Posted by ru...@apache.org.
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 <no...@anthropic.com>
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/setup-uv@v7.0.0, 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/chart-testing-action@v2.8.0 depends internally on
+ # astral-sh/setup-uv@v7.0.0, 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