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

data azurerm_linux_web_app is not handling virtual_network_subnet_id correctly #20490

Closed
1 task done
algompluecker opened this issue Feb 15, 2023 · 3 comments · Fixed by #20577
Closed
1 task done

data azurerm_linux_web_app is not handling virtual_network_subnet_id correctly #20490

algompluecker opened this issue Feb 15, 2023 · 3 comments · Fixed by #20577

Comments

@algompluecker
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version

1.3.7

AzureRM Provider Version

3.43.0

Affected Resource(s)/Data Source(s)

data azurerm_linux_web_app, data azurerm_windows_web_app

Terraform Configuration Files

data "azurerm_linux_web_app" "example" {
  name                = "existing"
  resource_group_name = "existing"
}

output "id" {
  value = data.azurerm_linux_web_app.example.id
}

Debug Output/Panic Output

Unable to provide a log, but I found the issue in the code (see Steps to Reproduce).

Expected Behaviour

The property virtual_network_subnet_id should contain the subnet id.

Actual Behaviour

The property virtual_network_subnet_id is an empty string.

Steps to Reproduce

terraform plan

Looking at linux_web_app_data_source.go and windows_web_app_data_source.go the handling for VirtualNetworkSubnetID is missing.

It requires similar handling as in linux_function_app_data_source.go

		state.VirtualNetworkSubnetID = utils.NormalizeNilableString(functionApp.VirtualNetworkSubnetID)

Important Factoids

No response

References

No response

@github-actions github-actions bot removed the bug label Feb 15, 2023
@xiaxyi
Copy link
Contributor

xiaxyi commented Feb 17, 2023

Thanks @algompluecker for raising this issue, i'll check the data source code and update later.

@algompluecker
Copy link
Author

Thanks a lot @xiaxyi and @stephybun!

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants