Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix data.azurerm_client_config.current.object_id return "" empty when upgrading az cli to 2.37.0 #403

Closed

Conversation

backwind1233
Copy link
Contributor

@backwind1233 backwind1233 commented Jun 28, 2022

Context

When running samples with terraform, the same issue happens as the hashicorp/terraform-provider-azurerm#16982 described.

Solution
Upgrade version of hashicorp/azurerm to 3.9.0.

Checklist:

  • Test all samples with terraform.

@backwind1233 backwind1233 changed the base branch from spring-cloud-azure_v4.2.0 to main June 28, 2022 13:54
@saragluna
Copy link
Contributor

Do we test them one by one? I remember having issues running one of the Event Hubs samples, but I don't know which one it is.

@backwind1233
Copy link
Contributor Author

Do we test them one by one? I remember having issues running one of the Event Hubs samples, but I don't know which one it is.

I will test them one by one.

@backwind1233
Copy link
Contributor Author

close this pr, there are breaking changes for hashicorp/azurerm from 2.x.x to 3.x.x, which is not compatible with our original code.

@backwind1233
Copy link
Contributor Author

backwind1233 commented Jul 11, 2022

Here is the result of running terraform scripts with different hashicorp/azurerm versions:

Module Version
Az Cli version 2.38.0
terraform Version v1.2.4 on darwin_arm64
hashicorp/azurerm 3.9.0

CleanShot 2022-07-11 at 13 34 26@2x


Module Version
Az Cli version 2.38.0
terraform Version v1.2.4 on darwin_arm64
hashicorp/azurerm 2.99.0

CleanShot 2022-07-11 at 13 36 28@2x

There are many failures here, we need to fix this issue.

@backwind1233
Copy link
Contributor Author

Link this pr with #430.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants