mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-04 02:04:03 +00:00
## What changed? - Schedule is paused in the flaky test before any visibility requests are made. ## Why? - Otherwise, the schedule may take long enough to have advanced in FutureActionTimes
1245 lines
41 KiB
Go
1245 lines
41 KiB
Go
package tests
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"fmt"
|
|
"strings"
|
|
"sync/atomic"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/google/uuid"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"github.com/stretchr/testify/suite"
|
|
commonpb "go.temporal.io/api/common/v1"
|
|
enumspb "go.temporal.io/api/enums/v1"
|
|
schedulepb "go.temporal.io/api/schedule/v1"
|
|
taskqueuepb "go.temporal.io/api/taskqueue/v1"
|
|
workflowpb "go.temporal.io/api/workflow/v1"
|
|
"go.temporal.io/api/workflowservice/v1"
|
|
sdkclient "go.temporal.io/sdk/client"
|
|
"go.temporal.io/sdk/converter"
|
|
"go.temporal.io/sdk/worker"
|
|
"go.temporal.io/sdk/workflow"
|
|
"go.temporal.io/server/common/dynamicconfig"
|
|
"go.temporal.io/server/common/headers"
|
|
"go.temporal.io/server/common/payload"
|
|
"go.temporal.io/server/common/payloads"
|
|
"go.temporal.io/server/common/searchattribute/sadefs"
|
|
"go.temporal.io/server/common/testing/protorequire"
|
|
"go.temporal.io/server/service/worker/scheduler"
|
|
"go.temporal.io/server/tests/testcore"
|
|
"google.golang.org/grpc/metadata"
|
|
"google.golang.org/protobuf/proto"
|
|
"google.golang.org/protobuf/types/known/durationpb"
|
|
)
|
|
|
|
/*
|
|
more tests to write:
|
|
|
|
various validation errors
|
|
|
|
overlap policies, esp. buffer
|
|
|
|
worker restart/long-poll activity failure:
|
|
get it in a state where it's waiting for a wf to exit, say with bufferone
|
|
restart the worker/force activity to fail
|
|
terminate the wf
|
|
check that new one starts immediately
|
|
*/
|
|
|
|
type (
|
|
scheduleFunctionalSuiteBase struct {
|
|
testcore.FunctionalTestBase
|
|
|
|
sdkClient sdkclient.Client
|
|
worker worker.Worker
|
|
taskQueue string
|
|
dataConverter converter.DataConverter
|
|
newContext func() context.Context
|
|
}
|
|
|
|
ScheduleCHASMFunctionalSuite struct {
|
|
scheduleFunctionalSuiteBase
|
|
}
|
|
|
|
ScheduleV1FunctionalSuite struct {
|
|
scheduleFunctionalSuiteBase
|
|
}
|
|
)
|
|
|
|
func TestScheduleFunctionalSuite(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
// CHASM tests must run as a separate suite, with a separate cluster/functional environment, because the tests
|
|
// assume a fully clean state. For example, TestBasics has assertions on visibility entries for workflow runs
|
|
// started by the scheduler, which would not be cleaned up even when the associated scheduler has been deleted.
|
|
suite.Run(t, new(ScheduleCHASMFunctionalSuite))
|
|
suite.Run(t, new(ScheduleV1FunctionalSuite))
|
|
}
|
|
|
|
func (s *ScheduleCHASMFunctionalSuite) SetupTest() {
|
|
s.newContext = func() context.Context {
|
|
baseCtx := testcore.NewContext()
|
|
return metadata.NewOutgoingContext(baseCtx, metadata.Pairs(
|
|
headers.ExperimentHeaderName, "chasm-scheduler",
|
|
))
|
|
}
|
|
s.scheduleFunctionalSuiteBase.SetupTest()
|
|
}
|
|
|
|
func (s *ScheduleV1FunctionalSuite) SetupTest() {
|
|
s.newContext = func() context.Context {
|
|
return testcore.NewContext()
|
|
}
|
|
s.scheduleFunctionalSuiteBase.SetupTest()
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) SetupTest() {
|
|
s.OverrideDynamicConfig(dynamicconfig.EnableChasm, true)
|
|
s.OverrideDynamicConfig(dynamicconfig.FrontendAllowedExperiments, []string{"*"})
|
|
s.FunctionalTestBase.SetupTest()
|
|
s.dataConverter = testcore.NewTestDataConverter()
|
|
|
|
var err error
|
|
s.sdkClient, err = sdkclient.Dial(sdkclient.Options{
|
|
HostPort: s.FrontendGRPCAddress(),
|
|
Namespace: s.Namespace().String(),
|
|
DataConverter: s.dataConverter,
|
|
})
|
|
s.NoError(err)
|
|
|
|
s.taskQueue = testcore.RandomizeStr("tq")
|
|
s.worker = worker.New(s.sdkClient, s.taskQueue, worker.Options{})
|
|
err = s.worker.Start()
|
|
s.NoError(err)
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) TearDownTest() {
|
|
if s.worker != nil {
|
|
s.worker.Stop()
|
|
}
|
|
if s.sdkClient != nil {
|
|
s.sdkClient.Close()
|
|
}
|
|
s.FunctionalTestBase.TearDownTest()
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) TestBasics() {
|
|
sid := "sched-test-basics"
|
|
wid := "sched-test-basics-wf"
|
|
wt := "sched-test-basics-wt"
|
|
wt2 := "sched-test-basics-wt2"
|
|
// switch this to test with search attribute mapper:
|
|
// csaKeyword := "AliasForCustomKeywordField"
|
|
csaKeyword := "CustomKeywordField"
|
|
csaInt := "CustomIntField"
|
|
csaBool := "CustomBoolField"
|
|
|
|
wfMemo := payload.EncodeString("workflow memo")
|
|
wfSAValue := payload.EncodeString("workflow sa value")
|
|
schMemo := payload.EncodeString("schedule memo")
|
|
schSAValue := payload.EncodeString("schedule sa value")
|
|
schSAIntValue, _ := payload.Encode(123)
|
|
schSABoolValue, _ := payload.Encode(true)
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(5 * time.Second)},
|
|
},
|
|
Calendar: []*schedulepb.CalendarSpec{
|
|
{DayOfMonth: "10", Year: "2010"},
|
|
},
|
|
CronString: []string{"11 11/11 11 11 1 2011"},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
Memo: &commonpb.Memo{
|
|
Fields: map[string]*commonpb.Payload{"wfmemo1": wfMemo},
|
|
},
|
|
SearchAttributes: &commonpb.SearchAttributes{
|
|
IndexedFields: map[string]*commonpb.Payload{csaKeyword: wfSAValue},
|
|
},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
Memo: &commonpb.Memo{
|
|
Fields: map[string]*commonpb.Payload{"schedmemo1": schMemo},
|
|
},
|
|
SearchAttributes: &commonpb.SearchAttributes{
|
|
IndexedFields: map[string]*commonpb.Payload{
|
|
csaKeyword: schSAValue,
|
|
csaInt: schSAIntValue,
|
|
csaBool: schSABoolValue,
|
|
},
|
|
},
|
|
}
|
|
|
|
var runs, runs2 int32
|
|
workflowFn := func(ctx workflow.Context) error {
|
|
workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
atomic.AddInt32(&runs, 1)
|
|
return 0
|
|
})
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
workflow2Fn := func(ctx workflow.Context) error {
|
|
workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
atomic.AddInt32(&runs2, 1)
|
|
return 0
|
|
})
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflow2Fn, workflow.RegisterOptions{Name: wt2})
|
|
|
|
// create
|
|
|
|
ctx := s.newContext()
|
|
createTime := time.Now()
|
|
_, err := s.FrontendClient().CreateSchedule(ctx, req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
// sleep until we see two runs, plus a bit more to ensure that the second run has completed
|
|
s.Eventually(func() bool { return atomic.LoadInt32(&runs) == 2 }, 15*time.Second, 500*time.Millisecond)
|
|
time.Sleep(2 * time.Second) //nolint:forbidigo
|
|
|
|
// wait for visibility to stabilize on completed before calling describe,
|
|
// otherwise their recent actions may flake and differ
|
|
|
|
visibilityResponse := s.getScheduleEntryFomVisibility(sid, func(ent *schedulepb.ScheduleListEntry) bool {
|
|
recentActions := ent.GetInfo().GetRecentActions()
|
|
return len(recentActions) >= 2 && recentActions[1].GetStartWorkflowStatus() == enumspb.WORKFLOW_EXECUTION_STATUS_COMPLETED
|
|
})
|
|
|
|
describeResp, err := s.FrontendClient().DescribeSchedule(s.newContext(), &workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
})
|
|
s.NoError(err)
|
|
|
|
// validate describe response
|
|
|
|
checkSpec := func(spec *schedulepb.ScheduleSpec) {
|
|
protorequire.ProtoSliceEqual(s.T(), schedule.Spec.Interval, spec.Interval)
|
|
s.Nil(spec.Calendar)
|
|
s.Nil(spec.CronString)
|
|
s.ProtoElementsMatch([]*schedulepb.StructuredCalendarSpec{
|
|
{
|
|
Second: []*schedulepb.Range{{Start: 0, End: 0, Step: 1}},
|
|
Minute: []*schedulepb.Range{{Start: 11, End: 11, Step: 1}},
|
|
Hour: []*schedulepb.Range{{Start: 11, End: 23, Step: 11}},
|
|
DayOfMonth: []*schedulepb.Range{{Start: 11, End: 11, Step: 1}},
|
|
Month: []*schedulepb.Range{{Start: 11, End: 11, Step: 1}},
|
|
DayOfWeek: []*schedulepb.Range{{Start: 1, End: 1, Step: 1}},
|
|
Year: []*schedulepb.Range{{Start: 2011, End: 2011, Step: 1}},
|
|
},
|
|
{
|
|
Second: []*schedulepb.Range{{Start: 0, End: 0, Step: 1}},
|
|
Minute: []*schedulepb.Range{{Start: 0, End: 0, Step: 1}},
|
|
Hour: []*schedulepb.Range{{Start: 0, End: 0, Step: 1}},
|
|
DayOfMonth: []*schedulepb.Range{{Start: 10, End: 10, Step: 1}},
|
|
Month: []*schedulepb.Range{{Start: 1, End: 12, Step: 1}},
|
|
DayOfWeek: []*schedulepb.Range{{Start: 0, End: 6, Step: 1}},
|
|
Year: []*schedulepb.Range{{Start: 2010, End: 2010, Step: 1}},
|
|
},
|
|
}, spec.StructuredCalendar)
|
|
}
|
|
checkSpec(describeResp.Schedule.Spec)
|
|
|
|
s.Equal(enumspb.SCHEDULE_OVERLAP_POLICY_SKIP, describeResp.Schedule.Policies.OverlapPolicy) // set to default value
|
|
s.EqualValues(365*24*3600, describeResp.Schedule.Policies.CatchupWindow.AsDuration().Seconds()) // set to default value
|
|
|
|
s.Equal(schSAValue.Data, describeResp.SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
s.Equal(schSAIntValue.Data, describeResp.SearchAttributes.IndexedFields[csaInt].Data)
|
|
s.Equal(schSABoolValue.Data, describeResp.SearchAttributes.IndexedFields[csaBool].Data)
|
|
s.Nil(describeResp.SearchAttributes.IndexedFields[sadefs.BinaryChecksums])
|
|
s.Nil(describeResp.SearchAttributes.IndexedFields[sadefs.BuildIds])
|
|
s.Nil(describeResp.SearchAttributes.IndexedFields[sadefs.TemporalNamespaceDivision])
|
|
s.Equal(schMemo.Data, describeResp.Memo.Fields["schedmemo1"].Data)
|
|
s.Equal(wfSAValue.Data, describeResp.Schedule.Action.GetStartWorkflow().SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
s.Equal(wfMemo.Data, describeResp.Schedule.Action.GetStartWorkflow().Memo.Fields["wfmemo1"].Data)
|
|
|
|
// GreaterOrEqual is used as we may have had other runs start while waiting for visibility
|
|
s.DurationNear(describeResp.Info.CreateTime.AsTime().Sub(createTime), 0, 3*time.Second)
|
|
s.GreaterOrEqual(describeResp.Info.ActionCount, int64(2))
|
|
s.EqualValues(0, describeResp.Info.MissedCatchupWindow)
|
|
s.EqualValues(0, describeResp.Info.OverlapSkipped)
|
|
s.GreaterOrEqual(len(describeResp.Info.RunningWorkflows), 0)
|
|
s.GreaterOrEqual(len(describeResp.Info.RecentActions), 2)
|
|
action0 := describeResp.Info.RecentActions[0]
|
|
s.WithinRange(action0.ScheduleTime.AsTime(), createTime, time.Now())
|
|
s.True(action0.ScheduleTime.AsTime().UnixNano()%int64(5*time.Second) == 0)
|
|
s.DurationNear(action0.ActualTime.AsTime().Sub(action0.ScheduleTime.AsTime()), 0, 3*time.Second)
|
|
|
|
// validate list response
|
|
|
|
s.Equal(sid, visibilityResponse.ScheduleId)
|
|
s.Equal(schSAValue.Data, visibilityResponse.SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
s.Equal(schSAIntValue.Data, describeResp.SearchAttributes.IndexedFields[csaInt].Data)
|
|
s.Equal(schSABoolValue.Data, describeResp.SearchAttributes.IndexedFields[csaBool].Data)
|
|
s.Nil(visibilityResponse.SearchAttributes.IndexedFields[sadefs.BinaryChecksums])
|
|
s.Nil(visibilityResponse.SearchAttributes.IndexedFields[sadefs.BuildIds])
|
|
s.Nil(visibilityResponse.SearchAttributes.IndexedFields[sadefs.TemporalNamespaceDivision])
|
|
s.Equal(schMemo.Data, visibilityResponse.Memo.Fields["schedmemo1"].Data)
|
|
checkSpec(visibilityResponse.Info.Spec)
|
|
s.Equal(wt, visibilityResponse.Info.WorkflowType.Name)
|
|
s.False(visibilityResponse.Info.Paused)
|
|
s.assertSameRecentActions(describeResp, visibilityResponse)
|
|
|
|
// list workflows
|
|
|
|
wfResp, err := s.FrontendClient().ListWorkflowExecutions(s.newContext(), &workflowservice.ListWorkflowExecutionsRequest{
|
|
Namespace: s.Namespace().String(),
|
|
PageSize: 5,
|
|
Query: "",
|
|
})
|
|
s.NoError(err)
|
|
s.GreaterOrEqual(len(wfResp.Executions), 2) // could have had a 3rd run while waiting for visibility
|
|
for _, ex := range wfResp.Executions {
|
|
s.Equal(wt, ex.Type.Name, "should only see started workflows")
|
|
}
|
|
ex0 := wfResp.Executions[0]
|
|
s.True(strings.HasPrefix(ex0.Execution.WorkflowId, wid))
|
|
matchingRunId := false
|
|
for _, recentAction := range describeResp.GetInfo().GetRecentActions() {
|
|
if ex0.GetExecution().GetRunId() == recentAction.GetStartWorkflowResult().GetRunId() {
|
|
matchingRunId = true
|
|
break
|
|
}
|
|
}
|
|
s.True(matchingRunId, "ListWorkflowExecutions returned a run ID wasn't in the describe response")
|
|
s.Equal(wt, ex0.Type.Name)
|
|
s.Nil(ex0.ParentExecution) // not a child workflow
|
|
s.Equal(wfMemo.Data, ex0.Memo.Fields["wfmemo1"].Data)
|
|
s.Equal(wfSAValue.Data, ex0.SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
s.Equal(payload.EncodeString(sid).Data, ex0.SearchAttributes.IndexedFields[sadefs.TemporalScheduledById].Data)
|
|
var ex0StartTime time.Time
|
|
s.NoError(payload.Decode(ex0.SearchAttributes.IndexedFields[sadefs.TemporalScheduledStartTime], &ex0StartTime))
|
|
s.WithinRange(ex0StartTime, createTime, time.Now())
|
|
s.True(ex0StartTime.UnixNano()%int64(5*time.Second) == 0)
|
|
|
|
// list schedules with search attribute filter
|
|
|
|
listResp, err := s.FrontendClient().ListSchedules(s.newContext(), &workflowservice.ListSchedulesRequest{
|
|
Namespace: s.Namespace().String(),
|
|
MaximumPageSize: 5,
|
|
Query: "CustomKeywordField = 'schedule sa value' AND TemporalSchedulePaused = false",
|
|
})
|
|
s.NoError(err)
|
|
s.Len(listResp.Schedules, 1)
|
|
entry := listResp.Schedules[0]
|
|
s.Equal(sid, entry.ScheduleId)
|
|
|
|
// list schedules with invalid search attribute filter
|
|
|
|
_, err = s.FrontendClient().ListSchedules(s.newContext(), &workflowservice.ListSchedulesRequest{
|
|
Namespace: s.Namespace().String(),
|
|
MaximumPageSize: 5,
|
|
Query: "ExecutionDuration > '1s'",
|
|
})
|
|
s.Error(err)
|
|
|
|
// update schedule, no updates to search attributes
|
|
|
|
schedule.Spec.Interval[0].Phase = durationpb.New(1 * time.Second)
|
|
schedule.Action.GetStartWorkflow().WorkflowType.Name = wt2
|
|
|
|
updateTime := time.Now()
|
|
_, err = s.FrontendClient().UpdateSchedule(s.newContext(), &workflowservice.UpdateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
})
|
|
s.NoError(err)
|
|
|
|
// wait for one new run
|
|
s.Eventually(
|
|
func() bool { return atomic.LoadInt32(&runs2) == 1 },
|
|
7*time.Second,
|
|
500*time.Millisecond,
|
|
)
|
|
|
|
// describe again
|
|
describeResp, err = s.FrontendClient().DescribeSchedule(
|
|
s.newContext(),
|
|
&workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
},
|
|
)
|
|
s.NoError(err)
|
|
|
|
s.Len(describeResp.SearchAttributes.GetIndexedFields(), 3)
|
|
s.Equal(schSAValue.Data, describeResp.SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
s.Equal(schSAIntValue.Data, describeResp.SearchAttributes.IndexedFields[csaInt].Data)
|
|
s.Equal(schSABoolValue.Data, describeResp.SearchAttributes.IndexedFields[csaBool].Data)
|
|
s.Equal(schMemo.Data, describeResp.Memo.Fields["schedmemo1"].Data)
|
|
s.Equal(wfSAValue.Data, describeResp.Schedule.Action.GetStartWorkflow().SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
s.Equal(wfMemo.Data, describeResp.Schedule.Action.GetStartWorkflow().Memo.Fields["wfmemo1"].Data)
|
|
|
|
s.DurationNear(describeResp.Info.UpdateTime.AsTime().Sub(updateTime), 0, 3*time.Second)
|
|
lastAction := describeResp.Info.RecentActions[len(describeResp.Info.RecentActions)-1]
|
|
s.True(lastAction.ScheduleTime.AsTime().UnixNano()%int64(5*time.Second) == 1000000000, lastAction.ScheduleTime.AsTime().UnixNano())
|
|
|
|
// update schedule and search attributes
|
|
|
|
schedule.Spec.Interval[0].Phase = durationpb.New(1 * time.Second)
|
|
schedule.Action.GetStartWorkflow().WorkflowType.Name = wt2
|
|
|
|
csaDouble := "CustomDoubleField"
|
|
schSADoubleValue, _ := payload.Encode(3.14)
|
|
schSAIntValue, _ = payload.Encode(321)
|
|
_, err = s.FrontendClient().UpdateSchedule(s.newContext(), &workflowservice.UpdateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
SearchAttributes: &commonpb.SearchAttributes{
|
|
IndexedFields: map[string]*commonpb.Payload{
|
|
csaKeyword: schSAValue, // same key, same value
|
|
csaInt: schSAIntValue, // same key, new value
|
|
csaDouble: schSADoubleValue, // new key
|
|
// csaBool is removed
|
|
},
|
|
},
|
|
})
|
|
s.NoError(err)
|
|
|
|
// wait until search attributes are updated
|
|
s.EventuallyWithT(
|
|
func(c *assert.CollectT) {
|
|
describeResp, err = s.FrontendClient().DescribeSchedule(
|
|
s.newContext(),
|
|
&workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
},
|
|
)
|
|
require.NoError(c, err)
|
|
require.Len(c, describeResp.SearchAttributes.GetIndexedFields(), 3)
|
|
require.Equal(c, schSAValue.Data, describeResp.SearchAttributes.IndexedFields[csaKeyword].Data)
|
|
require.Equal(c, schSAIntValue.Data, describeResp.SearchAttributes.IndexedFields[csaInt].Data)
|
|
require.Equal(c, schSADoubleValue.Data, describeResp.SearchAttributes.IndexedFields[csaDouble].Data)
|
|
require.NotContains(c, describeResp.SearchAttributes.IndexedFields, csaBool)
|
|
},
|
|
2*time.Second,
|
|
500*time.Millisecond,
|
|
)
|
|
|
|
// update schedule and unset search attributes
|
|
|
|
schedule.Spec.Interval[0].Phase = durationpb.New(1 * time.Second)
|
|
schedule.Action.GetStartWorkflow().WorkflowType.Name = wt2
|
|
|
|
_, err = s.FrontendClient().UpdateSchedule(s.newContext(), &workflowservice.UpdateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
SearchAttributes: &commonpb.SearchAttributes{},
|
|
})
|
|
s.NoError(err)
|
|
|
|
// wait until search attributes are updated
|
|
s.EventuallyWithT(
|
|
func(c *assert.CollectT) {
|
|
describeResp, err = s.FrontendClient().DescribeSchedule(
|
|
s.newContext(),
|
|
&workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
},
|
|
)
|
|
require.NoError(c, err)
|
|
require.Empty(c, describeResp.SearchAttributes.GetIndexedFields())
|
|
},
|
|
5*time.Second,
|
|
500*time.Millisecond,
|
|
)
|
|
|
|
// pause
|
|
|
|
_, err = s.FrontendClient().PatchSchedule(s.newContext(), &workflowservice.PatchScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Patch: &schedulepb.SchedulePatch{
|
|
Pause: "because I said so",
|
|
},
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
})
|
|
s.NoError(err)
|
|
|
|
time.Sleep(7 * time.Second) //nolint:forbidigo
|
|
s.EqualValues(1, atomic.LoadInt32(&runs2), "has not run again")
|
|
|
|
describeResp, err = s.FrontendClient().DescribeSchedule(s.newContext(), &workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
})
|
|
s.NoError(err)
|
|
|
|
s.True(describeResp.Schedule.State.Paused)
|
|
s.Equal("because I said so", describeResp.Schedule.State.Notes)
|
|
|
|
// don't loop to wait for visibility, we already waited 7s from the patch
|
|
listResp, err = s.FrontendClient().ListSchedules(s.newContext(), &workflowservice.ListSchedulesRequest{
|
|
Namespace: s.Namespace().String(),
|
|
MaximumPageSize: 5,
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(1, len(listResp.Schedules))
|
|
entry = listResp.Schedules[0]
|
|
s.Equal(sid, entry.ScheduleId)
|
|
s.True(entry.Info.Paused)
|
|
s.Equal("because I said so", entry.Info.Notes)
|
|
|
|
// finally delete
|
|
|
|
_, err = s.FrontendClient().DeleteSchedule(s.newContext(), &workflowservice.DeleteScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Identity: "test",
|
|
})
|
|
s.NoError(err)
|
|
|
|
describeResp, err = s.FrontendClient().DescribeSchedule(s.newContext(), &workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
})
|
|
s.Error(err)
|
|
|
|
s.Eventually(func() bool { // wait for visibility
|
|
listResp, err := s.FrontendClient().ListSchedules(s.newContext(), &workflowservice.ListSchedulesRequest{
|
|
Namespace: s.Namespace().String(),
|
|
MaximumPageSize: 5,
|
|
})
|
|
s.NoError(err)
|
|
return len(listResp.Schedules) == 0
|
|
}, 10*time.Second, 1*time.Second)
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) TestInput() {
|
|
sid := "sched-test-input"
|
|
wid := "sched-test-input-wf"
|
|
wt := "sched-test-input-wt"
|
|
|
|
type myData struct {
|
|
Stuff string
|
|
Things []int
|
|
}
|
|
|
|
input1 := &myData{
|
|
Stuff: "here's some data",
|
|
Things: []int{7, 8, 9},
|
|
}
|
|
input2 := map[int]float64{11: 1.4375}
|
|
inputPayloads, err := s.dataConverter.ToPayloads(input1, input2)
|
|
s.NoError(err)
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(3 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
Input: inputPayloads,
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
|
|
var runs int32
|
|
workflowFn := func(ctx workflow.Context, arg1 *myData, arg2 map[int]float64) error {
|
|
workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
s.Equal(*input1, *arg1)
|
|
s.Equal(input2, arg2)
|
|
atomic.AddInt32(&runs, 1)
|
|
return 0
|
|
})
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
|
|
ctx := s.newContext()
|
|
_, err = s.FrontendClient().CreateSchedule(ctx, req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
s.Eventually(func() bool { return atomic.LoadInt32(&runs) == 1 }, 8*time.Second, 200*time.Millisecond)
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) TestLastCompletionAndError() {
|
|
sid := "sched-test-last"
|
|
wid := "sched-test-last-wf"
|
|
wt := "sched-test-last-wt"
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(3 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
|
|
runs := make(map[string]struct{})
|
|
var testComplete int32
|
|
|
|
workflowFn := func(ctx workflow.Context) (string, error) {
|
|
var num int
|
|
_ = workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
runs[workflow.GetInfo(ctx).WorkflowExecution.ID] = struct{}{}
|
|
return len(runs)
|
|
}).Get(&num)
|
|
|
|
var lcr string
|
|
if workflow.HasLastCompletionResult(ctx) {
|
|
s.NoError(workflow.GetLastCompletionResult(ctx, &lcr))
|
|
}
|
|
|
|
lastErr := workflow.GetLastError(ctx)
|
|
|
|
switch num {
|
|
case 1:
|
|
s.Equal("", lcr)
|
|
s.NoError(lastErr)
|
|
return "this one succeeds", nil
|
|
case 2:
|
|
s.NoError(lastErr)
|
|
s.Equal("this one succeeds", lcr)
|
|
return "", errors.New("this one fails")
|
|
case 3:
|
|
s.Equal("this one succeeds", lcr)
|
|
s.ErrorContains(lastErr, "this one fails")
|
|
atomic.StoreInt32(&testComplete, 1)
|
|
return "done", nil
|
|
default:
|
|
panic("shouldn't be running anymore")
|
|
}
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
|
|
ctx := s.newContext()
|
|
_, err := s.FrontendClient().CreateSchedule(ctx, req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
s.Eventually(func() bool { return atomic.LoadInt32(&testComplete) == 1 }, 20*time.Second, 200*time.Millisecond)
|
|
}
|
|
|
|
// Tests that a schedule created in the V1 stack will also be visible in the V2 stack.
|
|
func (s *ScheduleV1FunctionalSuite) TestCHASMCanListV1Schedules() {
|
|
sid := "schedule-created-on-v1"
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(3 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: "wf-",
|
|
WorkflowType: &commonpb.WorkflowType{Name: "action"},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
|
|
// Create on V1 stack.
|
|
_, err := s.FrontendClient().CreateSchedule(s.newContext(), req)
|
|
s.NoError(err)
|
|
|
|
// Pause so that `FutureActionTimes` doesn't change between calls.
|
|
_, err = s.FrontendClient().PatchSchedule(s.newContext(), &workflowservice.PatchScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Patch: &schedulepb.SchedulePatch{
|
|
Pause: "halt",
|
|
},
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
})
|
|
s.NoError(err)
|
|
|
|
// Sanity test, list with V1 handler.
|
|
v1Entry := s.getScheduleEntryFomVisibility(sid, func(sle *schedulepb.ScheduleListEntry) bool {
|
|
return sle.GetInfo().Paused
|
|
})
|
|
s.NotNil(v1Entry.GetInfo())
|
|
|
|
// Flip on CHASM experiment and make sure we can still list.
|
|
s.newContext = func() context.Context {
|
|
return metadata.NewOutgoingContext(testcore.NewContext(), metadata.Pairs(
|
|
headers.ExperimentHeaderName, "chasm-scheduler",
|
|
))
|
|
}
|
|
chasmEntry := s.getScheduleEntryFomVisibility(sid, nil)
|
|
s.NotNil(chasmEntry.GetInfo())
|
|
s.ProtoEqual(chasmEntry.GetInfo(), v1Entry.GetInfo())
|
|
}
|
|
|
|
// TestRefresh applies to V1 scheduler only; V2 does not support/need manual refresh.
|
|
func (s *ScheduleV1FunctionalSuite) TestRefresh() {
|
|
sid := "sched-test-refresh"
|
|
wid := "sched-test-refresh-wf"
|
|
wt := "sched-test-refresh-wt"
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{
|
|
Interval: durationpb.New(30 * time.Second),
|
|
// start within three seconds
|
|
Phase: durationpb.New(time.Duration((time.Now().Unix()+3)%30) * time.Second),
|
|
},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
WorkflowExecutionTimeout: durationpb.New(3 * time.Second),
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
|
|
var runs int32
|
|
workflowFn := func(ctx workflow.Context) error {
|
|
workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
atomic.AddInt32(&runs, 1)
|
|
return 0
|
|
})
|
|
s.NoError(workflow.Sleep(ctx, 10*time.Second)) // longer than execution timeout
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
|
|
_, err := s.FrontendClient().CreateSchedule(s.newContext(), req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
s.Eventually(func() bool { return atomic.LoadInt32(&runs) == 1 }, 6*time.Second, 200*time.Millisecond)
|
|
|
|
// workflow has started but is now sleeping. it will timeout in 2 seconds.
|
|
|
|
describeResp, err := s.FrontendClient().DescribeSchedule(s.newContext(), &workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
})
|
|
s.NoError(err)
|
|
s.EqualValues(1, len(describeResp.Info.RunningWorkflows))
|
|
|
|
events1 := s.GetHistory(s.Namespace().String(), &commonpb.WorkflowExecution{WorkflowId: scheduler.WorkflowIDPrefix + sid})
|
|
expectedHistory := `
|
|
1 WorkflowExecutionStarted
|
|
2 WorkflowTaskScheduled
|
|
3 WorkflowTaskStarted
|
|
4 WorkflowTaskCompleted
|
|
5 MarkerRecorded
|
|
6 MarkerRecorded
|
|
7 UpsertWorkflowSearchAttributes
|
|
8 TimerStarted
|
|
9 TimerFired
|
|
10 WorkflowTaskScheduled
|
|
11 WorkflowTaskStarted
|
|
12 WorkflowTaskCompleted
|
|
13 MarkerRecorded
|
|
14 MarkerRecorded
|
|
15 WorkflowPropertiesModified
|
|
16 TimerStarted`
|
|
|
|
s.EqualHistoryEvents(expectedHistory, events1)
|
|
|
|
time.Sleep(4 * time.Second) //nolint:forbidigo
|
|
// now it has timed out, but the scheduler hasn't noticed yet. we can prove it by checking
|
|
// its history.
|
|
|
|
events2 := s.GetHistory(s.Namespace().String(), &commonpb.WorkflowExecution{WorkflowId: scheduler.WorkflowIDPrefix + sid})
|
|
s.EqualHistoryEvents(expectedHistory, events2)
|
|
|
|
// when we describe we'll force a refresh and see it timed out
|
|
describeResp, err = s.FrontendClient().DescribeSchedule(s.newContext(), &workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
})
|
|
s.NoError(err)
|
|
s.EqualValues(0, len(describeResp.Info.RunningWorkflows))
|
|
|
|
// check scheduler has gotten the refresh and done some stuff. signal is sent without waiting so we need to wait.
|
|
s.Eventually(func() bool {
|
|
events3 := s.GetHistory(s.Namespace().String(), &commonpb.WorkflowExecution{WorkflowId: scheduler.WorkflowIDPrefix + sid})
|
|
return len(events3) > len(events2)
|
|
}, 5*time.Second, 100*time.Millisecond)
|
|
}
|
|
|
|
// TestListBeforeRun only applies to V1, as V2 scheduler does not involve the
|
|
// per-NS worker or workflow.
|
|
func (s *ScheduleV1FunctionalSuite) TestListBeforeRun() {
|
|
sid := "sched-test-list-before-run"
|
|
wid := "sched-test-list-before-run-wf"
|
|
wt := "sched-test-list-before-run-wt"
|
|
|
|
// disable per-ns worker so that the schedule workflow never runs
|
|
s.OverrideDynamicConfig(dynamicconfig.WorkerPerNamespaceWorkerCount, 0)
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(3 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
|
|
startTime := time.Now()
|
|
|
|
_, err := s.FrontendClient().CreateSchedule(s.newContext(), req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
entry := s.getScheduleEntryFomVisibility(sid, nil)
|
|
s.NotNil(entry.Info)
|
|
s.ProtoEqual(schedule.Spec, entry.Info.Spec)
|
|
s.Equal(wt, entry.Info.WorkflowType.Name)
|
|
s.False(entry.Info.Paused)
|
|
s.Greater(len(entry.Info.FutureActionTimes), 1)
|
|
s.True(entry.Info.FutureActionTimes[0].AsTime().After(startTime))
|
|
}
|
|
|
|
// TestRateLimit applies only to V1, as V2 scheduler does not impose its own
|
|
// rate limiting.
|
|
func (s *ScheduleV1FunctionalSuite) TestRateLimit() {
|
|
sid := "sched-test-rate-limit-%d"
|
|
wid := "sched-test-rate-limit-wf-%d"
|
|
wt := "sched-test-rate-limit-wt"
|
|
|
|
// Set 1/sec rate limit per namespace.
|
|
s.OverrideDynamicConfig(dynamicconfig.SchedulerNamespaceStartWorkflowRPS, 1.0)
|
|
|
|
var runs int32
|
|
workflowFn := func(ctx workflow.Context) error {
|
|
workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
atomic.AddInt32(&runs, 1)
|
|
return 0
|
|
})
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
|
|
// create 10 copies of the schedule
|
|
for i := 0; i < 10; i++ {
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(1 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: fmt.Sprintf(wid, i),
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
_, err := s.FrontendClient().CreateSchedule(s.newContext(), &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: fmt.Sprintf(sid, i),
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
})
|
|
s.NoError(err)
|
|
s.cleanup(fmt.Sprintf(sid, i))
|
|
}
|
|
|
|
time.Sleep(5 * time.Second) //nolint:forbidigo
|
|
|
|
// With no rate limit, we'd see 10/second == 50 workflows run. With a limit of 1/sec, we
|
|
// expect to see around 5.
|
|
s.Less(atomic.LoadInt32(&runs), int32(10))
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) TestListSchedulesReturnsWorkflowStatus() {
|
|
sid := "sched-test-list-running"
|
|
wid := "sched-test-list-running-wf"
|
|
wt := "sched-test-list-running-wt"
|
|
|
|
// Set up a schedule that immediately starts a single running workflow
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(3 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
patch := &schedulepb.SchedulePatch{
|
|
TriggerImmediately: &schedulepb.TriggerImmediatelyRequest{},
|
|
}
|
|
|
|
// The workflow sits open until we've asserted it can be listed as running
|
|
resumeSignal := "resume"
|
|
workflowFn := func(ctx workflow.Context) error {
|
|
selector := workflow.NewSelector(ctx)
|
|
selector.AddReceive(workflow.GetSignalChannel(ctx, resumeSignal), func(c workflow.ReceiveChannel, more bool) {
|
|
// nothing to do
|
|
})
|
|
selector.Select(ctx)
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
InitialPatch: patch,
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
ctx := s.newContext()
|
|
_, err := s.FrontendClient().CreateSchedule(ctx, req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
// validate RecentActions made it to visibility
|
|
listResp := s.getScheduleEntryFomVisibility(sid, func(listResp *schedulepb.ScheduleListEntry) bool {
|
|
return len(listResp.Info.RecentActions) >= 1
|
|
})
|
|
s.Equal(1, len(listResp.Info.RecentActions))
|
|
|
|
a1 := listResp.Info.RecentActions[0]
|
|
s.True(strings.HasPrefix(a1.StartWorkflowResult.WorkflowId, wid))
|
|
s.Equal(enumspb.WORKFLOW_EXECUTION_STATUS_RUNNING, a1.StartWorkflowStatus)
|
|
|
|
// let the started workflow complete
|
|
_, err = s.FrontendClient().SignalWorkflowExecution(s.newContext(), &workflowservice.SignalWorkflowExecutionRequest{
|
|
Namespace: s.Namespace().String(),
|
|
WorkflowExecution: &commonpb.WorkflowExecution{
|
|
WorkflowId: a1.StartWorkflowResult.WorkflowId,
|
|
RunId: a1.StartWorkflowResult.RunId,
|
|
},
|
|
SignalName: resumeSignal,
|
|
})
|
|
s.NoError(err)
|
|
|
|
// now wait for second recent action to land in visbility
|
|
listResp = s.getScheduleEntryFomVisibility(sid, func(listResp *schedulepb.ScheduleListEntry) bool {
|
|
return len(listResp.Info.RecentActions) >= 2
|
|
})
|
|
|
|
a1 = listResp.Info.RecentActions[0]
|
|
a2 := listResp.Info.RecentActions[1]
|
|
s.Equal(enumspb.WORKFLOW_EXECUTION_STATUS_COMPLETED, a1.StartWorkflowStatus)
|
|
s.Equal(enumspb.WORKFLOW_EXECUTION_STATUS_RUNNING, a2.StartWorkflowStatus)
|
|
|
|
// Also verify that DescribeSchedule's output matches
|
|
descResp, err := s.FrontendClient().DescribeSchedule(s.newContext(), &workflowservice.DescribeScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
})
|
|
s.NoError(err)
|
|
s.assertSameRecentActions(descResp, listResp)
|
|
}
|
|
|
|
// A schedule's memo should have an upper bound on the number of spec items stored.
|
|
func (s *scheduleFunctionalSuiteBase) TestLimitMemoSpecSize() {
|
|
expectedLimit := scheduler.CurrentTweakablePolicies.SpecFieldLengthLimit
|
|
|
|
sid := "sched-test-limit-memo-size"
|
|
wid := "sched-test-limit-memo-size-wf"
|
|
wt := "sched-test-limit-memo-size-wt"
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
|
|
// Set up a schedule with a large number of spec items that should be trimmed in
|
|
// the memo block.
|
|
for i := 0; i < expectedLimit*2; i++ {
|
|
schedule.Spec.Interval = append(schedule.Spec.Interval, &schedulepb.IntervalSpec{
|
|
Interval: durationpb.New(time.Duration(i+1) * time.Second),
|
|
})
|
|
schedule.Spec.StructuredCalendar = append(schedule.Spec.StructuredCalendar, &schedulepb.StructuredCalendarSpec{
|
|
Minute: []*schedulepb.Range{
|
|
{
|
|
Start: int32(i + 1),
|
|
End: int32(i + 1),
|
|
},
|
|
},
|
|
})
|
|
schedule.Spec.ExcludeStructuredCalendar = append(schedule.Spec.ExcludeStructuredCalendar, &schedulepb.StructuredCalendarSpec{
|
|
Second: []*schedulepb.Range{
|
|
{
|
|
Start: int32(i + 1),
|
|
End: int32(i + 1),
|
|
},
|
|
},
|
|
})
|
|
}
|
|
|
|
// Create the schedule.
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(
|
|
func(ctx workflow.Context) error { return nil },
|
|
workflow.RegisterOptions{Name: wt},
|
|
)
|
|
ctx := s.newContext()
|
|
_, err := s.FrontendClient().CreateSchedule(ctx, req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
// Verify the memo field length limit was enforced.
|
|
entry := s.getScheduleEntryFomVisibility(sid, nil)
|
|
s.Require().NotNil(entry)
|
|
spec := entry.GetInfo().GetSpec()
|
|
s.Require().Equal(expectedLimit, len(spec.GetInterval()))
|
|
s.Require().Equal(expectedLimit, len(spec.GetStructuredCalendar()))
|
|
s.Require().Equal(expectedLimit, len(spec.GetExcludeStructuredCalendar()))
|
|
}
|
|
|
|
// TestNextTimeCache only applies to V1.
|
|
func (s *ScheduleV1FunctionalSuite) TestNextTimeCache() {
|
|
sid := "sched-test-next-time-cache"
|
|
wid := "sched-test-next-time-cache-wf"
|
|
wt := "sched-test-next-time-cache-wt"
|
|
|
|
schedule := &schedulepb.Schedule{
|
|
Spec: &schedulepb.ScheduleSpec{
|
|
Interval: []*schedulepb.IntervalSpec{
|
|
{Interval: durationpb.New(1 * time.Second)},
|
|
},
|
|
},
|
|
Action: &schedulepb.ScheduleAction{
|
|
Action: &schedulepb.ScheduleAction_StartWorkflow{
|
|
StartWorkflow: &workflowpb.NewWorkflowExecutionInfo{
|
|
WorkflowId: wid,
|
|
WorkflowType: &commonpb.WorkflowType{Name: wt},
|
|
TaskQueue: &taskqueuepb.TaskQueue{Name: s.taskQueue, Kind: enumspb.TASK_QUEUE_KIND_NORMAL},
|
|
},
|
|
},
|
|
},
|
|
}
|
|
req := &workflowservice.CreateScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Schedule: schedule,
|
|
Identity: "test",
|
|
RequestId: uuid.NewString(),
|
|
}
|
|
|
|
var runs atomic.Int32
|
|
workflowFn := func(ctx workflow.Context) error {
|
|
workflow.SideEffect(ctx, func(ctx workflow.Context) any {
|
|
runs.Add(1)
|
|
return 0
|
|
})
|
|
return nil
|
|
}
|
|
s.worker.RegisterWorkflowWithOptions(workflowFn, workflow.RegisterOptions{Name: wt})
|
|
|
|
_, err := s.FrontendClient().CreateSchedule(s.newContext(), req)
|
|
s.NoError(err)
|
|
s.cleanup(sid)
|
|
|
|
// wait for at least 13 runs
|
|
const count = 13
|
|
s.Eventually(func() bool { return runs.Load() >= count }, (count+10)*time.Second, 500*time.Millisecond)
|
|
|
|
// there should be only four side effects for 13 runs, and only two mentioning "Next"
|
|
// (cache refills)
|
|
events := s.GetHistory(s.Namespace().String(), &commonpb.WorkflowExecution{WorkflowId: scheduler.WorkflowIDPrefix + sid})
|
|
var sideEffects, nextTimeSideEffects int
|
|
for _, e := range events {
|
|
if marker := e.GetMarkerRecordedEventAttributes(); marker.GetMarkerName() == "SideEffect" {
|
|
sideEffects++
|
|
if p, ok := marker.Details["data"]; ok && len(p.Payloads) == 1 {
|
|
if string(p.Payloads[0].Metadata["messageType"]) == "temporal.server.api.schedule.v1.NextTimeCache" ||
|
|
strings.Contains(payloads.ToString(p), `"Next"`) {
|
|
nextTimeSideEffects++
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
const (
|
|
// These match the ones in the scheduler workflow, but they're not exported.
|
|
// Change these if those change.
|
|
FutureActionCountForList = 5
|
|
NextTimeCacheV2Size = 14
|
|
|
|
// Calculate expected results
|
|
expectedCacheSize = NextTimeCacheV2Size - FutureActionCountForList + 1
|
|
expectedRefills = (count + expectedCacheSize - 1) / expectedCacheSize
|
|
uuidCacheRefills = (count + 9) / 10
|
|
)
|
|
s.Equal(expectedRefills+uuidCacheRefills, sideEffects)
|
|
s.Equal(expectedRefills, nextTimeSideEffects)
|
|
}
|
|
|
|
// getScheduleEntryFomVisibility polls visibility using ListSchedules until it finds a schedule
|
|
// with the given id and for which the optional predicate function returns true.
|
|
func (s *scheduleFunctionalSuiteBase) getScheduleEntryFomVisibility(sid string, predicate func(*schedulepb.ScheduleListEntry) bool) *schedulepb.ScheduleListEntry {
|
|
var slEntry *schedulepb.ScheduleListEntry
|
|
s.Require().Eventually(func() bool { // wait for visibility
|
|
listResp, err := s.FrontendClient().ListSchedules(s.newContext(), &workflowservice.ListSchedulesRequest{
|
|
Namespace: s.Namespace().String(),
|
|
MaximumPageSize: 5,
|
|
})
|
|
if err != nil {
|
|
return false
|
|
}
|
|
for _, ent := range listResp.Schedules {
|
|
if ent.ScheduleId == sid {
|
|
if predicate != nil && !predicate(ent) {
|
|
return false
|
|
}
|
|
slEntry = ent
|
|
return true
|
|
}
|
|
}
|
|
return false
|
|
}, 15*time.Second, 1*time.Second)
|
|
return slEntry
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) assertSameRecentActions(
|
|
expected *workflowservice.DescribeScheduleResponse, actual *schedulepb.ScheduleListEntry,
|
|
) {
|
|
s.T().Helper()
|
|
if len(expected.Info.RecentActions) != len(actual.Info.RecentActions) {
|
|
s.T().Fatalf(
|
|
"RecentActions have different length expected %d, got %d",
|
|
len(expected.Info.RecentActions),
|
|
len(actual.Info.RecentActions))
|
|
}
|
|
for i := range expected.Info.RecentActions {
|
|
if !proto.Equal(expected.Info.RecentActions[i], actual.Info.RecentActions[i]) {
|
|
s.T().Errorf(
|
|
"RecentActions are differ at index %d. Expected %v, got %v",
|
|
i,
|
|
expected.Info.RecentActions[i],
|
|
actual.Info.RecentActions[i],
|
|
)
|
|
}
|
|
}
|
|
}
|
|
|
|
func (s *scheduleFunctionalSuiteBase) cleanup(sid string) {
|
|
s.T().Cleanup(func() {
|
|
_, _ = s.FrontendClient().DeleteSchedule(s.newContext(), &workflowservice.DeleteScheduleRequest{
|
|
Namespace: s.Namespace().String(),
|
|
ScheduleId: sid,
|
|
Identity: "test",
|
|
})
|
|
})
|
|
}
|