Add WorkflowTypeName to completion metrics (#7756)

## What changed?
Added the `WorkflowTypeName` as a label on completion metrics. 

## Why?
Being able to track metrics (`operation="CompletionStats"`) for
completion and failures (`workflow_success`, `workflow_failed`, ...) per
`WorkflowTypeName`.

## How did you test it?
- [x] built
- [x] run locally and tested manually
- [ ] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)

## Potential risks
In cases where the `WorkflowTypeName` is automatically generated or in
general not suitable for usage as a label, it could lead to high
cardinality of the metrics.

Author: josesa

service/history/workflow/metrics.go

@@ -79,12 +79,14 @@ func emitWorkflowCompletionStats(
 	namespace namespace.Name,
 	namespaceState string,
 	taskQueue string,
+	workflowTypeName string,
 	status enumspb.WorkflowExecutionStatus,
 	config *configs.Config,
 ) {
 	handler := GetPerTaskQueueFamilyScope(metricsHandler, namespace, taskQueue, config,
 		metrics.OperationTag(metrics.WorkflowCompletionStatsScope),
 		metrics.NamespaceStateTag(namespaceState),
+		metrics.WorkflowTypeTag(workflowTypeName),
 	)
 
 	switch status {

service/history/workflow/transaction_impl.go

@@ -18,10 +18,11 @@ import (
 
 type (
 	completionMetric struct {
-		initialized    bool
-		taskQueue      string
-		namespaceState string
-		status         enumspb.WorkflowExecutionStatus
+		initialized      bool
+		taskQueue        string
+		namespaceState   string
+		workflowTypeName string
+		status           enumspb.WorkflowExecutionStatus
 	}
 	TransactionImpl struct {
 		shard  historyi.ShardContext
@@ -680,10 +681,11 @@ func snapshotToCompletionMetric(
 		return completionMetric{initialized: false}
 	}
 	return completionMetric{
-		initialized:    true,
-		taskQueue:      workflowSnapshot.ExecutionInfo.TaskQueue,
-		namespaceState: namespaceState,
-		status:         workflowSnapshot.ExecutionState.Status,
+		initialized:      true,
+		taskQueue:        workflowSnapshot.ExecutionInfo.TaskQueue,
+		namespaceState:   namespaceState,
+		workflowTypeName: workflowSnapshot.ExecutionInfo.WorkflowTypeName,
+		status:           workflowSnapshot.ExecutionState.Status,
 	}
 }
 
@@ -695,10 +697,11 @@ func mutationToCompletionMetric(
 		return completionMetric{initialized: false}
 	}
 	return completionMetric{
-		initialized:    true,
-		taskQueue:      workflowMutation.ExecutionInfo.TaskQueue,
-		namespaceState: namespaceState,
-		status:         workflowMutation.ExecutionState.Status,
+		initialized:      true,
+		taskQueue:        workflowMutation.ExecutionInfo.TaskQueue,
+		namespaceState:   namespaceState,
+		workflowTypeName: workflowMutation.ExecutionInfo.WorkflowTypeName,
+		status:           workflowMutation.ExecutionState.Status,
 	}
 }
 
@@ -719,6 +722,7 @@ func emitCompletionMetrics(
 			namespaceName,
 			completionMetric.namespaceState,
 			completionMetric.taskQueue,
+			completionMetric.workflowTypeName,
 			completionMetric.status,
 			shardContext.GetConfig(),
 		)