Skip to content

Commit

Permalink
chore(ci): add digest for images of the pr (#1736)
Browse files Browse the repository at this point in the history
  • Loading branch information
0fatal authored Dec 19, 2023
1 parent 821a273 commit eb25b73
Show file tree
Hide file tree
Showing 4 changed files with 33 additions and 27 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/dockerize-runtime-exporter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ on:
# pull-requests: write
# packages: write
# contents: read

concurrency:
group: dockerize-runtime-exporter-${{ github.ref }}
cancel-in-progress: true
Expand Down Expand Up @@ -72,6 +72,7 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}

- name: Build and push
id: docker-build
uses: docker/build-push-action@v3
with:
context: ./services/runtime-exporter
Expand All @@ -85,12 +86,13 @@ jobs:
if: github.event_name == 'pull_request_target'
uses: mshick/add-pr-comment@v2
with:
message-id: 'dockerize-runtime-exporter'
message-id: "dockerize-runtime-exporter"
message: |
**:boom: laf runtime exporter image built successfully for pr ${{ github.event.pull_request.number }}**
```
${{ steps.meta.outputs.tags }}
```
digest: `${{ steps.docker-build.outputs.digest }}`
trigger-workflow-build-cluster-image:
needs: [dockerize-runtime-exporter]
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/dockerize-runtime-nodejs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ on:
# pull-requests: write
# packages: write
# contents: read

concurrency:
group: dockerize-runtime-nodejs-${{ github.ref }}
cancel-in-progress: true
Expand Down Expand Up @@ -88,6 +88,7 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}

- name: Build and push
id: docker-build
uses: docker/build-push-action@v3
with:
context: ./runtimes/nodejs
Expand All @@ -101,12 +102,13 @@ jobs:
if: github.event_name == 'pull_request_target'
uses: mshick/add-pr-comment@v2
with:
message-id: 'dockerize-runtime-nodejs'
message-id: "dockerize-runtime-nodejs"
message: |
**:boom: laf nodejs runtime image built successfully for pr ${{ github.event.pull_request.number }}**
```
${{ steps.meta.outputs.tags }}
```
digest: `${{ steps.docker-build.outputs.digest }}`
dockerize-init-image:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -165,15 +167,13 @@ jobs:
if: github.event_name == 'pull_request_target'
uses: mshick/add-pr-comment@v2
with:
message-id: 'dockerize-runtime-nodejs-init'
message-id: "dockerize-runtime-nodejs-init"
message: |
**:boom: laf nodejs runtime init image built successfully for pr ${{ github.event.pull_request.number }}**
```
${{ steps.meta.outputs.tags }}
```

# trigger-workflow-e2e:
# needs: [dockerize-main-image]
# runs-on: ubuntu-latest
Expand Down
22 changes: 12 additions & 10 deletions .github/workflows/dockerize-server.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,17 @@ on:
branches: [main]
types: [opened, synchronize, reopened]
paths:
- 'server/**'
- '.github/workflows/dockerize-server.yml'
- '!**/*.md'
- '!server/package-lock.json'
- "server/**"
- ".github/workflows/dockerize-server.yml"
- "!**/*.md"
- "!server/package-lock.json"
push:
branches: [main]
paths:
- 'server/**'
- '.github/workflows/dockerize-server.yml'
- '!**/*.md'
- '!server/package-lock.json'
- "server/**"
- ".github/workflows/dockerize-server.yml"
- "!**/*.md"
- "!server/package-lock.json"

# permissions:
# pull-requests: write
Expand Down Expand Up @@ -93,6 +93,7 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}

- name: Build and push
id: docker-build
uses: docker/build-push-action@v3
with:
context: ./server
Expand All @@ -101,17 +102,18 @@ jobs:
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
platforms: linux/amd64, linux/arm64

- name: Comment docker image tags to PR
if: github.event_name == 'pull_request_target'
uses: mshick/add-pr-comment@v2
with:
message-id: 'dockerize-server'
message-id: "dockerize-server"
message: |
**:boom: laf server image built successfully for pr ${{ github.event.pull_request.number }}**
```
${{ steps.meta.outputs.tags }}
```
digest: `${{ steps.docker-build.outputs.digest }}`
trigger-workflow-build-cluster-image:
needs: [dockerize-server]
Expand Down
22 changes: 12 additions & 10 deletions .github/workflows/dockerize-web.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,18 @@ on:
branches: [main]
types: [opened, synchronize, reopened]
paths:
- 'web/**'
- '.github/workflows/dockerize-web.yml'
- '!**/*.md'
- '!web/package-lock.json'
- "web/**"
- ".github/workflows/dockerize-web.yml"
- "!**/*.md"
- "!web/package-lock.json"
push:
branches:
- main
paths:
- 'web/**'
- '.github/workflows/dockerize-web.yml'
- '!**/*.md'
- '!web/package-lock.json'
- "web/**"
- ".github/workflows/dockerize-web.yml"
- "!**/*.md"
- "!web/package-lock.json"

# permissions:
# pull-requests: write
Expand Down Expand Up @@ -72,7 +72,7 @@ jobs:
SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
with:
sourcemaps: dist/assets
url_prefix: '~/assets/'
url_prefix: "~/assets/"
version: ${{ format('laf@{0}', github.sha) }}
working_directory: web
ignore_missing: true
Expand Down Expand Up @@ -114,6 +114,7 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}

- name: Build and push
id: docker-build
uses: docker/build-push-action@v3
with:
context: ./web
Expand All @@ -127,12 +128,13 @@ jobs:
if: github.event_name == 'pull_request_target'
uses: mshick/add-pr-comment@v2
with:
message-id: 'dockerize-web'
message-id: "dockerize-web"
message: |
**:boom: laf web image built successfully for pr ${{ github.event.pull_request.number }}**
```
${{ steps.meta.outputs.tags }}
```
digest: `${{ steps.docker-build.outputs.digest }}`
trigger-workflow-build-cluster-image:
needs: [dockerize-web]
Expand Down

0 comments on commit eb25b73

Please sign in to comment.