-
Notifications
You must be signed in to change notification settings - Fork 1
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:change in count condition #13
Conversation
@@ -65,13 +65,13 @@ module "load-balancer" { | |||
|
|||
# Load Balancer | |||
frontend_name = "mypublicIP" | |||
frontend_private_ip_address_allocation = "Static" | |||
frontend_private_ip_address_allocation = "static" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we are changing this?
frontend_private_ip_address = "10.0.1.6" | ||
lb_sku = "Standard" | ||
|
||
# Public IP | ||
ip_count = 1 | ||
allocation_method = "Static" | ||
allocation_method = "static" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we are changing this?
} | ||
|
||
output "azurerm_public_ip_id" { | ||
description = "the id for the azurerm_lb_public_ip resource" | ||
value = azurerm_public_ip.default[*].id | ||
value = azurerm_public_ip.default[0].id |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why we are not using try in this? I think it will give error when we set enable
variable false.
what
• Test each module when the enable variable is true.
• Deploy the module, then set the enable variable to false, ensuring all resources are destroyed without errors.
why