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

fix duplicate exit checker, add longer initial task wait #1412

Merged
merged 3 commits into from Feb 16, 2017

Conversation

Projects
None yet
2 participants
@wsorenson
Member

wsorenson commented Feb 3, 2017

Prevents an extra exit checker enrollment which will fail (and prevent normal shutdown)

Adds a new initial task wait that is longer than task wait.

Note that the executor already did the right thing WRT to locks/concurrency when the task is submitted after a shutdown has started, it was the extra exit checker enrollment which could cause a failure to propagate the TASK_LOST.

@wsorenson wsorenson added the hs_staging label Feb 3, 2017

wsorenson added some commits Feb 3, 2017

@wsorenson wsorenson added the hs_qa label Feb 6, 2017

@ssalinas ssalinas modified the milestone: 0.14.0 Feb 9, 2017

@ssalinas

This comment has been minimized.

Show comment
Hide comment
@ssalinas

ssalinas Feb 13, 2017

Member

🚢

Member

ssalinas commented on 44eb96a Feb 13, 2017

🚢

@ssalinas

This comment has been minimized.

Show comment
Hide comment
@ssalinas

ssalinas Feb 13, 2017

Member

🚢

Member

ssalinas commented on 6837db1 Feb 13, 2017

🚢

@ssalinas

This comment has been minimized.

Show comment
Hide comment
@ssalinas

ssalinas Feb 13, 2017

Member

🚢

Member

ssalinas commented on b33c75d Feb 13, 2017

🚢

@ssalinas ssalinas added the hs_stable label Feb 13, 2017

@ssalinas ssalinas merged commit c9900f2 into master Feb 16, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@ssalinas ssalinas deleted the executor_slow_launch branch Feb 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment