-
Notifications
You must be signed in to change notification settings - Fork 17
Add Flower alongside live environment Celery workers #1617
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 13 commits
f967bdb
07b980f
c8a6941
df37314
7fae63b
8c93efa
fb419bf
8b7f412
6900277
a4bba21
f583dee
0b23beb
94a1d1b
0642c12
8b410b8
c78ef57
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -98,17 +98,10 @@ jobs: | |
| # to do with the GHA workflow "env" settings. | ||
| export PATH="/home/runner/.pulumi/bin:$PATH" | ||
|
|
||
| # Create a YAML config stump containing only the nested tree leading to the image tag update | ||
|
|
||
| # Create a YAML config stump containing only the new image tag as a YAML variable | ||
| cd pulumi | ||
| cat << EOF > newimage.yaml | ||
| resources: | ||
| tb:fargate:FargateClusterWithLogging: | ||
| backend: | ||
| task_definition: | ||
| container_definitions: | ||
| backend: | ||
| image: "${{ steps.pulumi-tag-extract.outputs.pulumi_tag }}" | ||
| EOF | ||
| echo ".apmt_image: &APMT_IMAGE ${{ steps.pulumi-tag-extract.outputs.pulumi_tag }}" > newimage.yaml | ||
|
|
||
| # Use yq to merge the stump into the main config | ||
| yq -i '. *= load("newimage.yaml")' config.prod.yaml | ||
|
|
@@ -118,8 +111,9 @@ jobs: | |
| pulumi login | ||
| pulumi stack select prod | ||
| TBPULUMI_DISABLE_PROTECTION=True \ | ||
| pulumi up -y --diff --target \ | ||
| 'urn:pulumi:prod::appointment::tb:fargate:FargateClusterWithLogging$aws:ecs/taskDefinition:TaskDefinition::appointment-prod-fargate-backend-taskdef' \ | ||
| pulumi up -y --diff \ | ||
| --target 'urn:pulumi:stage::appointment::tb:fargate:FargateClusterWithLogging$aws:ecs/taskDefinition:TaskDefinition::appointment-stage-fargate-backend-taskdef' \ | ||
| --target 'urn:pulumi:stage::appointment::tb:fargate:AutoscalingFargateCluster::appointment-stage-afc-appointment' \ | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This makes sure that images get deployed to both clusters when we do a release.
davinotdavid marked this conversation as resolved.
Outdated
|
||
| --target-dependents | ||
|
|
||
| prod-sanity-browserstack: | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -36,6 +36,17 @@ services: | |
| - backend | ||
| - redis | ||
|
|
||
| celery-flower: | ||
| <<: *backend | ||
| ports: | ||
| - 5556:5555 | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Davi requested this port exposure change so that this does not conflict with a simultaneously running Flower container for a local dev instance of Accounts. |
||
| environment: | ||
| - CONTAINER_ROLE=flower | ||
| - FLOWER_UNAUTHENTICATED_API=true | ||
| depends_on: | ||
| - backend | ||
| - redis | ||
|
|
||
| frontend: | ||
| build: ./frontend | ||
| volumes: | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is simpler now due to the DRYing out of the task definitions.