Skip to content

Commit

Permalink
fetch-and-ingest-branch: Send Slack notications to test channel
Browse files Browse the repository at this point in the history
This change was motivated by the unintentional bug introduced in
#222 that would only be triggered
by using Slack notifications. This allows to test branches and send
notifications to the testing channel.

As part of this change, I've added an organization level variable
`TEST_SLACK_CHANNEL` that points our #scratch channel for testing
Slack notifications.
  • Loading branch information
joverlee521 committed Nov 15, 2023
1 parent 0ba58eb commit a03f06d
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .github/workflows/fetch-and-ingest-branch.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ jobs:
runtime: aws-batch
env: |
NEXTSTRAIN_DOCKER_IMAGE: ${{ inputs.image }}
GITHUB_RUN_ID: ${{ github.run_id }}
SLACK_CHANNELS: ${{ vars.TEST_SLACK_CHANNEL }}
UPLOAD_CONFIG: ${{ needs.set_config_overrides.outputs.upload_config }}
run: |
nextstrain build \
Expand All @@ -44,6 +46,9 @@ jobs:
--memory 64gib \
--env AWS_ACCESS_KEY_ID \
--env AWS_SECRET_ACCESS_KEY \
--env GITHUB_RUN_ID \
--env SLACK_TOKEN \
--env SLACK_CHANNELS \
ingest \
--configfiles config/config.yaml config/optional.yaml \
--config trigger_rebuild=False send_slack_notifications=False upload="$UPLOAD_CONFIG"
--config trigger_rebuild=False send_slack_notifications=True upload="$UPLOAD_CONFIG"

0 comments on commit a03f06d

Please sign in to comment.