This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-kubernetes-operator.git
The following commit(s) were added to refs/heads/main by this push:
new e54e4f5 [SPARK-49362] Simplify snapshot HelmChart to use
`apache/spark-kubernetes-operator:main-snapshot` by default
e54e4f5 is described below
commit e54e4f5e9fda12382e2ca7d8ff64f1ec0e2e0aa4
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Aug 22 16:56:42 2024 -0700
[SPARK-49362] Simplify snapshot HelmChart to use
`apache/spark-kubernetes-operator:main-snapshot` by default
### What changes were proposed in this pull request?
This PR aims to simplify snapshot HelmChart to use
`apache/spark-kubernetes-operator:main-snapshot` by default.
### Why are the changes needed?
To help users use it without overriding via `--set`.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Manual review.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #95 from dongjoon-hyun/SPARK-49362.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.github/workflows/publish_snapshot_chart.yml | 11 +++++++----
README.md | 3 +--
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/.github/workflows/publish_snapshot_chart.yml
b/.github/workflows/publish_snapshot_chart.yml
index 1f1edea..42680a9 100644
--- a/.github/workflows/publish_snapshot_chart.yml
+++ b/.github/workflows/publish_snapshot_chart.yml
@@ -38,14 +38,17 @@ jobs:
- name: Build Chart
run: |
cd build-tools/helm
- mkdir charts
- tar cvfz charts/spark-kubernetes-operator-0.1.0-SNAPSHOT.tgz
spark-kubernetes-operator
- helm repo index charts --url https://nightlies.apache.org/spark/charts
+ sed -i 's/repository: /repository: apache\//'
build-tools/helm/spark-kubernetes-operator/values.yaml
+ sed -i 's/tag: .*$/tag: main-snapshot/'
spark-kubernetes-operator/values.yaml
+ mkdir -p tmp/charts
+ helm package spark-kubernetes-operator -d tmp/charts --app-version
main-snapshot
+ helm repo index tmp/charts --url
https://nightlies.apache.org/spark/charts
+ helm show chart tmp/charts/spark-kubernetes-operator-*.tgz
- name: Upload
uses: burnett01/[email protected]
with:
switches: -avzr
- path: build-tools/helm/*
+ path: build-tools/helm/tmp/*
remote_path: ${{ secrets.NIGHTLIES_RSYNC_PATH }}/spark
remote_host: ${{ secrets.NIGHTLIES_RSYNC_HOST }}
remote_port: ${{ secrets.NIGHTLIES_RSYNC_PORT }}
diff --git a/README.md b/README.md
index 8fc6422..a4d89f2 100644
--- a/README.md
+++ b/README.md
@@ -114,8 +114,7 @@ As of now, you can try `spark-kubernetes-operator` nightly
version in the follow
```
$ helm install spark-kubernetes-operator \
-https://nightlies.apache.org/spark/charts/spark-kubernetes-operator-0.1.0-SNAPSHOT.tgz
\
---set image.repository=dongjoon/spark-kubernetes-operator
+https://nightlies.apache.org/spark/charts/spark-kubernetes-operator-0.1.0-SNAPSHOT.tgz
```
## Contributing
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
