Skip to content

Commit

Permalink
ci: fix publish job
Browse files Browse the repository at this point in the history
Fixes the publish job, the the charts are pushed into the fright dircetory.
  • Loading branch information
phbelitz committed Mar 19, 2024
1 parent d510385 commit 6ca19ae
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
9 changes: 4 additions & 5 deletions .github/workflows/.reusable-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,17 +38,16 @@ jobs:
CHART_VERSION="${{ inputs.chart_version }}"
helm package charts/connaisseur
git checkout . # Remove changes to Chart for git checkout
mkdir -p tmp_charts
mv connaisseur*.tgz ./tmp_charts
git checkout gh-pages
cd tmp_charts
mv connaisseur*.tgz ./charts
cd ./charts
helm repo index . --url https://sse-secure-systems.github.io/connaisseur/charts
cd ..
git add ./tmp_charts
git add ./charts
git commit -m "Publish helm chart ${CHART_VERSION}"
git push https://${{ secrets.GITHUB_TOKEN }}@github.com/sse-secure-systems/connaisseur.git
publish_docs:
uses: ./.github/workflows/.reusable-docs.yaml
permissions:
contents: write
contents: write

Check failure

Code scanning / Scorecard

Token-Permissions High

score is 0: jobLevel 'contents' permission set to 'write'
Remediation tip: Verify which permissions are needed and consider whether you can reduce them.
Click Remediation section below for further remediation help
2 changes: 1 addition & 1 deletion .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ jobs:
permissions:
contents: write
with:
chart_version: ${{ needs.build.outputs.chart_version }}
chart_version: ${{ inputs.chart_version }}

0 comments on commit 6ca19ae

Please sign in to comment.