[ServiceFabric] Fix Add-AzServiceFabricNodeType. Add node type Before…

… creating vmss (#13648) * Add node type before vmss * fix typo Co-authored-by: Alfredo Santamaria Gomez <[email protected]>
Azure · Dec 4, 2020 · 033e0eb · 033e0eb
1 parent 9e339f9
commit 033e0eb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/ServiceFabric/ServiceFabric/ChangeLog.md b/src/ServiceFabric/ServiceFabric/ChangeLog.md
@@ -22,6 +22,7 @@
 * Add example to Set-AzServiceFabricSetting with SettingsSectionDescription param
 * Update application related cmdlets to call out that support is only for ARM deployed resources
 * Mark for deprecation cluster cert cmdlets Add-AzureRmServiceFabricClusterCertificate and Remove-AzureRmServiceFabricClusterCertificate
+* Fix Add-AzServiceFabricNodeType. Add node type to service fabric cluster before creating virtual machine scale set.
 
 ## Version 2.2.0
 

diff --git a/src/ServiceFabric/ServiceFabric/Commands/AddAzureRmServiceFabricNodeType.cs b/src/ServiceFabric/ServiceFabric/Commands/AddAzureRmServiceFabricNodeType.cs
@@ -124,8 +124,8 @@ public override void ExecuteCmdlet()
             {
                 var cluster = GetCurrentCluster();
                 this.diagnosticsStorageName = cluster.DiagnosticsStorageAccountConfig.StorageAccountName;
-                CreateVmss(cluster.ClusterId);
                 var pscluster = AddNodeTypeToSfrp(cluster);
+                CreateVmss(cluster.ClusterId);
                 WriteObject(pscluster, true);
             }
         }