Fix a stupid bug with parameters to deploy.

auslavs · Apr 15, 2020 · 5a888a1 · 5a888a1
1 parent c4dec46
commit 5a888a1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -1,5 +1,8 @@
 Release Notes
 =============
+## 0.11.1
+* Fix a bug with deploy parameterisation.
+
 ## 0.11.0
 * Remove REST API support.
 * Enhance Azure CLI support.

diff --git a/src/Farmer/Deploy.fs b/src/Farmer/Deploy.fs
@@ -81,12 +81,11 @@ module Az =
         executeAz (sprintf "group create -l %s -n %s" location resourceGroup) |> Result.ignore
     /// Deploys an ARM template to an existing resource group.
     let deploy resourceGroup deploymentName templateFilename parameters =
-        sprintf "group deployment create -g %s -n %s --template-file %s --parameters %s"
-            resourceGroup
-            deploymentName
-            templateFilename
-            (parameters |> Seq.map(fun (a,b) -> sprintf "%s=%s" a b) |> String.concat " ")
-        |> executeAz
+        let parametersArgument =
+            match parameters with
+            | [] -> ""
+            | parameters -> sprintf "--parameters %s" (parameters |> List.map(fun (a,b) -> sprintf "%s=%s" a b) |> String.concat " ")
+        executeAz (sprintf "group deployment create -g %s -n %s --template-file %s %s" resourceGroup deploymentName templateFilename parametersArgument)
     /// Deploys a zip file to a web app using the Zip Deploy mechanism.
     let zipDeploy webAppName (zipDeployKind:ZipDeployKind) resourceGroup =
         let packageFilename = zipDeployKind.GetZipPath deployFolder

diff --git a/src/Farmer/Farmer.fsproj b/src/Farmer/Farmer.fsproj
@@ -2,7 +2,7 @@
   <PropertyGroup>
     <!-- General -->
     <AssemblyName>Farmer</AssemblyName>
-    <Version>0.11.0</Version>
+    <Version>0.11.1</Version>
     <Description>A DSL for rapidly generating non-complex ARM templates.</Description>
     <Copyright>Copyright 2019, 2020 Compositional IT Ltd.</Copyright>
     <Company>Compositional IT</Company>