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

correction #561 #562

Merged
merged 1 commit into from
Nov 7, 2015
Merged

correction #561 #562

merged 1 commit into from
Nov 7, 2015

Conversation

rapidoo
Copy link

@rapidoo rapidoo commented Sep 28, 2015

Test on master

@kapil-malik
Copy link
Contributor

Hi,

Thanks @rapidoo for finding and fixing this. We were also puzzled as chronos seemed to honor constraints for some jobs and not for others. I found the same issue and then noticed your issue + PR.

Chronos admin(s), this patch is critical for our usecase (i..e. constraints must be honored in all cases), so can someone please review and merge this PR asap?
Further, any suggestions on 2.5 timelines ?

Thanks and regards,

@petard
Copy link

petard commented Nov 4, 2015

+1 on merging this.

florianleibert added a commit that referenced this pull request Nov 7, 2015
@florianleibert florianleibert merged commit d0c04cb into mesos:master Nov 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants