Re: Review Request 52074: switching from launchTask to acceptOffers

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52074/#review149604 --- Master (4c4040f) is red with this patch. ./build-support/jenkins

Review Request 52074: switching from launchTask to acceptOffers

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52074/ --- Review request for Aurora, Maxim Khutornenko, Mehrdad Nurolahzade, and Zameer Ma

Re: Review Request 51876: Modify executor state transition logic to rely on health checks (if enabled)

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51876/#review149588 --- Ship it! Master (4c4040f) is green with this patch. ./build-s

Re: Review Request 51876: Modify executor state transition logic to rely on health checks (if enabled)

2016-09-19 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51876/#review149581 --- src/main/python/apache/aurora/executor/common/status_checker.py (

Re: Review Request 51876: Modify executor state transition logic to rely on health checks (if enabled)

2016-09-19 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51876/ --- (Updated Sept. 20, 2016, 12:25 a.m.) Review request for Aurora, Joshua Cohen, M

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149547 --- Ship it! Ship It! - Zameer Manji On Sept. 19, 2016, 11:28 a.

Re: Review Request 52051: Add a release note about full task filesystem isolation.

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52051/#review149540 --- Ship it! Master (4c4040f) is green with this patch. ./build-s

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149539 --- Ship it! Master (4c4040f) is green with this patch. ./build-s

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149530 --- @ReviewBot retry - Dmitriy Shirchenko On Sept. 19, 2016, 6:28 p

Re: Review Request 52049: Clarifying documentation for new contributors by adding a step for them to ask for their JIRA id to get whitelisted

2016-09-19 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52049/#review149528 --- Ship it! Ship It! - Jake Farrell On Sept. 19, 2016, 6:42 p.m

Re: Review Request 52051: Add a release note about full task filesystem isolation.

2016-09-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52051/#review149526 --- Ship it! Ship It! - Zameer Manji On Sept. 19, 2016, 12:04 p.

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149527 --- Master (330e8ee) is red with this patch. ./build-support/jenkins

Review Request 52051: Add a release note about full task filesystem isolation.

2016-09-19 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52051/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description --

Re: Review Request 52046: Adding gpg key for jco...@apache.org

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52046/#review149521 --- Master (330e8ee) is red with this patch. ./build-support/jenkins

Re: Review Request 52046: Adding gpg key for jco...@apache.org

2016-09-19 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52046/#review149520 --- Ship it! Ship It! - Jake Farrell On Sept. 19, 2016, 6:28 p.m

Re: Review Request 52049: Clarifying documentation for new contributors by adding a step for them to ask for their JIRA id to get whitelisted

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52049/ --- (Updated Sept. 19, 2016, 6:42 p.m.) Review request for Aurora, Joshua Cohen, Ja

Re: Review Request 52049: Clarifying documentation for new contributors by adding a step for them to ask for their JIRA id to get whitelisted

2016-09-19 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52049/#review149516 --- Ship it! CONTRIBUTING.md (lines 19 - 20)

Re: Review Request 52049: Clarifying documentation for new contributors by adding a step for them to ask for their JIRA id to get whitelisted

2016-09-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52049/#review149515 --- Ship it! Ship It! - Zameer Manji On Sept. 19, 2016, 11:38 a.

Review Request 52049: Clarifying documentation for new contributors by adding a step for them to ask for their JIRA id to get whitelisted

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52049/ --- Review request for Aurora, Joshua Cohen, Jake Farrell, and Zameer Manji. Reposi

Review Request 52046: Adding gpg key for jco...@apache.org

2016-09-19 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52046/ --- Review request for Aurora and Jake Farrell. Repository: aurora Description --

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/ --- (Updated Sept. 19, 2016, 6:28 p.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149507 --- src/main/java/org/apache/aurora/scheduler/events/Webhook.java (li

Re: Review Request 51993: Added the 'reason' to the /pendingTasks endpoint

2016-09-19 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51993/#review149487 --- src/main/java/org/apache/aurora/scheduler/scheduling/TaskGroups.j