Skip to content

Commit 88058dd

Browse files
Fix Terraform type mismatches, broken README link, and undefined variable
Agent-Logs-Url: https://github.com/localstack/localstack-azure-samples/sessions/406e9650-f8ef-48d0-875c-d862428b657f Co-authored-by: paolosalvatori <1658419+paolosalvatori@users.noreply.github.com>
1 parent 7102482 commit 88058dd

5 files changed

Lines changed: 6 additions & 6 deletions

File tree

samples/web-app-custom-image/python/README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ The solution is composed of the following Azure resources:
2828

2929
- [Azure Subscription](https://azure.microsoft.com/free/)
3030
- [Azure CLI](https://learn.microsoft.com/en-us/cli/azure/install-azure-cli)
31-
- [Docker](https://docs.docker.com/get-docker/)udio.com/items?itemName=ms-azuretools.vscode-bicep), if you plan to install the sample via Bicep.
31+
- [Docker](https://docs.docker.com/get-docker/)
32+
- [Bicep](https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-bicep), if you plan to install the sample via Bicep.
3233
- [Terraform](https://developer.hashicorp.com/terraform/downloads), if you plan to install the sample via Terraform.
3334
- [Azlocal CLI](https://azure.localstack.cloud/user-guides/sdks/az/): LocalStack Azure CLI wrapper
3435
- [jq](https://jqlang.org/): JSON processor for scripting and parsing command outputs

samples/web-app-custom-image/python/scripts/deploy.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,8 +168,7 @@ if [[ $? != 0 ]]; then
168168
az identity create \
169169
--name "$MANAGED_IDENTITY_NAME" \
170170
--resource-group "$RESOURCE_GROUP_NAME" \
171-
--location "$LOCATION" \
172-
--subscription "$SUBSCRIPTION_ID" 1>/dev/null
171+
--location "$LOCATION" 1>/dev/null
173172

174173
if [[ $? == 0 ]]; then
175174
echo "[$MANAGED_IDENTITY_NAME] user-assigned managed identity successfully created in the [$RESOURCE_GROUP_NAME] resource group"

samples/web-app-custom-image/python/terraform/modules/log_analytics/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ resource "azurerm_log_analytics_workspace" "example" {
44
resource_group_name = var.resource_group_name
55
sku = var.sku
66
tags = var.tags
7-
retention_in_days = var.retention_in_days != "" ? var.retention_in_days : null
7+
retention_in_days = var.retention_in_days != null ? var.retention_in_days : null
88

99
lifecycle {
1010
ignore_changes = [

samples/web-app-custom-image/python/terraform/modules/private_endpoint/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ resource "azurerm_private_endpoint" "example" {
99
name = "${var.name}Connection"
1010
private_connection_resource_id = var.private_connection_resource_id
1111
is_manual_connection = var.is_manual_connection
12-
subresource_names = try([var.subresource_name], null)
12+
subresource_names = var.subresource_name != null ? [var.subresource_name] : null
1313
request_message = try(var.request_message, null)
1414
}
1515

samples/web-app-custom-image/python/terraform/modules/private_endpoint/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ variable "subnet_id" {
2525

2626
variable "is_manual_connection" {
2727
description = "(Optional) Specifies whether the Azure Private Endpoint connection requires manual approval from the remote resource owner."
28-
type = string
28+
type = bool
2929
default = false
3030
}
3131

0 commit comments

Comments
 (0)