Files
tasks/tests/describe_task_queue.go
ShivamandLina Jodoin d224c054fb Removed non-required helpers (#6465)
## What changed?
<!-- Describe what has changed in this PR -->
- removed helpers which were not being used

## Why?
<!-- Tell your future self why have you made these changes -->
- keeping code nice and clean

## How did you test it?
<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->
- exiting CI/CD run

## Potential risks
<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->
none

## Is hotfix candidate?
<!-- Is this PR a hotfix candidate or does it require a notification to
be sent to the broader community? (Yes/No) -->
- nope
2024-08-30 12:55:49 -07:00

281 lines
12 KiB
Go

// The MIT License
//
// Copyright (c) 2024 Temporal Technologies Inc. All rights reserved.
//
// Permission is hereby granted, free of charge, to any person obtaining a copy
// of this software and associated documentation files (the "Software"), to deal
// in the Software without restriction, including without limitation the rights
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
// copies of the Software, and to permit persons to whom the Software is
// furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included in
// all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
// THE SOFTWARE.
package tests
import (
"context"
"time"
"github.com/pborman/uuid"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
commandpb "go.temporal.io/api/command/v1"
commonpb "go.temporal.io/api/common/v1"
enumspb "go.temporal.io/api/enums/v1"
taskqueuepb "go.temporal.io/api/taskqueue/v1"
"go.temporal.io/api/workflowservice/v1"
"google.golang.org/protobuf/types/known/durationpb"
"go.temporal.io/server/common/dynamicconfig"
)
type (
DescribeTaskQueueSuite struct {
*require.Assertions
FunctionalTestBase
}
)
func (s *DescribeTaskQueueSuite) SetupSuite() {
s.setupSuite("testdata/es_cluster.yaml")
}
func (s *DescribeTaskQueueSuite) TearDownSuite() {
s.tearDownSuite()
}
func (s *DescribeTaskQueueSuite) SetupTest() {
s.FunctionalTestBase.SetupTest()
// Have to define our overridden assertions in the test setup. If we did it earlier, s.T() will return nil
s.Assertions = require.New(s.T())
}
func (s *DescribeTaskQueueSuite) TestAddNoTasks_ValidateStats() {
// Override the ReadPartitions and WritePartitions
s.OverrideDynamicConfig(dynamicconfig.MatchingNumTaskqueueReadPartitions, 4)
s.OverrideDynamicConfig(dynamicconfig.MatchingNumTaskqueueWritePartitions, 4)
s.OverrideDynamicConfig(dynamicconfig.MatchingLongPollExpirationInterval, 10*time.Second)
s.publishConsumeWorkflowTasksValidateStats(0, true)
}
func (s *DescribeTaskQueueSuite) TestAddSingleTask_ValidateStats() {
s.OverrideDynamicConfig(dynamicconfig.MatchingUpdateAckInterval, 5*time.Second)
s.testWithMatchingBehavior(func() { s.publishConsumeWorkflowTasksValidateStats(1, true) })
}
func (s *DescribeTaskQueueSuite) TestAddMultipleTasksMultiplePartitions_ValidateStats() {
// Override the ReadPartitions and WritePartitions
s.OverrideDynamicConfig(dynamicconfig.MatchingNumTaskqueueReadPartitions, 4)
s.OverrideDynamicConfig(dynamicconfig.MatchingNumTaskqueueWritePartitions, 4)
s.OverrideDynamicConfig(dynamicconfig.MatchingLongPollExpirationInterval, 10*time.Second)
s.publishConsumeWorkflowTasksValidateStats(100, true)
}
func (s *DescribeTaskQueueSuite) TestAddSingleTask_ValidateStatsLegacyAPIMode() {
// Override the ReadPartitions and WritePartitions
s.OverrideDynamicConfig(dynamicconfig.MatchingNumTaskqueueReadPartitions, 1)
s.OverrideDynamicConfig(dynamicconfig.MatchingNumTaskqueueWritePartitions, 1)
s.OverrideDynamicConfig(dynamicconfig.MatchingLongPollExpirationInterval, 10*time.Second)
s.publishConsumeWorkflowTasksValidateStats(1, false)
}
func (s *DescribeTaskQueueSuite) publishConsumeWorkflowTasksValidateStats(workflows int, isEnhancedMode bool) {
expectedBacklogCount := make(map[enumspb.TaskQueueType]int64)
maxBacklogExtraTasks := make(map[enumspb.TaskQueueType]int64)
expectedAddRate := make(map[enumspb.TaskQueueType]bool)
expectedDispatchRate := make(map[enumspb.TaskQueueType]bool)
// Actual counter can be greater than the expected due to History->Matching retries. We make sure the counter is in
// range [expected, expected+maxExtraTasksAllowed]
maxExtraTasksAllowed := int64(3)
if workflows <= 0 {
maxExtraTasksAllowed = int64(0)
}
expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = 0
maxBacklogExtraTasks[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = 0
expectedAddRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = false
expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = false
tqName := s.randomizeStr("backlog-counter-task-queue")
tq := &taskqueuepb.TaskQueue{Name: tqName, Kind: enumspb.TASK_QUEUE_KIND_NORMAL}
identity := "worker-multiple-tasks"
for i := 0; i < workflows; i++ {
id := uuid.New()
wt := "functional-workflow-multiple-tasks"
workflowType := &commonpb.WorkflowType{Name: wt}
request := &workflowservice.StartWorkflowExecutionRequest{
RequestId: uuid.New(),
Namespace: s.namespace,
WorkflowId: id,
WorkflowType: workflowType,
TaskQueue: tq,
Input: nil,
WorkflowRunTimeout: durationpb.New(10 * time.Minute),
WorkflowTaskTimeout: durationpb.New(10 * time.Minute),
Identity: identity,
}
_, err0 := s.client.StartWorkflowExecution(NewContext(), request)
s.NoError(err0)
}
expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = int64(workflows)
maxBacklogExtraTasks[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = maxExtraTasksAllowed
expectedAddRate[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = workflows > 0
expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = false
s.validateDescribeTaskQueue(tqName, expectedBacklogCount, maxBacklogExtraTasks, expectedAddRate, expectedDispatchRate, isEnhancedMode)
// Poll the tasks
for i := 0; i < workflows; {
resp1, err1 := s.client.PollWorkflowTaskQueue(NewContext(), &workflowservice.PollWorkflowTaskQueueRequest{
Namespace: s.namespace,
TaskQueue: tq,
Identity: identity,
})
s.NoError(err1)
if resp1 == nil || resp1.GetAttempt() < 1 {
continue // poll again on empty responses
}
i++
_, err := s.client.RespondWorkflowTaskCompleted(NewContext(), &workflowservice.RespondWorkflowTaskCompletedRequest{
Namespace: s.namespace,
Identity: identity,
TaskToken: resp1.TaskToken,
Commands: []*commandpb.Command{
{
CommandType: enumspb.COMMAND_TYPE_SCHEDULE_ACTIVITY_TASK,
Attributes: &commandpb.Command_ScheduleActivityTaskCommandAttributes{
ScheduleActivityTaskCommandAttributes: &commandpb.ScheduleActivityTaskCommandAttributes{
ActivityId: "activity1",
ActivityType: &commonpb.ActivityType{Name: "activity_type1"},
TaskQueue: tq,
StartToCloseTimeout: durationpb.New(time.Minute),
RequestEagerExecution: false,
},
},
},
},
})
s.NoError(err)
}
// call describeTaskQueue to verify if the WTF backlog decreased and activity backlog increased
expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = int64(0)
expectedAddRate[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = workflows > 0
expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_WORKFLOW] = workflows > 0
expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = int64(workflows)
maxBacklogExtraTasks[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = maxExtraTasksAllowed
expectedAddRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = workflows > 0
expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = false
s.validateDescribeTaskQueue(tqName, expectedBacklogCount, maxBacklogExtraTasks, expectedAddRate, expectedDispatchRate, isEnhancedMode)
// Poll the tasks
for i := 0; i < workflows; {
resp1, err1 := s.client.PollActivityTaskQueue(
NewContext(), &workflowservice.PollActivityTaskQueueRequest{
Namespace: s.namespace,
TaskQueue: tq,
Identity: identity,
},
)
s.NoError(err1)
if resp1 == nil || resp1.GetAttempt() < 1 {
continue // poll again on empty responses
}
i++
}
expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = int64(0)
expectedAddRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = workflows > 0
expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY] = workflows > 0
s.validateDescribeTaskQueue(tqName, expectedBacklogCount, maxBacklogExtraTasks, expectedAddRate, expectedDispatchRate, isEnhancedMode)
}
func (s *DescribeTaskQueueSuite) validateDescribeTaskQueue(
tq string,
expectedBacklogCount map[enumspb.TaskQueueType]int64,
maxBacklogExtraTasks map[enumspb.TaskQueueType]int64,
expectedAddRate map[enumspb.TaskQueueType]bool,
expectedDispatchRate map[enumspb.TaskQueueType]bool,
isEnhancedMode bool,
) {
ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
defer cancel()
var resp *workflowservice.DescribeTaskQueueResponse
var err error
if isEnhancedMode {
s.EventuallyWithT(func(t *assert.CollectT) {
resp, err = s.client.DescribeTaskQueue(ctx, &workflowservice.DescribeTaskQueueRequest{
Namespace: s.namespace,
TaskQueue: &taskqueuepb.TaskQueue{Name: tq, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
ApiMode: enumspb.DESCRIBE_TASK_QUEUE_MODE_ENHANCED,
Versions: nil, // default version, in this case unversioned queue
TaskQueueTypes: nil, // both types
ReportPollers: true,
ReportTaskReachability: true,
ReportStats: true,
})
s.NoError(err)
s.NotNil(resp)
s.Equal(1, len(resp.GetVersionsInfo()), "should be 1 because only default/unversioned queue")
versionInfo := resp.GetVersionsInfo()[""]
s.Equal(enumspb.BUILD_ID_TASK_REACHABILITY_REACHABLE, versionInfo.GetTaskReachability())
types := versionInfo.GetTypesInfo()
s.Equal(len(types), len(expectedBacklogCount))
wfStats := types[int32(enumspb.TASK_QUEUE_TYPE_WORKFLOW)].Stats
actStats := types[int32(enumspb.TASK_QUEUE_TYPE_ACTIVITY)].Stats
a := assert.New(t)
// Actual counter can be greater than the expected due to history retries. We make sure the counter is in
// range [expected, expected+maxBacklogExtraTasks]
a.GreaterOrEqual(wfStats.ApproximateBacklogCount, expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_WORKFLOW])
a.LessOrEqual(wfStats.ApproximateBacklogCount, expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_WORKFLOW]+maxBacklogExtraTasks[enumspb.TASK_QUEUE_TYPE_WORKFLOW])
a.GreaterOrEqual(actStats.ApproximateBacklogCount, expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_ACTIVITY])
a.LessOrEqual(actStats.ApproximateBacklogCount, expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_ACTIVITY]+maxBacklogExtraTasks[enumspb.TASK_QUEUE_TYPE_ACTIVITY])
a.Equal(wfStats.ApproximateBacklogCount == 0, wfStats.ApproximateBacklogAge.AsDuration() == time.Duration(0))
a.Equal(actStats.ApproximateBacklogCount == 0, actStats.ApproximateBacklogAge.AsDuration() == time.Duration(0))
a.Equal(expectedAddRate[enumspb.TASK_QUEUE_TYPE_WORKFLOW], wfStats.TasksAddRate > 0)
a.Equal(expectedAddRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY], actStats.TasksAddRate > 0)
a.Equal(expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_WORKFLOW], wfStats.TasksDispatchRate > 0)
a.Equal(expectedDispatchRate[enumspb.TASK_QUEUE_TYPE_ACTIVITY], actStats.TasksDispatchRate > 0)
}, 6*time.Second, 100*time.Millisecond)
} else {
// Querying the Legacy API
s.Eventually(func() bool {
resp, err = s.client.DescribeTaskQueue(ctx, &workflowservice.DescribeTaskQueueRequest{
Namespace: s.namespace,
TaskQueue: &taskqueuepb.TaskQueue{Name: tq, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
ApiMode: enumspb.DESCRIBE_TASK_QUEUE_MODE_UNSPECIFIED,
IncludeTaskQueueStatus: true,
})
s.NoError(err)
s.NotNil(resp)
return resp.TaskQueueStatus.GetBacklogCountHint() == expectedBacklogCount[enumspb.TASK_QUEUE_TYPE_WORKFLOW]
}, 6*time.Second, 100*time.Millisecond)
}
}