Skip to content

Commit

Permalink
[GHA] Remove unneeded step, fix not using build-env in some steps, ad…
Browse files Browse the repository at this point in the history
…d missing chart repo
  • Loading branch information
meln5674 committed Jul 24, 2024
1 parent 290a091 commit 1eaec57
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions .github/workflows/cd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,18 +33,14 @@ jobs:
- name: Checkout repository
uses: actions/checkout@v3

- uses: actions/setup-go@v4
with:
go-version: '^${{ env.GO_VERSION }}'

- name: Install Dependencies
run: |
make -j deps all-helm-tools chart-deps
./build-env.sh make -j deps all-helm-tools chart-deps
# Run Linter
- name: Lint standalone chart
run: |
make helm-hog
./build-env.sh make helm-hog
test:
# runs-on: ubuntu-latest
Expand Down Expand Up @@ -202,6 +198,7 @@ jobs:
./build-env.sh make -j all-helm-tools
./build-env.sh helm repo add jetstack https://charts.jetstack.io
./build-env.sh helm repo add zalando https://opensource.zalando.com/postgres-operator/charts/postgres-operator
./build-env.sh helm repo add mino https://charts.min.io/
./build-env.sh make -j chart-deps
- name: Log into Registry
Expand All @@ -221,6 +218,6 @@ jobs:
GITHUB_TOKEN: '${{ secrets.GITHUB_TOKEN }}'
run: |
for chart in bin/charts/*.tgz ; do
helm push "${chart}" "${HELM_REPO}"
./build-env.sh helm push "${chart}" "${HELM_REPO}"
done

0 comments on commit 1eaec57

Please sign in to comment.