DataBiosphere / terra-docker

BSD 3-Clause "New" or "Revised" License
27 stars 34 forks source link

Update terra-jupyter-aou to Hail 0.2.130 #482

Closed daniel-goldstein closed 7 months ago

yonghaoy commented 7 months ago

@LizBaldo @jdcanas GHA failed when getting GCP credential. Can you help to check. Also I do not have permission to force merge a PR, would need your help if this change LG

LizBaldo commented 7 months ago

The secret might have been moved, I will need to ask Devops and will get back to you shortly with the updated secret

yonghaoy commented 7 months ago

The secret might have been moved, I will need to ask Devops and will get back to you shortly with the updated secret

Thanks. btw, secret might be managed here: https://github.com/broadinstitute/terraform-ap-deployments/blob/master/github/tfvars/databiosphere-terra-docker.tfvars

yonghaoy commented 7 months ago

@LizBaldo this GHA is still failing

LizBaldo commented 7 months ago

@yonghaoy you need to update the actions as suggested by Devops

LizBaldo commented 7 months ago

You need actions/checkout@v4 and google-github-actions/auth@v2

yonghaoy commented 7 months ago

Turns out github secret does not work with folk repo. Close this and will use https://github.com/DataBiosphere/terra-docker/pull/485 instead