nextstrain / rsv

Workflow for RSV analyses on Nextstrain.org
https://nextstrain.org/rsv
6 stars 3 forks source link

Job AWS console link broken #53

Closed jameshadfield closed 8 months ago

jameshadfield commented 8 months ago

Current Behavior

The URL behind the "AWS Batch Console" link in the run-build summary (e.g.) is broken as it contains region=***. This is probably a misconfiguration in how we're using the pathogen-repo-build workflow.

I can fix this next week, just noting it here.

tsibley commented 8 months ago

That looks like AWS_REGION is being passed as a secret, so its value is being masked in the logs by GitHub Actions.

tsibley commented 8 months ago

(This is a reason not to use secrets for non-secret things if you can help it.)

joverlee521 commented 8 months ago

Ah, I forgot to remove the AWS_DEFAULT_REGION secret after I removed references to it in the GH Action workflows.

I've deleted the secret from this repo. If we need to set the value for any reason in the future, we should use the non-secret GH Action variables.

jameshadfield commented 8 months ago

Thanks for fixing Jover. For anyone else who wants to understand why this happened: