Re: Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-15 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46241/#review129144 --- Ship it! Ship It! - Zameer Manji On April 14, 2016, 6:01

Re: Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-15 Thread John Sirois
> On April 14, 2016, 7:08 p.m., John Sirois wrote: > > Looks like we do ~nothing with executorLost so LGTM. > > I can only imagine folks pagers going off more if they happened to alert on > > `scheduler_lost_executors`, which should now be higher if the upgrade notes > > are to be believed. >

Re: Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-15 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46241/#review129142 --- Ship it! Ship It! - Maxim Khutornenko On April 15, 2016,

Re: Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-15 Thread Joshua Cohen
> On April 15, 2016, 1:08 a.m., John Sirois wrote: > > Looks like we do ~nothing with executorLost so LGTM. > > I can only imagine folks pagers going off more if they happened to alert on > > `scheduler_lost_executors`, which should now be higher if the upgrade notes > > are to be believed.

Re: Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-14 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46241/#review129055 --- Ship it! Master (de046f7) is green with this patch.

Re: Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-14 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46241/#review129054 --- Ship it! Looks like we do ~nothing with executorLost so LGTM.

Review Request 46241: Upgrade to Mesos 0.27.2

2016-04-14 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46241/ --- Review request for Aurora, John Sirois and Zameer Manji. Repository: aurora