Skip to content

Commit

Permalink
fix: split up build and deploy steps
Browse files Browse the repository at this point in the history
  • Loading branch information
firestack committed Oct 4, 2024
1 parent 2c26b55 commit 4951d75
Showing 1 changed file with 37 additions and 19 deletions.
56 changes: 37 additions & 19 deletions .github/workflows/deploy-base.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,37 +18,25 @@ on:
SLACK_WEBHOOK:
required: true


jobs:
deploy:
name: Deploy
build:
name: Build/push Docker Image
runs-on: ubuntu-latest
permissions:
id-token: write
contents: read
environment: ${{ inputs.env }}
env:
ECS_CLUSTER: skate
ECS_SERVICE: skate-${{ inputs.env }}
outputs:
docker-tag-suffix: ${{ steps.build-push.outputs.docker-tag-suffix }}
sentry-release: ${{ steps.version-ids.outputs.sentry-release }}
steps:
- name: Configure AWS credentials
uses: aws-actions/configure-aws-credentials@v4
with:
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
aws-region: us-east-1
- uses: actions/checkout@v4
- name: Get version ids
id: version-ids
run: |
echo "sentry-release=${{github.ref}}_${{github.sha}}" | tr / - >> "$GITHUB_OUTPUT"
- uses: getsentry/action-release@v1
env:
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
with:
environment: ${{ inputs.env }}
version: ${{steps.version-ids.outputs.sentry-release}}
ignore_missing: true
- uses: mbta/actions/build-push-ecr@v2
id: build-push
with:
Expand All @@ -61,13 +49,43 @@ jobs:
SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}


deploy_ecs:
name: Deploy (ECS)
runs-on: ubuntu-latest
needs: build
permissions:
id-token: write
contents: read
environment: ${{ inputs.env }}
env:
ECS_CLUSTER: skate
ECS_SERVICE: skate-${{ inputs.env }}
steps:
- uses: getsentry/action-release@v1
env:
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
with:
environment: ${{ inputs.env }}
version: ${{needs.build.outputs.sentry-release}}
ignore_missing: true
- uses: mbta/actions/deploy-ecs@v2
with:
role-to-assume: ${{ secrets.AWS_ROLE_ARN }}
ecs-cluster: ${{ env.ECS_CLUSTER }}
ecs-service: ${{ env.ECS_SERVICE }}
docker-tag: ${{ steps.build-push.outputs.docker-tag }}
- uses: mbta/actions/notify-slack-deploy@v1

notify:
name: Notify
runs-on: ubuntu-latest
needs: deploy_ecs
if: always()
steps:
- uses: mbta/actions/notify-slack-deploy@v2
if: ${{ !cancelled() }}
with:
webhook-url: ${{ secrets.SLACK_WEBHOOK }}
Expand Down

0 comments on commit 4951d75

Please sign in to comment.