From aae19b4723a9be6228860bcd340095615ab145dd Mon Sep 17 00:00:00 2001
From: Daniel Brennand <52419383+dbrennand@users.noreply.github.com>
Date: Mon, 11 Nov 2019 19:52:26 +0000
Subject: [PATCH] Sql 2016 alwayson changes (#448)

* Fixed parameter values to match defaults.

* Fixed typo in public IP addresses. Removed clustered option from SQL  server as it's not an option.

* Fixed avalibility sets. Two are deployed in the template. Not just one.
---
 sql-2016-alwayson/Readme.md                   | 6 +++---
 sql-2016-alwayson/azuredeploy.parameters.json | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/sql-2016-alwayson/Readme.md b/sql-2016-alwayson/Readme.md
index b1dc0c3..d5bf935 100644
--- a/sql-2016-alwayson/Readme.md
+++ b/sql-2016-alwayson/Readme.md
@@ -5,12 +5,12 @@ This template deploys two SQL Server Enterprise, Standard or Developer instances
 * A network security group
 * A virtual network
 * Four storage accounts (One for AD, One for SQL, One for File Share witness and One for VM diagnostic)
-* Four public IP address (One for AD, Two for each SQL VM and One for Public LB bound to SQL Always On Listener)
+* Four public IP address (One for AD, One for each SQL VM and One for Public LB bound to SQL Always On Listener)
 * One external load balancer for SQL VMs with Public IP bound to the SQL Always On listener
 * One VM (WS2016) configured as Domain Controller for a new forest with a single domain
-* Two VM (WS2016) configured as SQL Server 2016 SP1 or SP2 Enterprise/Standard/Developer and clustered (must use the marketplace images)
+* Two VM (WS2016) configured as SQL Server 2016 SP1 or SP2 Enterprise/Standard/Developer (must use the marketplace images)
 * One VM (WS2016) configured as File Share Witness for the cluster
-* One Availability Set containing the SQL and FSW 2016 VMs
+* Two Availability Sets, one containing the SQL and FSW 2016 VMs, the other containing the Domain Controller VM.
 
 ## Notes
 
diff --git a/sql-2016-alwayson/azuredeploy.parameters.json b/sql-2016-alwayson/azuredeploy.parameters.json
index 7b728d0..ed47260 100644
--- a/sql-2016-alwayson/azuredeploy.parameters.json
+++ b/sql-2016-alwayson/azuredeploy.parameters.json
@@ -3,13 +3,13 @@
     "contentVersion": "1.0.0.0",
     "parameters": {
         "domainName": {
-            "value": "contoso.local"
+            "value": "fabrikam.local"
         },
         "sqlServerServiceAccountUserName": {
             "value": "sqlservice"
         },
         "adminUsername": {
-            "value": "locladmin"
+            "value": "localadmin"
         }
     }
 }