Skip to content

Commit

Permalink
chore: improve workflows and helm chart (eclipse-tractusx#51)
Browse files Browse the repository at this point in the history
- chore: combine helm chart release and image build
- chore: change to absolute paths for image build
- chore: add file header
- chore: remove unused value
- chore: improve owasp scan
- docs: change chart description
  • Loading branch information
evegufy authored Apr 3, 2024
1 parent 9033076 commit ca87837
Show file tree
Hide file tree
Showing 12 changed files with 146 additions and 316 deletions.
86 changes: 0 additions & 86 deletions .github/workflows/chart-release.yaml

This file was deleted.

8 changes: 4 additions & 4 deletions .github/workflows/chart-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ jobs:
uses: docker/build-push-action@v3
with:
context: .
file: docker/Dockerfile-credential-issuer-migrations
file: ./docker/Dockerfile-credential-issuer-migrations
push: true
tags: kind-registry:5000/credential-issuer-migrations:testing

Expand All @@ -74,7 +74,7 @@ jobs:
uses: docker/build-push-action@v3
with:
context: .
file: docker/Dockerfile-credential-issuer-service
file: ./docker/Dockerfile-credential-issuer-service
push: true
tags: kind-registry:5000/credential-issuer-service:testing

Expand All @@ -83,7 +83,7 @@ jobs:
uses: docker/build-push-action@v3
with:
context: .
file: docker/Dockerfile-credential-expiry-app
file: ./docker/Dockerfile-credential-expiry-app
push: true
tags: kind-registry:5000/credential-expiry-app:testing

Expand All @@ -92,7 +92,7 @@ jobs:
uses: docker/build-push-action@v3
with:
context: .
file: docker/Dockerfile-credential-issuer-processes-worker
file: ./docker/Dockerfile-credential-issuer-processes-worker
push: true
tags: kind-registry:5000/credential-issuer-processes-worker:testing

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/credential-expiry-app-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-expiry-app
file: ./docker/Dockerfile-credential-expiry-app
platforms: linux/amd64, linux/arm64
pull: true
push: ${{ github.event_name != 'pull_request' }}
Expand All @@ -88,4 +88,4 @@ jobs:
username: ${{ secrets.DOCKER_HUB_USER }}
password: ${{ secrets.DOCKER_HUB_TOKEN }}
repository: ${{ env.IMAGE_NAMESPACE }}/${{ env.IMAGE_NAME }}
readme-filepath: "./docker/notice-credential-expiry-app.md"
readme-filepath: ./docker/notice-credential-expiry-app.md
4 changes: 2 additions & 2 deletions .github/workflows/migrations-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-issuer-migrations
file: ./docker/Dockerfile-credential-issuer-migrations
platforms: linux/amd64, linux/arm64
pull: true
push: ${{ github.event_name != 'pull_request' }}
Expand All @@ -89,4 +89,4 @@ jobs:
username: ${{ secrets.DOCKER_HUB_USER }}
password: ${{ secrets.DOCKER_HUB_TOKEN }}
repository: ${{ env.IMAGE_NAMESPACE }}/${{ env.IMAGE_NAME }}
readme-filepath: "./docker/notice-credential-issuer-migrations.md"
readme-filepath: ./docker/notice-credential-issuer-migrations.md
15 changes: 5 additions & 10 deletions .github/workflows/owasp-zap.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-issuer-migrations
file: ./docker/Dockerfile-credential-issuer-migrations
push: true
tags: kind-registry:5000/credential-issuer-migrations:testing

Expand All @@ -74,7 +74,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-issuer-service
file: ./docker/Dockerfile-credential-issuer-service
push: true
tags: kind-registry:5000/credential-issuer-service:testing

Expand All @@ -83,7 +83,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-issuer-processes-worker
file: ./docker/Dockerfile-credential-issuer-processes-worker
push: true
tags: kind-registry:5000/credential-issuer-processes-worker:testing

Expand All @@ -92,7 +92,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-expiry-app
file: ./docker/Dockerfile-credential-expiry-app
push: true
tags: kind-registry:5000/credential-expiry-app:testing

Expand Down Expand Up @@ -142,15 +142,10 @@ jobs:
echo "... done."
- name: Add Summary
if: success() || failure()
run: |
echo "Publishing Job summary... "
cat report_md.md >> $GITHUB_STEP_SUMMARY
- name: Upload HTML report
if: success() || failure()
uses: actions/upload-artifact@5d5d22a31266ced268874388b861e4b58bb5c2f3 # v4.3.1
with:
name: ZAP scan report
path: ./report_html.html
retention-days: 1
4 changes: 2 additions & 2 deletions .github/workflows/processes-worker-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
file: docker/Dockerfile-credential-issuer-processes-worker
file: ./docker/Dockerfile-credential-issuer-processes-worker
platforms: linux/amd64, linux/arm64
pull: true
push: ${{ github.event_name != 'pull_request' }}
Expand All @@ -88,4 +88,4 @@ jobs:
username: ${{ secrets.DOCKER_HUB_USER }}
password: ${{ secrets.DOCKER_HUB_TOKEN }}
repository: ${{ env.IMAGE_NAMESPACE }}/${{ env.IMAGE_NAME }}
readme-filepath: "./docker/notice-credential-issuer-processes-worker.md"
readme-filepath: ./docker/notice-credential-issuer-processes-worker.md
Loading

0 comments on commit ca87837

Please sign in to comment.