Skip to content

Commit

Permalink
F2calv/2023 01 disable chart push (#8)
Browse files Browse the repository at this point in the history
* re-enable

* fix
  • Loading branch information
f2calv authored Jan 11, 2023
1 parent 4889fcb commit ae46be9
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,8 +132,7 @@ jobs:
push_chart:
runs-on: ubuntu-latest
needs: [build_app, build_image]
#if: github.ref == 'refs/heads/main'
if: false #TODO: implement helm chart later
if: github.ref == 'refs/heads/main'

permissions:
packages: write #for pushing helm package
Expand All @@ -152,7 +151,7 @@ jobs:
fetch-depth: 0 #Note: shallow clone may break git tag & ct lint...

- name: set vars
run: echo "REPOSITORY=${{ github.repository_owner }}/$IMAGE_NAME" >> $GITHUB_ENV
run: echo "REPOSITORY=${{ github.repository_owner }}/charts/$IMAGE_NAME" >> $GITHUB_ENV

- name: install helm
uses: azure/setup-helm@v3
Expand Down Expand Up @@ -197,10 +196,12 @@ jobs:
- name: helm package
run: |
helm package ${{ github.workspace }}/charts/$IMAGE_NAME/ --destination ${{ github.workspace }}/artifacts/ --app-version ${{ env.GIT_TAG }}
if: false #not implemented/working yet

- name: helm push
run: |
helm push ${{ github.workspace }}/artifacts/$IMAGE_NAME-0.1.0.tgz oci://ghcr.io/${{ github.repository_owner }}
if: false #not implemented/working yet, need to change the chart version in filename

release:
needs: [build_app, build_image, push_chart]
Expand Down

0 comments on commit ae46be9

Please sign in to comment.