Don't clobber existing helm chart in release! (#2643)
Switch to different github release action: - avoids clobbering existing release if already published, updates existing draft only with latest Helm chart - also sets name to `Browsertrix <version>`, fills in changelist. - fixes #2642 Tested: - New draft release created (since branch ends in `-release`) - Running multiple types to ensure chart is updated in draft - Switching to older release to ensure chart is *NOT* clobbered
This commit is contained in:
parent
0e06ccd746
commit
f5c120b529
17
.github/workflows/publish-helm-chart.yaml
vendored
17
.github/workflows/publish-helm-chart.yaml
vendored
@ -1,4 +1,4 @@
|
|||||||
name: Publish Helm Chart
|
name: Make Draft + Upload Helm Chart
|
||||||
|
|
||||||
on:
|
on:
|
||||||
push:
|
push:
|
||||||
@ -30,7 +30,7 @@ jobs:
|
|||||||
echo "version=$(cat version.txt)" >> "$GITHUB_ENV"
|
echo "version=$(cat version.txt)" >> "$GITHUB_ENV"
|
||||||
|
|
||||||
- name: Upload Chart To GitHub
|
- name: Upload Chart To GitHub
|
||||||
uses: softprops/action-gh-release@v1
|
uses: ncipollo/release-action@v1
|
||||||
with:
|
with:
|
||||||
body: |
|
body: |
|
||||||
## Deployment
|
## Deployment
|
||||||
@ -50,9 +50,12 @@ jobs:
|
|||||||
|
|
||||||
See [the development guide](https://docs.browsertrix.com/deploy/) for more info how to deploy Browsertrix.
|
See [the development guide](https://docs.browsertrix.com/deploy/) for more info how to deploy Browsertrix.
|
||||||
|
|
||||||
files: |
|
allowUpdates: true
|
||||||
.chart-out/browsertrix-v${{ env.version }}.tgz
|
artifactErrorsFailBuild: true
|
||||||
.chart-out/btrix-proxies-0.1.0.tgz
|
artifacts: .chart-out/browsertrix-v${{ env.version }}.tgz,.chart-out/btrix-proxies-0.1.0.tgz
|
||||||
tag_name: v${{ env.version }}
|
|
||||||
draft: true
|
draft: true
|
||||||
fail_on_unmatched_files: true
|
generateReleaseNotes: true
|
||||||
|
name: Browsertrix v${{ env.version }}
|
||||||
|
skipIfReleaseExists: true
|
||||||
|
tag: v${{ env.version }}
|
||||||
|
updateOnlyUnreleased: true
|
||||||
|
Loading…
Reference in New Issue
Block a user