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

Also check pending requests on the track task endpoint #1688

Merged
merged 1 commit into from Dec 22, 2017
Merged

Conversation

@ssalinas
Copy link
Member

@ssalinas ssalinas commented Dec 20, 2017

If a RunNowRequest was enqueued, but still in the pending request queue waiting to be processed by the scheduler, the track endpoint would return an absent. We should also be able to mark the task as pending when it is in the pending request queue. This adds an additional check of the pending requests

@ssalinas ssalinas added this to the 0.19.0 milestone Dec 21, 2017
@darcatron
Copy link
Contributor

@darcatron darcatron commented Dec 21, 2017

🚢

@ssalinas ssalinas added the hs_stable label Dec 21, 2017
@ssalinas ssalinas merged commit 5811850 into master Dec 22, 2017
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/push The Travis CI build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ssalinas ssalinas deleted the track_pending_req branch Dec 22, 2017
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

2 participants