-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding delaystart a new workflow with 30 sec delay #85
Adding delaystart a new workflow with 30 sec delay #85
Conversation
Could you add description to the pr introducing the changes you made |
cmd/samples/README.md
Outdated
make <WORKFLOW NAME> | ||
|
||
Example: | ||
make delaystart |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I'd avoid adding an extra example here. make helloworld
seems sufficient to show how to build an individual workflow
@@ -0,0 +1,159 @@ | |||
[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not used. lets delete
Context: one customer was complaining about delay start didn't work.
But in CDNC-12162 we've had enough proof that it worked.
We don't have a delayed start workflow sample, need to have one.
The simple way to do it is to use the hello world workflow as a template to create a new workflow. Need to:
copy and paste the entire hello world workflow
change the workflow name so that it won't conflict with helloworld workflow
add a delay start option in workflowOptions
update the readme about how to run the new workflow
One example of delay start option can be added in here. In the attached picture it also shows how to add the option.
DelayStart: time.Minute * 5,