excluding workflow metadata query types from actions history (#6663)

## What changed?
We are excluding `__temporal_workflow_metadata` actions from metrics
emission.

## Why?
The pricing committee has decided to not charge for
[`__temporal_workflow_metadata`](https://www.notion.so/temporalio/Do-not-charge-for-built-in-metadata-query-10c8fc5677388033845ffb1aa504e4d4?pvs=4)
for billable actions.

## How did you test it?
unit tests

## Potential risks
Potential to expose a workaround for query types to be supplied at the
client level to bypass metrics for billing purposes.

Author: pauloh-temporal

common/rpc/interceptor/telemetry.go

@@ -80,12 +80,13 @@ var (
 	respondWorkflowTaskCompleted = "RespondWorkflowTaskCompleted"
 	pollActivityTaskQueue        = "PollActivityTaskQueue"
 	startWorkflowExecution       = "StartWorkflowExecution"
+	queryWorkflow                = "QueryWorkflow"
 
 	grpcActions = map[string]struct{}{
 		startWorkflowExecution:             {},
 		respondWorkflowTaskCompleted:       {},
 		pollActivityTaskQueue:              {},
-		"QueryWorkflow":                    {},
+		queryWorkflow:                      {},
 		"RecordActivityTaskHeartbeat":      {},
 		"RecordActivityTaskHeartbeatById":  {},
 		"ResetWorkflowExecution":           {},
@@ -258,7 +259,6 @@ func (ti *TelemetryInterceptor) emitActionMetric(
 		if resp.Started {
 			metrics.ActionCounter.With(metricsHandler).Record(1, metrics.ActionType("grpc_"+methodName))
 		}
-
 	case respondWorkflowTaskCompleted:
 		// handle commands
 		completedRequest, ok := req.(*workflowservice.RespondWorkflowTaskCompletedRequest)
@@ -321,7 +321,18 @@ func (ti *TelemetryInterceptor) emitActionMetric(
 		if activityPollResponse.Attempt > 1 {
 			metrics.ActionCounter.With(metricsHandler).Record(1, metrics.ActionType("activity_retry"))
 		}
-
+	case queryWorkflow:
+		queryWorkflowReq, ok := req.(*workflowservice.QueryWorkflowRequest)
+		if !ok {
+			return
+		}
+		queryType := queryWorkflowReq.GetQuery().GetQueryType()
+		switch queryType {
+		case "__temporal_workflow_metadata":
+			return
+		default:
+			metrics.ActionCounter.With(metricsHandler).Record(1, metrics.ActionType("grpc_"+methodName))
+		}
 	default:
 		// grpc action
 		metrics.ActionCounter.With(metricsHandler).Record(1, metrics.ActionType("grpc_"+methodName))

common/rpc/interceptor/telemetry_test.go

@@ -32,6 +32,7 @@ import (
 	commonpb "go.temporal.io/api/common/v1"
 	enumspb "go.temporal.io/api/enums/v1"
 	protocolpb "go.temporal.io/api/protocol/v1"
+	"go.temporal.io/api/query/v1"
 	"go.temporal.io/api/serviceerror"
 	"go.temporal.io/api/workflowservice/v1"
 	"go.temporal.io/server/api/historyservice/v1"
@@ -48,7 +49,6 @@ import (
 
 const (
 	startWorkflow = "StartWorkflowExecution"
-	queryWorkflow = "QueryWorkflow"
 )
 
 func TestEmitActionMetric(t *testing.T) {
@@ -78,11 +78,6 @@ func TestEmitActionMetric(t *testing.T) {
 			resp:              &workflowservice.StartWorkflowExecutionResponse{Started: true},
 			expectEmitMetrics: true,
 		},
-		{
-			methodName:        queryWorkflow,
-			fullName:          api.WorkflowServicePrefix + queryWorkflow,
-			expectEmitMetrics: true,
-		},
 		{
 			methodName: queryWorkflow,
 			fullName:   api.AdminServicePrefix + queryWorkflow,
@@ -144,6 +139,25 @@ func TestEmitActionMetric(t *testing.T) {
 				},
 			},
 		},
+		{
+			methodName: queryWorkflow,
+			fullName:   api.WorkflowServicePrefix + queryWorkflow,
+			req: &workflowservice.QueryWorkflowRequest{
+				Query: &query.WorkflowQuery{
+					QueryType: "some_type",
+				},
+			},
+			expectEmitMetrics: true,
+		},
+		{
+			methodName: queryWorkflow,
+			fullName:   api.WorkflowServicePrefix + queryWorkflow,
+			req: &workflowservice.QueryWorkflowRequest{
+				Query: &query.WorkflowQuery{
+					QueryType: "__temporal_workflow_metadata",
+				},
+			},
+		},
 	}
 
 	for _, tt := range testCases {