Skip to content
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.

Sign up for GitHub

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

Add optional shutdown delay to tasks #3043

Merged
merged 6 commits into from
Aug 17, 2017
Merged

Add optional shutdown delay to tasks #3043

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
beae45b
Add optional shutdown delay to tasks
schmichael Aug 17, 2017
ecf0d20
Make shutdown delay log DEBUG, not INFO
schmichael Aug 17, 2017
32d782d
Add ShutdownDelay to JSON Job docs
schmichael Aug 17, 2017
b4480cd
Lower shutdown delay in test
schmichael Aug 17, 2017
08d9c4b
Fix ShutdownDelay diff test
schmichael Aug 17, 2017
fbe85b5
Lower ShutdownDelay for non-Travis testing
schmichael Aug 17, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Lower shutdown delay in test
  • Loading branch information
@schmichael
schmichael committed Aug 17, 2017
commit b4480cdcffbbbfb968b7d987fc5045c47af48dc4
2 changes: 1 addition & 1 deletion client/task_runner_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1626,7 +1626,7 @@ func TestTaskRunner_ShutdownDelay(t *testing.T) {
}

// No shutdown escape hatch for this delay, so don't set it too high
task.ShutdownDelay = 5 * time.Second
task.ShutdownDelay = 2 * time.Second

ctx := testTaskRunnerFromAlloc(t, true, alloc)
ctx.tr.MarkReceived()
Expand Down