Skip to content
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

feat: use same image for cas api/workers and send worker discord notifs #60

Conversation

smrz2001
Copy link
Collaborator

  • Added exit code when checking for ECS task status, so that we can detect non-zero exit codes.
  • Remove list of anchor workers from in-progress jobs because that was causing Discord messages to get too large.
  • Send anchor worker started/completed/failed Discord notifications since we're removing the separate runner image that sent those notifications.
  • Use base image when updating worker task def during CAS deployments.
  • Don't push CAS runner image to ECR since it's no longer needed.

@smrz2001 smrz2001 requested a review from 3benbox May 20, 2024 11:12
@smrz2001 smrz2001 self-assigned this May 20, 2024
Copy link

linear bot commented May 20, 2024

Copy link
Contributor

@3benbox 3benbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smrz2001 smrz2001 merged commit a611c49 into develop May 20, 2024
1 check passed
@smrz2001 smrz2001 deleted the feature/ws2-3214-replace-cas-worker-runner-script-with-cd-manager-reporting branch May 20, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants