Skip to content

[DO NOT MERGE] [TSPS-899] Update Glimpse docker images#1823

Open
salonishah11 wants to merge 17 commits intodevelopfrom
sps_glimpse_docker_images
Open

[DO NOT MERGE] [TSPS-899] Update Glimpse docker images#1823
salonishah11 wants to merge 17 commits intodevelopfrom
sps_glimpse_docker_images

Conversation

@salonishah11
Copy link
Copy Markdown
Contributor

@salonishah11 salonishah11 commented Apr 22, 2026

Description

Jira: https://broadworkbench.atlassian.net/browse/TSPS-899

This PR updates the docker images for GlimpsePhase, GlimpseLigate and CollectQCMetrics tasks.

  • GlimpsePhase and GlimpseLigate will now use a docker image generated from warp-tools repo (using build script from palantir-workflows repo that builds Glimpse base image and adds necessary tools on top of it). Related PR - [TSPS-899] Add Glimpse docker build GHA and versioning warp-tools#193
  • CollectQCMetrics will now use a mirro.gcr.io version of Hail image

There are also some other small wdl changes that were made after we ported the Glimpse wdl to Warp. These are either just small bash related changes or necessary changes to make the wdl work with the new Glimpse docker image.

Testing

Tested the image by updating wdl and running the workflow manually. For GlimpsePhase task CPU was set to 1 and preemptibles were removed to test the outputs are same.

Submission with the new docker image - https://app.terra.bio/#workspaces/allofus-drc-imputation/Imputation_Server_Hydrogen_Dev/submission_history/8b8e0f87-1ddf-448c-a8a1-5c3d5b1cca3a
Submission with the existing docker image - https://app.terra.bio/#workspaces/allofus-drc-imputation/Imputation_Server_Hydrogen_Dev/submission_history/2dc2e352-0d01-4d25-ac22-0110ba7db94c

Verified the contents of bcf outputs for GlimpsePhase task for 1 shard for both submissions were same using these commands

bcftools view -H file1.bcf > file1.body
bcftools view -H file2.bcf > file2.body
diff file1.body file2.body           --> returned nothing

bcftools view file1.bcf -Ov -o file1.vcf
bcftools view file2.bcf -Ov -o file2.vcf
diff <(grep -v "^#" file1.vcf) <(grep -v "^#" file2.vcf)        --> returned nothing

Checklist

If you can answer "yes" to the following items, please add a checkmark next to the appropriate checklist item(s) and notify our WARP team by tagging @broadinstitute/warp-admins in a comment on this PR.

  • Did you add inputs, outputs, or tasks to a workflow?
  • Did you modify, delete or move: file paths, file names, input names, output names, or task names?
  • If you made a changelog update, did you update the pipeline version number?

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.


String gatk_docker = "us.gcr.io/broad-gatk/gatk:4.6.0.0"
String glimpse_docker = "us.gcr.io/broad-dsde-methods/glimpse:kachulis_ck_bam_reader_retry_cf5822c"
String glimpse_docker = "us.gcr.io/broad-gotc-prod/imputation-glimpse:sps_glimpse_docker"
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.

I will update this once warp-tools PR is merged and replace it with the new official image

set -euo pipefail

export GCS_OAUTH_TOKEN=$(/root/google-cloud-sdk/bin/gcloud auth application-default print-access-token)
export GCS_OAUTH_TOKEN=$(/google-cloud-sdk/bin/gcloud auth application-default print-access-token)
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 change is needed to make the wdl work with the new Glimpse docker image

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

Comment on lines -555 to -556
cpu: cpu
preemptible: preemptible
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 will be reverted after testing

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants