Skip to content

CTM-575 Clean up AOU images - #522

Merged
aednichols merged 11 commits into
masterfrom
aen_ctm_575_aou_cleanup
Aug 25, 2026
Merged

CTM-575 Clean up AOU images#522
aednichols merged 11 commits into
masterfrom
aen_ctm_575_aou_cleanup

Conversation

@aednichols

@aednichols aednichols commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Simplify the build ahead of upcoming work to update Python, Ubuntu, etc.

  • Remove definitions for terra-rstudio-aou and terra-jupyter-aou
  • Also remove wondershaper, seemingly used to slow egress by throttling connections to 16 Mbps
  • Fix incorrect bash array syntax in build_all.sh
    • Would have produced terra-base,
  • Adopt docker buildx and upgrade runner hardware, making builds... considerably faster

Supporting PRs:


terra-jupyter-hail build before & after:

Screenshot 2026-08-16 at 10-55-37 CTM-575 Clean up AOU images · DataBiosphere_terra-docker@9cc7641 Screenshot 2026-08-16 at 10-55-26 CTM-575 Clean up AOU images · DataBiosphere_terra-docker@45079aa

@aednichols
aednichols requested a review from lucymcnatt August 12, 2026 21:51

@LizBaldo LizBaldo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is wonderful to see!


tf.io.gfile.copy(src='gs://genomics-public-data/1000-genomes/other/sample_info/sample_info.csv',
dst='/tmp/genomics-public-data-1000-genomes-sample_info.csv',
tf.io.gfile.copy(src='gs://gcp-public-data--broad-references/refdisk_manifest.json',

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This file inexplicably went missing, evidently the test had not been run in a while. Seems like any file will do, found a replacement.

@jgainerdewar jgainerdewar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice!!

type: choice
options:
- terra-base
- terra-jupyter-aou

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👋

@aednichols
aednichols merged commit ad35423 into master Aug 25, 2026
8 checks passed
@aednichols
aednichols deleted the aen_ctm_575_aou_cleanup branch August 25, 2026 15:52
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