Skip to content

Commit

Permalink
ci: upgrade upload-artifact action to v4 (#464)
Browse files Browse the repository at this point in the history
ci: fix when update-repo is auto run

---------

Co-authored-by: Ana-Maria Calin <calina1391@gmail.com>
  • Loading branch information
cjorge-graphops and calinah authored Jan 28, 2025
1 parent 6fa8b23 commit d70869f
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 16 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/make-chart-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ jobs:
corepack yarn git-cliff -c ".cliff.toml" --include-path "charts/$CHART/**" --strip all -l -t "${{ needs.init.outputs.tag }}" -o "RELEASE-NOTES.md"
- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: "${{ needs.init.outputs.tag }}.release-notes"
path: RELEASE-NOTES.md
Expand Down Expand Up @@ -120,7 +120,7 @@ jobs:
helm package "charts/${{ needs.init.outputs.name }}"
- name: Upload artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: "${{ needs.init.outputs.tag }}.helm-package"
path: "${{ needs.init.outputs.tag }}.tgz"
Expand All @@ -144,7 +144,7 @@ jobs:
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: "${{ needs.init.outputs.tag }}.release-notes"
path: "."
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/make-charts-index.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
repo: ${{ github.event.repository.name }}
owner: ${{ github.event.repository.owner.login }}

- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: releases

Expand Down Expand Up @@ -70,7 +70,7 @@ jobs:
.releases/index.yaml
done
- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: "chart-index.${{ inputs.repoName }}"
path: .releases/index.yaml
5 changes: 1 addition & 4 deletions .github/workflows/trigger-push-canary.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,6 @@ jobs:
uses: ./.github/workflows/update-helm-repo.yaml
permissions:
contents: write # for updating index.yaml
strategy:
matrix:
repo: ${{ fromJson(needs.find-tags-to-release.outputs.release-streams) }}
if: ${{ needs.find-tags-to-release.outputs.release-streams != '[]' }}
with:
repo: ${{ matrix.repo }}
repo: "canary"
7 changes: 1 addition & 6 deletions .github/workflows/trigger-push.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,12 +86,7 @@ jobs:
uses: ./.github/workflows/update-helm-repo.yaml
permissions:
contents: write # for updating index.yaml
strategy:
matrix:
repo: ${{ fromJson(needs.find-tags-to-release.outputs.release-streams) }}
#if: ${{ needs.find-tags-to-release.outputs.release-streams != '[]' }}
#with:
# repo: ${{ matrix.repo }}
if: ${{ contains(needs.find-tags-to-release.outputs.release-streams, 'canary') }}
# update only canary repo, as stable releases are being set as draft
with:
repo: canary
2 changes: 1 addition & 1 deletion .github/workflows/update-helm-repo.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
git config user.name "${{ github.actor }}"
git config user.email "${{ github.actor}}@users.noreply.github.com"
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: "chart-index.${{ inputs.repo }}"
path: ".nindex"
Expand Down

0 comments on commit d70869f

Please sign in to comment.