Review Request 55507: Update packaging scripts for Mesos 1.1.0

2017-01-13 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55507/ --- Review request for Aurora, John Sirois and Renan DelValle. Repository: aurora-p

Review Request 55507: Update packaging scripts for Mesos 1.1.0

2017-01-13 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55507/ --- Review request for Aurora, John Sirois and Renan DelValle. Repository: aurora-p

Re: Review Request 55507: Update packaging scripts for Mesos 1.1.0

2017-01-13 Thread Renan DelValle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55507/#review161559 --- Ship it! Ship It! - Renan DelValle On Jan. 13, 2017, 9:15 a.

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55471/#review161566 --- @ReviewBot retry - Mehrdad Nurolahzade On Jan. 12, 2017, 2:46 p

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55471/#review161571 --- Ship it! Master (06a221f) is green with this patch. ./build-s

Re: Review Request 55477: AURORA-1874 Expose stats on SlotSizeCounter runs

2017-01-13 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55477/#review161572 --- @ReviewBot retry - Mehrdad Nurolahzade On Jan. 12, 2017, 3:07 p

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55471/#review161573 --- Ship it! Ship It! - Zameer Manji On Jan. 12, 2017, 2:46 p.m.

Re: Review Request 55477: AURORA-1874 Expose stats on SlotSizeCounter runs

2017-01-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55477/#review161575 --- Ship it! Ship It! - Zameer Manji On Jan. 12, 2017, 3:07 p.m.

Re: Review Request 55477: AURORA-1874 Expose stats on SlotSizeCounter runs

2017-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55477/#review161576 --- Master (06a221f) is green with this patch. ./build-support/jenki

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55471/#review161577 --- Thinking about it, maybe we should expose this metric as a counter

Re: Review Request 54960: Exposed stats on number of offers rescinded and number of slaves lost.

2017-01-13 Thread Pradyumna Kaushik
> On Jan. 13, 2017, 1:12 a.m., Aurora ReviewBot wrote: > > Master (06a221f) is red with this patch. > > ./build-support/jenkins/build.sh > > > > thrift-0.9.1/tutorial/py/Makefile.am > > thrift-0.9.1/tutorial/py/Makefile.in > > thrift-0.9.1/tutorial/py/PythonClient.py > > thrift-0.9.1/tutorial/

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Zameer Manji
> On Jan. 13, 2017, 11:47 a.m., Mehrdad Nurolahzade wrote: > > Thinking about it, maybe we should expose this metric as a counter rather > > than time. Thoughts? What would it be counting exactly? - Zameer --- This is an automatically

Re: Review Request 54960: Exposed stats on number of offers rescinded and number of slaves lost.

2017-01-13 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54960/#review161581 --- @ReviewBot retry - Mehrdad Nurolahzade On Jan. 12, 2017, 5:02 p

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Mehrdad Nurolahzade
> On Jan. 13, 2017, 11:47 a.m., Mehrdad Nurolahzade wrote: > > Thinking about it, maybe we should expose this metric as a counter rather > > than time. Thoughts? > > Zameer Manji wrote: > What would it be counting exactly? Number of times sleep time has been > 0; with time it is not possib

Re: Review Request 55471: AURORA-1876 Expose stats on scheduler rate limiter blocking time

2017-01-13 Thread Zameer Manji
> On Jan. 13, 2017, 11:47 a.m., Mehrdad Nurolahzade wrote: > > Thinking about it, maybe we should expose this metric as a counter rather > > than time. Thoughts? > > Zameer Manji wrote: > What would it be counting exactly? > > Mehrdad Nurolahzade wrote: > Number of times sleep time has

Re: Review Request 55243: AURORA-1868 Evaluate multiple preemption proposals per round

2017-01-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55243/#review161586 --- Ship it! Ship It! - Zameer Manji On Jan. 5, 2017, 5:17 p.m.,

Re: Review Request 54960: Exposed stats on number of offers rescinded and number of slaves lost.

2017-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54960/#review161587 --- Ship it! Master (2645278) is green with this patch. ./build-s

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161590 --- Ship it! Ship It! - Zameer Manji On Jan. 11, 2017, 8:48 a.m.

Re: Review Request 54960: Exposed stats on number of offers rescinded and number of slaves lost.

2017-01-13 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54960/#review161591 --- Ship it! Ship It! - Zameer Manji On Jan. 13, 2017, 12:23 p.m

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-13 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161598 --- @ReviewBot retry - Stephan Erb On Jan. 11, 2017, 5:48 p.m., Meh

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-13 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review161603 --- Master (b75dfb8) is red with this patch. ./build-support/jenkins

Review Request 55510: Only include basenames in package checksums

2017-01-13 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55510/ --- Review request for Aurora and John Sirois. Bugs: AURORA-1815 https://issues

Re: Review Request 55507: Update packaging scripts for Mesos 1.1.0

2017-01-13 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55507/#review161620 --- Ship it! Ship It! - John Sirois On Jan. 13, 2017, 10:15 a.m.

Re: Review Request 55510: Only include basenames in package checksums

2017-01-13 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55510/#review161622 --- Ship it! Ship It! - John Sirois On Jan. 13, 2017, 4:25 p.m.,