magodo
magodo
> @magodo For clarity, where you mention `private dns zone`, this is the `Azure DNS zone endpoints` preview feature: https://learn.microsoft.com/en-us/azure/storage/common/storage-account-overview#azure-dns-zone-endpoints-preview? You are right, sorry for the wrong wording... I've updated...
@jaredfholgate I agree that *nclusion or exclusion of certain parameters to determine* different behavior is not a good choice! I'll update the code and flow to be as what it...
A new property is introduced: `lookup_blob_endpoint`. When this is set to `true`, a GET will be called to the SA to get the exact blob endpoint. Otherwise, the endpoint is...
@mbfrahry I've added the test and passed it: ```shell $ TF_ACC=1 go test -v -timeout=20h -parallel=20 -run='TestAccBackendAzureADAuthBasicWithBlobEndpointLookup' === RUN TestAccBackendAzureADAuthBasicWithBlobEndpointLookup === PAUSE TestAccBackendAzureADAuthBasicWithBlobEndpointLookup === CONT TestAccBackendAzureADAuthBasicWithBlobEndpointLookup ... --- PASS: TestAccBackendAzureADAuthBasicWithBlobEndpointLookup...
@mbfrahry Thank you for the review! I've merged the `main` branch now. The CI failure of Changelog is saying: > Error: Currently this PR would target a v1.12 release. Please...
I think it is related to https://github.com/hashicorp/terraform-provider-azurerm/pull/24024, but I can't reproduce it locally. Could you please try to run the following snippet and provides the state and the log file:...
I've just created a tool to display the progress (and more): https://github.com/magodo/pipeform
@jaredfholgate Assuming you will get error: `No valid credentials found` in this case. As a workaround, you can just provide a dummy PAT token. While https://github.com/microsoft/terraform-provider-azuredevops/pull/1402 will solve your problem.
Sorry * for the breaking change.. I've made a fix in #36623, and tested it as much as possible to ensure all the cases are covered (see the PR description)....
@Klaas- There is conflict, once resolved I'm happy to merge this.