Skip to content
This repository has been archived by the owner on Nov 13, 2024. It is now read-only.

Remove datasource azurerm_resource_group.vnet. #72

Merged
merged 2 commits into from
Oct 14, 2022

Conversation

lonegunmanb
Copy link
Member

Remove datasource azurerm_resource_group.vnet.
Make var.vnet_location required.
This change is a breaking change, and it will fix #62.

@lonegunmanb lonegunmanb temporarily deployed to acctests October 11, 2022 08:53 Inactive
@github-actions
Copy link
Contributor

MAIN BRANCH PUSH DETECTED DUE TO #71, THIS PR NEED TO BE UPDATED TO TRIGGER CI.

Make `var.vnet_location` required.
This change is a breaking change, and it will fix Azure#62.
@lonegunmanb lonegunmanb temporarily deployed to acctests October 11, 2022 13:07 Inactive
@lonegunmanb lonegunmanb temporarily deployed to acctests October 11, 2022 13:48 Inactive
Copy link

@a-z-i-z a-z-i-z left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @lonegunmanb. Looks good to go 🚀

Copy link
Collaborator

@jiaweitao001 jiaweitao001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks. 🚢

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

Successfully merging this pull request may close these issues.

Circular dependency on resource group
3 participants