Add host attribute for endpoints of blob, table, queue, and file.

[metacpp]

This PR is to export host attributes for storage account resource, covers primary/secondary endpoints for table, file, queue, and blob.

Fixes #2785

[JunyiYi]

Thanks @metacpp for the contributions. I have taken a look throughout the code change and left the comment. It will be good to go if we could fix them.

[katbyte]

Hi @metacpp,

I've left some comments inline

[metacpp]

@katbyte @JunyiYi Just updated the PR to address all the issues in the PR.

[JunyiYi]

Thanks for the quick update, there are some additional comments left, please take a look.

[metacpp]

Thanks for the quick update, there are some additional comments left, please take a look.

@JunyiYi I've address your comments in the PR.

[katbyte]

Hi @metacpp,

Thanks for the PR, I've left some comments that need to be addressed before mege. AS well as i do question to reasoning behind pulling out the d.sets into a separate function here. I don't think much is gained by it.

Comments have been resolved.

[metacpp]

@katbyte Thanks for providing these useful comments, I've updated the PR.

[WodansSon]

@metacpp LGTM

dismissing since changes have been pushed

[ghost]

This has been released in version 1.23.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
	version = "~> 1.23.0"
}
# ... other configuration ...

[ghost]

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!