[batch] Use default credentials for the Azure SAS token test #13981
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A very small PR but here's the background and context behind this change. When talking to either GCP or Azure, hail chooses credentials in the following order from highest priority to lowest priority:
credential_file
argument passed to the relevant credentials classGOOGLE_APPLICATION_CREDENTIALS
orAZURE_APPLICATION_CREDENTIALS
) (from this you can see why the code that was here is totally redundant)gcloud
oraz
credentials, or the metadata server if you're on a cloud VM.I'm trying to rid the codebase of most explicit providing of credentials file paths, for two reasons:
gcloud
andaz
. I shouldn't need to download some file and provideAZURE_APPLICATION_CREDENTIALS
to run this test. It should just use the latent credentials.