Skip to content

Commit 8cc9265

Browse files
authored
Merge pull request #34 from Keyfactor/oauth-58225
fix(ci): Bug fix in Helm chart release action Approved on 00086751
2 parents 286e16c + e41a98e commit 8cc9265

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

.github/workflows/helm.yml

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
1-
name: Build and Release
1+
name: Build and Release Helm Chart
22
on:
3-
push:
4-
branches:
5-
- '*'
63
pull_request:
74
branches:
85
- 'release-*'
@@ -14,9 +11,6 @@ on:
1411
# pushed to the pull request's branch
1512
- synchronize
1613

17-
env:
18-
REGISTRY: ghcr.io
19-
2014
jobs:
2115
helm:
2216
runs-on: ubuntu-latest
@@ -42,14 +36,14 @@ jobs:
4236
# Set version from DOCKER_METADATA_OUTPUT_VERSION as environment variable
4337
- name: Set Version
4438
run: |
45-
echo "VERSION=${DOCKER_METADATA_OUTPUT_VERSION:1}" >> $GITHUB_ENV
39+
echo "VERSION=${DOCKER_METADATA_OUTPUT_VERSION:8}.0" >> $GITHUB_ENV # Eventually will build this into Keyfactor bootstrap
4640
4741
# Change version and appVersion in Chart.yaml to the tag in the closed PR
4842
- name: Update Helm App/Chart Version
4943
shell: bash
5044
run: |
5145
sed -i "s/^version: .*/version: ${{ env.VERSION }}/g" deploy/charts/command-cert-manager-issuer/Chart.yaml
52-
sed -i "s/^appVersion: .*/appVersion: \"${{ env.DOCKER_METADATA_OUTPUT_VERSION }}\"/g" deploy/charts/command-cert-manager-issuer/Chart.yaml
46+
sed -i "s/^appVersion: .*/appVersion: \"v${{ env.VERSION }}\"/g" deploy/charts/command-cert-manager-issuer/Chart.yaml
5347
5448
# Setup Helm
5549
# https://github.com/Azure/setup-helm
@@ -72,3 +66,4 @@ jobs:
7266
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
7367
with:
7468
charts_dir: deploy/charts
69+

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,8 @@
3737

3838
## Fixes
3939
- Change Helm release trigger from `v*` to `release-*` to support Keyfactor Bootstrap Workflow
40+
41+
# v2.0.2
42+
43+
## Fixes
44+
- Bug fix in Helm chart release action

0 commit comments

Comments
 (0)