Skip to content

Conversation

@salonishah11
Copy link

@salonishah11 salonishah11 commented Nov 17, 2025

Description

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

This PR updates the docker image for Phase and Impute tasks in ImputationBeagle wdl. The docker image contains Beagle jar that is generated from code that lives in GitHub repo tmp-sharing/imp-server. The reason for this is to have the code for Beagle jar in a version controlled place.

GHA run using Scientific type testing with this new docker image: https://github.com/broadinstitute/warp/actions/runs/19650001245

warp-tools PR: broadinstitute/warp-tools#176


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

Remember to squash merge!

@github-actions
Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
ImputationBeagle.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

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
ImputationBeagle.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

Remember to squash merge!

@github-actions
Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
ImputationBeagle.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

Remember to squash merge!

@github-actions
Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
ImputationBeagle.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

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
ImputationBeagle.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

Remember to squash merge!

@github-actions
Copy link

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
ImputationBeagle.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

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍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

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@salonishah11 salonishah11 changed the title [DO NOT MERGE] [TSPS-682] Update beagle docker image [TSPS-682] Update beagle docker image Nov 25, 2025
@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍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

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

IlluminaGenotypingArray 1.12.26 2025-10-09
Imputation 1.1.23 2025-10-03
ImputationBeagle 2.3.0 2025-11-12
ImputationBeagle 2.4.0 2025-11-25
Copy link
Author

Choose a reason for hiding this comment

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

I thought changing the Docker image and how Beagle jar is generated in docker image warrants for a minor version change but happy to discuss if this isn't right.

Copy link
Contributor

Choose a reason for hiding this comment

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

i think this is fine, patch would have been fine too since there is no "actual" changes to the output but your reasoning is very valid.

Copy link
Member

Choose a reason for hiding this comment

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

yeah I like the minor version change here

@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍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

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
ImputationBeagle.wdl has not been changed and needs updating
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍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

🔍Changelog Validation Results:

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

@salonishah11 salonishah11 requested review from a team, jsotobroad and mmorgantaylor November 25, 2025 15:39
Copy link
Contributor

@jsotobroad jsotobroad left a comment

Choose a reason for hiding this comment

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

looks good, can you link the warp-tools PR in the description of this doc so we can reference it if need be.

java -ea -XX:MaxRAMPercentage=90.0 -XX:MinRAMPercentage=90.0 -XX:-UseCompressedOops \
-jar /usr/gitc/beagle.17Dec24.224.jar \
-jar /usr/gitc/beagle.d820c4e.jar \
Copy link
Contributor

Choose a reason for hiding this comment

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

oh man this si gonna be so much better moving forward with provenance

IlluminaGenotypingArray 1.12.26 2025-10-09
Imputation 1.1.23 2025-10-03
ImputationBeagle 2.3.0 2025-11-12
ImputationBeagle 2.4.0 2025-11-25
Copy link
Contributor

Choose a reason for hiding this comment

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

i think this is fine, patch would have been fine too since there is no "actual" changes to the output but your reasoning is very valid.

IlluminaGenotypingArray 1.12.26 2025-10-09
Imputation 1.1.23 2025-10-03
ImputationBeagle 2.3.0 2025-11-12
ImputationBeagle 2.4.0 2025-11-25
Copy link
Member

Choose a reason for hiding this comment

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

yeah I like the minor version change here

2025-11-25 (Date of Last Commit)

* Update Phase and Impute tasks to use a docker image that contains Beagle JAR
generated from GitHub repo [tmp-sharing/imp-server](https://github.com/tmp-sharing/imp-server/tree/master).
Copy link
Member

Choose a reason for hiding this comment

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

man I wish this repo wasn't called tmp-sharing 😬

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.

5 participants