-
Notifications
You must be signed in to change notification settings - Fork 596
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
VS_492 - Beta User Jar release #7934
Conversation
Successful joint genoytping run here: https://app.terra.bio/#workspaces/gvs-dev/GVS%20Quickstart%20v3%20ggrant/job_history/90b934d7-27dc-4745-9230-e89d4bd77e3e |
Codecov Report
@@ Coverage Diff @@
## ah_var_store #7934 +/- ##
================================================
Coverage ? 86.241%
Complexity ? 35198
================================================
Files ? 2173
Lines ? 165003
Branches ? 17793
================================================
Hits ? 142300
Misses ? 16377
Partials ? 6326 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you copy the new jar to gs://gvs_quickstart_storage/jars/ and reference that location in the WDLs? That way a user doesn't need access to the broad-dsp-spec-ops
bucket (which is not public).
Thanks - I've made the change. |
@@ -214,7 +214,7 @@ task CreateCostObservabilityTable { | |||
fi | |||
>>> | |||
runtime { | |||
docker: "us.gcr.io/broad-dsde-methods/variantstore:rsa_metadata_from_python_20220628" | |||
docker: "us.gcr.io/broad-dsde-methods/variantstore:ah_var_store_2022_07_08" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we sure our users will have access to this image? and/or is it public?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
poked around the methods GCR / GCS bucket and it and all images within it appears to be public
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
us.gcr.io/broad-dsde-methods/variantstore is publicly accessible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
Have updated the gatk override jar with a fresh build (ah_var_store on 7/8/2022)
And updated the docker similarly.