Took care of review comments

Author: praries880

src/Common/Commands.Common/MetricHelper.cs

@@ -226,9 +226,15 @@ private void LogExceptionEvent(AzurePSQoSEvent qos)
                 eventProperties.Add("Message", "Message removed due to PII.");
                 eventProperties.Add("StackTrace", qos.Exception.StackTrace);
                 eventProperties.Add("ExceptionType", qos.Exception.GetType().ToString());
-                if (qos.Exception.InnerException != null)
+                Exception innerEx = qos.Exception.InnerException;
+                int exceptionCount = 0;
+                //keep goin till we get to the last inner exception
+                while (innerEx != null)
                 {
-                    eventProperties.Add("InnerExceptionType", qos.Exception.InnerException.GetType().ToString());
+                    //Increment the inner exception count so that we can tell which is the outermmost
+                    //n which the innermost
+                    eventProperties.Add("InnerExceptionType-"+exceptionCount++, innerEx.GetType().ToString());
+                    innerEx = innerEx.InnerException;
                 }
                 client.TrackException(null, eventProperties, eventMetrics);
             }
@@ -271,10 +277,7 @@ private void PopulatePropertiesFromQos(AzurePSQoSEvent qos, IDictionary<string,
             eventProperties.Add("PowerShellVersion", PSVersion);
             eventProperties.Add("Version", AzurePowerShell.AssemblyVersion);
             eventProperties.Add("CommandParameterSetName", qos.ParameterSetName);
-            if (!string.IsNullOrWhiteSpace(qos.InvocationName))
-            {
-                eventProperties.Add("CommandInvocationName", qos.InvocationName);
-            }
+            eventProperties.Add("CommandInvocationName", qos.InvocationName);
 
             if (qos.InputFromPipeline != null)
             {

src/ResourceManager/Common/Commands.ResourceManager.Common/AzureRMCmdlet.cs

@@ -204,7 +204,8 @@ protected override void InitializeQosEvent()
                 ClientRequestId = this._clientRequestId,
                 SessionId = _sessionId,
                 IsSuccess = true,
-            };
+                ParameterSetName = this.ParameterSetName
+        };
 
             if (this.MyInvocation != null && !string.IsNullOrWhiteSpace(this.MyInvocation.InvocationName))
             {
@@ -219,11 +220,6 @@ protected override void InitializeQosEvent()
                         s => string.Format(CultureInfo.InvariantCulture, "-{0} ***", s)));
             }
 
-            if (!string.IsNullOrWhiteSpace(this.ParameterSetName))
-            {
-                _qosEvent.ParameterSetName = this.ParameterSetName;
-            }
-
             IAzureContext context;
             if (TryGetDefaultContext(out context) 
                 && context.Account != null 

src/ResourceManager/Profile/ChangeLog.md

@@ -18,7 +18,11 @@
         - Additional information about change #1
 -->
 ## Current Release
-
+* Added the following three values to the telemetry :
+  - Inner exception types
+  - ParameterSetName
+  - InvocationName
+  
 ## Version 5.1.0
 * Fix issue where running `Clear-AzureRmContext` would keep an empty context with the name of the previous default context, which prevented the user from creating a new context with the old name