Skip to content

batch: Add new job queue and compute environment with larger instance#74

Merged
victorlin merged 2 commits into
mainfrom
victorlin/update-batch
Jun 24, 2026
Merged

batch: Add new job queue and compute environment with larger instance#74
victorlin merged 2 commits into
mainfrom
victorlin/update-batch

Conversation

@victorlin

@victorlin victorlin commented Jun 24, 2026

Copy link
Copy Markdown
Member

Description of proposed changes

Motivation is to allow GISAID ingest to run with more memory (it failed with c7a.8xlarge).

Use with --aws-batch-queue nextstrain-job-queue-c7a-12xlarge.

Plan: 2 to add, 0 to change, 0 to destroy.

Related issue(s)

nextstrain/ncov-ingest#532

Checklist

  • Checks pass

Put each instance type on its own line for readability, and add a link
to the pricing page for reference.

@jameshadfield jameshadfield left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Aside: thanks for the super readable commits - makes it really easy to interpret what's changing

Comment thread env/production/aws-batch.tf
@victorlin victorlin force-pushed the victorlin/update-batch branch from 219e58f to 29f7ca3 Compare June 24, 2026 22:42
@victorlin victorlin changed the title batch: Use new compute environment with larger instance batch: Add new job queue and compute environment with larger instance Jun 24, 2026
Motivation is to allow GISAID ingest to run with more memory (it failed
with c7a.8xlarge).

Use with --aws-batch-queue nextstrain-job-queue-c7a-12xlarge.

Plan: 2 to add, 0 to change, 0 to destroy.
@victorlin victorlin force-pushed the victorlin/update-batch branch from 29f7ca3 to 1633962 Compare June 24, 2026 22:43
@victorlin victorlin merged commit ff97155 into main Jun 24, 2026
1 check passed
@victorlin victorlin deleted the victorlin/update-batch branch June 24, 2026 23:19
Comment on lines +150 to +151
resource "aws_batch_compute_environment" "c7a_12xlarge" {
name = "c7a-instances-2026-05-24"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oops, there was a copy-paste error here. Wasn't caught by plan, only during apply. Fixed in 084134a

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.

3 participants