8000 Fix Helm Chart Build by nigthknight · Pull Request #319 · secureCodeBox/secureCodeBox · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@nigthknight
Copy link
Member

We noticed that our current release process did not use the right docker
images anymore. This problem was created after switching to the new
build and push docker action because the images are now tagged with a
valid semver instead of a version with a leading 'v' but the release
tried to use the 'old' version style with a leading 'v'.

To resolve this issue we decided to trim the leading 'v' in our Release
Action because we want to have consistent tags in GitHub

Signed-off-by: Yannik Fuhrmeister yannik.fuhrmeister@iteratec.com

We noticed that our current release process did not use the right docker
images anymore. This problem was created after switching to the new
build and push docker action because the images are now tagged with a
valid semver instead of a version with a leading 'v' but the release
tried to use the 'old' version style with a leading 'v'.

To resolve this issue we decided to trim the leading 'v' in our Release
Action because we want to have consistent tags in Github

Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
@nigthknight nigthknight force-pushed the bugfix/helm-chart-version branch from f4e60ea to 3d01d5e Compare March 8, 2021 09:59
@nigthknight nigthknight changed the title Del leading 'v' In git-tag To create valid Semver Fix Helm Chart Build Mar 8, 2021
@nigthknight nigthknight enabled auto-merge March 8, 2021 10:03
@nigthknight nigthknight merged commit 4de4a5c into main Mar 8, 2021
@nigthknight nigthknight deleted the bugfix/helm-chart-version branch March 8, 2021 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0