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.

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

Instantly overdue tasks #859

Merged
merged 2 commits into from Jan 28, 2016
Merged

Instantly overdue tasks #859

merged 2 commits into from Jan 28, 2016

Conversation

@Calvinp
Copy link
Contributor

@Calvinp Calvinp commented Jan 25, 2016

Scheduled tasks are shown as overdue the second after they were supposed to start up.
That is fine for most use cases, but when a task is bounced the replacement tasks are scheduled to start right away. Since it takes them a few seconds to start up, they're instantly marked as overdue.
With this pull request, tasks will show as scheduled if they're scheduled to start up more than a minute in the future. They'll show as pending if they're scheduled to start within a minute of now. And they'll only show as overdue if they are scheduled to start more than a minute ago.

Calvinp added 2 commits Jan 25, 2016
…ut it won't show the task at all where it should show the task as "PENDING"
…ing for within 1 minute of now, and overdue for >1 minute in the past
@tpetr tpetr modified the milestone: 0.4.9 Jan 27, 2016
ssalinas added a commit that referenced this pull request Jan 28, 2016
Instantly overdue tasks
@ssalinas ssalinas merged commit 1557b7a into master Jan 28, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
@ssalinas ssalinas deleted the instantly_overdue_tasks branch Jan 28, 2016
@ssalinas ssalinas removed hs_qa labels Jan 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.