Integrate with opt in/out cmdlets

Author: xindzhan

src/Common/Commands.Common/AzurePSCmdlet.cs

@@ -219,6 +219,22 @@ protected static AzurePSDataCollectionProfile GetDataCollectionProfile()
             return _dataCollectionProfile;
         }
 
+        /// <summary>
+        /// Check whether the data collection is opted in from user
+        /// </summary>
+        /// <returns>true if allowed</returns>
+        public static bool IsDataCollectionAllowed()
+        {
+            if (_dataCollectionProfile != null &&
+                _dataCollectionProfile.EnableAzureDataCollection.HasValue &&
+                _dataCollectionProfile.EnableAzureDataCollection.Value)
+            {
+                return true;
+            }
+
+            return false;
+        }
+
         /// <summary>
         /// Save the current data collection profile Json data into the default file path
         /// </summary>
@@ -551,24 +567,29 @@ protected void LogQosEvent(bool waitForMetricSending = false)
             }
 
             QosEvent.FinishQosEvent();
-            //TODO change to debug
-            WriteVerbose(QosEvent.ToString());
 
             if (!IsUsageMetricEnabled && (!IsErrorMetricEnabled || QosEvent.IsSuccess))
             {
                 return;
             }
 
+            if (!IsDataCollectionAllowed())
+            {
+                return;
+            }
+
+            WriteDebug(QosEvent.ToString());
+
             try
             {
                 MetricHelper.LogQoSEvent(QosEvent, IsUsageMetricEnabled, IsErrorMetricEnabled);
                 MetricHelper.FlushMetric(waitForMetricSending);
-                WriteVerbose("Finish sending metric.");
+                WriteDebug("Finish sending metric.");
             }
             catch (Exception e)
             {
                 //Swallow error from Application Insights event collection.
-                WriteErrorWithTimestamp(e.ToString());
+                WriteWarning(e.ToString());
             }
         }
 

src/Common/Commands.Common/MetricHelper.cs

@@ -20,19 +20,14 @@ public static class MetricHelper
         static MetricHelper()
         {            
             TelemetryClient = new TelemetryClient();
-            //InstrumentationKey shall be injected in build server
-            TelemetryClient.InstrumentationKey = "ce08abab-065a-4af8-a997-fdd4cd5481b4";
-            //TelemetryClient.Context.Location.Ip = "0.0.0.0";
-
-            if (!IsMetricTermAccepted())
-            {
-                TelemetryConfiguration.Active.DisableTelemetry = true;
-            }
+            // TODO: InstrumentationKey shall be injected in build server
+            TelemetryClient.InstrumentationKey = "7df6ff70-8353-4672-80d6-568517fed090";
+            // Disable IP collection
+            TelemetryClient.Context.Location.Ip = "0.0.0.0";
 
             if (TestMockSupport.RunningMocked)
             {
-                //TODO enable in final cr
-                //TelemetryConfiguration.Active.DisableTelemetry = true;
+                TelemetryConfiguration.Active.DisableTelemetry = true;
             }
         }
 
@@ -84,8 +79,7 @@ private static void LogExceptionEvent(AzurePSQoSEvent qos)
 
         public static bool IsMetricTermAccepted()
         {
-            //TODO check the config/preference
-            return true;
+            return AzurePSCmdlet.IsDataCollectionAllowed();
         }
 
         public static void FlushMetric(bool waitForMetricSending)