Log AL-Go Action duration to telemetry (#1765)

Log the AL-Go Action duration to telemetry in order to get a better
understanding of the performance of actions like RunPipeline over time
as well as give an estimate of what the total compute time is for all
AL-Go actions in a particular run.

---------

Co-authored-by: Copilot <[email protected]>

Author: aholstrup1

Actions/Invoke-AlGoAction.ps1

@@ -4,7 +4,9 @@ param(
     [Parameter(Mandatory = $true)]
     [scriptblock]$Action,
     [Parameter(Mandatory = $false)]
-    [switch]$SkipTelemetry
+    [switch]$SkipTelemetry,
+    [Parameter(Mandatory = $false)]
+    [System.Collections.Generic.Dictionary[[System.String], [System.String]]] $AdditionalData = @{}
 )
 
 $errorActionPreference = "Stop"
@@ -14,15 +16,18 @@ Set-StrictMode -Version 2.0
 Import-Module (Join-Path -Path $PSScriptRoot -ChildPath "TelemetryHelper.psm1" -Resolve)
 
 try {
+    $startTime = Get-Date
     Invoke-Command -ScriptBlock $Action
 
     if (-not $SkipTelemetry) {
-        Trace-Information -ActionName $ActionName
+        $AdditionalData["ActionDuration"] = (((Get-Date) - $startTime).TotalSeconds).ToString()
+        Trace-Information -ActionName $ActionName -AdditionalData $AdditionalData
     }
 }
 catch {
     if (-not $SkipTelemetry) {
-        Trace-Exception -ActionName $ActionName -ErrorRecord $_
+        $AdditionalData["ActionDuration"] = (((Get-Date) - $startTime).TotalSeconds).ToString()
+        Trace-Exception -ActionName $ActionName -ErrorRecord $_ -AdditionalData $AdditionalData
     }
 
     Write-Host "::ERROR::Unexpected error when running action. Error Message: $($_.Exception.Message.Replace("`r",'').Replace("`n",' ')), StackTrace: $($_.ScriptStackTrace.Replace("`r",'').Replace("`n",' <- '))";

RELEASENOTES.md

@@ -10,6 +10,7 @@ As stated in [AL-Go Deprecations](https://aka.ms/algodeprecations#cleanModePrepr
 
 - Issue 1697 Error in CheckForUpdates: "Internet Explorer engine is not available" when using self-hosted runners
 - Issue 1685 HttpError: Resource not accessible by integration
+- Issue 1757 Error when signing apps with key vault signing
 
 ### Workflow input validation
 
@@ -27,6 +28,10 @@ AL-Go for GitHub settings now has a schema. The following line is added at the b
 
 By setting failOn to 'newWarning', pull requests will fail if new warnings are introduced. This feature compares the warnings in the pull request build against those in the latest successful CI/CD build and fails if new warnings are detected.
 
+### AL-Go Telemetry
+
+Now AL-Go telemetry also logs `ActionDuration` which makes it possible to track the duration of the different steps in the AL-Go workflows (e.g. RunPipeline or Sign)
+
 ## v7.1
 
 ### Issues

Scenarios/EnablingTelemetry.md

@@ -42,7 +42,10 @@ Telemetry message: AL-Go action ran
 
 SeverityLevel: 1
 
-Additional Dimensions: None
+Additional Dimensions:
+| Dimension | Description |
+|-----------|-------------|
+| ActionDuration | The duration of the action |
 
 ### AL-Go action failed