Skip to content

Commit

Permalink
Merge pull request aztfmod#1754 from blinqas/fix-issue-with-vm-webhoo…
Browse files Browse the repository at this point in the history
…k-url-required-variarble

Fix webhook_url logic, variable should not be required
  • Loading branch information
arnaudlh authored Aug 16, 2023
2 parents cb07391 + bc51656 commit 4b64c72
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/compute/virtual_machine/shutdown_schedule.tf
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,6 @@ resource "azurerm_dev_test_global_vm_shutdown_schedule" "enabled" {
enabled = try(var.settings.shutdown_schedule.notification_settings.enabled, null)
time_in_minutes = try(var.settings.shutdown_schedule.notification_settings.time_in_minutes, null)
email = try(var.settings.shutdown_schedule.notification_settings.email, null)
webhook_url = try(var.settings.shutdown_schedule.notification_settings.enabled, false) ? var.settings.shutdown_schedule.notification_settings.webhook_url : try(var.settings.shutdown_schedule.notification_settings.webhook_url, null)
webhook_url = try(var.settings.shutdown_schedule.notification_settings.enabled, false) ? try(var.settings.shutdown_schedule.notification_settings.webhook_url, null) : null
}
}

0 comments on commit 4b64c72

Please sign in to comment.