Petr Stupka

Results 26 comments of Petr Stupka

Seems like there is already work in progress - https://github.com/hashicorp/terraform-provider-azuread/issues/547

Helper already available via https://github.com/hashicorp/go-azure-helpers/pull/115 Terraform 1.2.0 introduced support for backend authentication OIDC as well

@peombwa thank you for the support! The `Invoke-MgGraphRequest` is very useful in such cases! I'm writing a script using those new Entitlement Management cmdlets. I found plenty of similar 'bugs'...

Hi @magodo if you would like to format the INFO log messages, please let me know. Thank you

Hi @magodo, `pluginsdk.StateChangeConf` make sense. I wasn't aware about it. In one of my test, two endpoints took time to get available, so that looks good ![image](https://github.com/hashicorp/terraform-provider-azurerm/assets/18548253/771b9f14-d620-4813-a938-a795b50b8187)

> Hi @petr-stupka, thanks for pushing those changes. I have left a few more comments and I don't think that the `resourceStorageAccountRead` function is the correct place for doing these...

Hi @WodansSon > I am not really sure where it should go to be honest, but my gut it telling me it makes more sense to live in the Create...

Hi @WodansSon do you think you guys would be able to review it before new release tomorrow?

Hi @tombuildsstuff not sure if request review will send notification or i should mention you to inform you about the PR ready to review

Hi @WodansSon > ... is there a way you can isolate that functionality to just the `resourceStorageAccountCreate` function? In the `resourceStorageAccountCreate` function is the logic i mentioned in https://github.com/hashicorp/terraform-provider-azurerm/pull/23002#discussion_r1314298344 ```go...