Azure / AKS-Construction

Accelerate your onboarding to AKS with; Helper Web App, bicep templating and CI/CD samples. Flexible & secure AKS baseline implementations in a Microsoft + community maintained reference implementation.
https://azure.github.io/AKS-Construction/
MIT License
356 stars 165 forks source link

Failed to parse string as JSON with the default authorizedIPRanges #700

Closed vienleidl closed 1 month ago

vienleidl commented 3 months ago

Describe the bug The default authorizedIPRanges seems to be invalid, when running in PowerShell, the following error appears:

Failed to parse string as JSON: [
    x.x.x.x/32
]
Error detail: Expecting ',' delimiter: line 1 column 8 (char 7)

To Reproduce The command was run as below:

az deployment group create -g aks-resource-group  --template-uri https://github.com/Azure/AKS-Construction/releases/download/0.10.5/main.json --parameters `
  resourceName=aks-resource-group `
  managedNodeResourceGroup=aks-southeastasia `
  kubernetesVersion=1.29.4 `
  agentCount=1 `
  upgradeChannel=stable `
  AksPaidSkuForSLA=true `
  agentVMSize=Standard_B2als_v2 `
  nodePoolName=systempool1 `
  osSKU=AzureLinux `
  byoAKSSubnetId=/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Network/virtualNetworks/xxx/subnets/k8s-cluster `
  enable_aad=true `
  AksDisableLocalAccounts=true `
  enableAzureRBAC=true `
  adminPrincipalId=$(az ad signed-in-user show --query id --out tsv) `
  registries_sku=Standard `
  acrPushRolePrincipalId=$(az ad signed-in-user show --query id --out tsv) `
  enableTelemetry=false `
  networkPluginMode=Overlay `
  availabilityZones='["1","2","3"]' `
  authorizedIPRanges='["x.x.x.x/32"]' `
  blobCSIDriver=true `
  oidcIssuer=true `
  workloadIdentity=true

Expected behavior The authorize IP address should be added successfully.

Additional context Changing to authorizedIPRanges="['x.x.x.x/32']" as a workaround.

github-actions[bot] commented 2 months ago

Issue smells stale, no activity for 30 days. Stale Label will be removed if the issue is updated, otherwise closed in a month.