fixing microsoft/service-fabric-issues#1054 flatten inner exception (KeyVaultClient.GetSecretAsync) and fixing add Cert to linux vmss ClientStore bug

Author: Alfredo Santamaria Gomez

src/ResourceManager/ServiceFabric/Commands.ServiceFabric/Commands/ServiceFabricClusterCertificateCmdlet.cs

@@ -415,6 +415,14 @@ internal Task AddCertToVmssTask(VirtualMachineScaleSet vmss, CertificateInformat
         {
             var secretGroup = vmss.VirtualMachineProfile.OsProfile.Secrets.SingleOrDefault(
                 s => s.SourceVault.Id.Equals(certInformation.KeyVault.Id, StringComparison.OrdinalIgnoreCase));
+
+
+            string configStore = null;
+            if (vmss.VirtualMachineProfile.OsProfile.WindowsConfiguration != null)
+            {
+                configStore = Constants.DefaultCertificateStore;
+            }
+
             if (secretGroup == null)
             {
                 vmss.VirtualMachineProfile.OsProfile.Secrets.Add(
@@ -428,7 +436,7 @@ internal Task AddCertToVmssTask(VirtualMachineScaleSet vmss, CertificateInformat
                         {
                           new VaultCertificate()
                           {
-                          CertificateStore = Constants.DefaultCertificateStore,
+                          CertificateStore = configStore,
                           CertificateUrl = certInformation.SecretUrl
                           }
                         }
@@ -449,7 +457,7 @@ internal Task AddCertToVmssTask(VirtualMachineScaleSet vmss, CertificateInformat
                         secretGroup.VaultCertificates.Add(
                             new VaultCertificate()
                             {
-                                CertificateStore = Constants.DefaultCertificateStore,
+                                CertificateStore = configStore,
                                 CertificateUrl = certInformation.SecretUrl
                             });
                     }
@@ -460,7 +468,7 @@ internal Task AddCertToVmssTask(VirtualMachineScaleSet vmss, CertificateInformat
                     {
                         new VaultCertificate()
                         {
-                            CertificateStore = Constants.DefaultCertificateStore,
+                            CertificateStore = configStore,
                             CertificateUrl = certInformation.SecretUrl
                         }
                    };
@@ -586,7 +594,16 @@ private string GetThumbprintFromSecret(string secretUrl)
                 throw new PSArgumentException("secretUrl");
             }
 
-            var secretBundle = this.KeyVaultClient.GetSecretAsync(secretUrl).Result;
+            SecretBundle secretBundle;
+            try
+            {
+                secretBundle = this.KeyVaultClient.GetSecretAsync(secretUrl).Result;
+            }
+            catch (Exception ex)
+            {
+                throw GetInnerException(ex);
+            }
+
             var secretValue = secretBundle.Value;
             try
             {

src/ResourceManager/ServiceFabric/Commands.ServiceFabric/Commands/ServiceFabricClusterCmdlet.cs

@@ -93,18 +93,7 @@ protected PSCluster SendPutRequest(Cluster clusterResource, bool runOnSameThread
             catch (Exception e)
             {
                 PrintSdkExceptionDetail(e);
-
-                if (e.InnerException != null)
-                {
-                    while (e.InnerException != null)
-                    {
-                        e = e.InnerException;
-                    }
-
-                    throw;
-                }
-
-                throw;
+                throw GetInnerException(e);
             }
 
             return new PSCluster(cluster);
@@ -163,18 +152,7 @@ protected PSCluster SendPatchRequest(ClusterUpdateParameters request, bool runOn
             catch (Exception e)
             {
                 PrintSdkExceptionDetail(e);
-
-                if (e.InnerException != null)
-                {
-                    while (e.InnerException != null)
-                    {
-                        e = e.InnerException;
-                    }
-
-                    throw;
-                }
-
-                throw;
+                throw GetInnerException(e);
             }
 
             return new PSCluster(cluster);
@@ -193,17 +171,7 @@ protected Cluster GetCurrentCluster()
             }
             catch (Exception e)
             {
-                if (e.InnerException != null)
-                {
-                    while (e.InnerException != null)
-                    {
-                        e = e.InnerException;
-                    }
-
-                    throw;
-                }
-
-                throw;
+                throw GetInnerException(e);
             }
         }
 
@@ -349,5 +317,15 @@ protected void WriteClusterAndVmssVerboseWhenUpdate(List<Task> allTasks, bool pr
             exceptions.ForEach(PrintSdkExceptionDetail);
             task.Wait();
         }
+
+        protected Exception GetInnerException(Exception exception)
+        {
+            while (exception.InnerException != null)
+            {
+                exception = exception.InnerException;
+            }
+
+            return exception;
+        }
     }
 }