Re: Review Request 41528: Fixup `getJobSummary` for cron jobs with invalid next run dates.

2015-12-18 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41528/#review57 --- Ship it! Thanks for looking into this! - Stephan Erb On Dec.

Re: Review Request 41559: Fix the ZK URI in the rpm.

2015-12-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41559/#review111226 --- 1 more - README needs updating: ``` $ git grep "zk:"

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Maxim Khutornenko
> On Dec. 18, 2015, 8:11 p.m., Maxim Khutornenko wrote: > > docs/installing.md, line 58 > > > > > > This is very important part of our infrastructure. I'd add a bit more > > details here highlighting its importance

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/ --- (Updated Dec. 18, 2015, 12:38 p.m.) Review request for Aurora, Joshua Cohen,

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/#review111253 --- docs/installing.md (line 58)

Review Request 41565: Improve review_feedback.py testability.

2015-12-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41565/ --- Review request for Aurora and Bill Farner. Repository: aurora Description

Re: Review Request 41565: Improve review_feedback.py testability.

2015-12-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41565/#review111257 --- Ship it! Master (1c73438) is green with this patch.

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/#review111251 --- docs/installing.md (line 155)

Re: Review Request 41565: Improve review_feedback.py testability.

2015-12-18 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41565/#review111259 --- Ship it! - Bill Farner On Dec. 18, 2015, noon, John Sirois

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Bill Farner
> On Dec. 18, 2015, 12:11 p.m., Maxim Khutornenko wrote: > > docs/installing.md, line 58 > > > > > > This is very important part of our infrastructure. I'd add a bit more > > details here highlighting its importance

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Bill Farner
> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote: > > docs/installing.md, line 29 > > > > > > You are happily mixing the terms jobs and services here. Other sections > > of this document also use the word task. It

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/#review111322 --- Not as thorough a check of Ubuntu, but so far:

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/#review111296 --- The comments below are from a test of CentOS 7 instructions -

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread John Sirois
> On Dec. 18, 2015, 3:16 p.m., Stephan Erb wrote: > > docs/installing.md, line 146 > > > > > > Isn't that outdated? I think so - I used `sudo yum install -y mesos-0.23` for this step. - John

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/#review111270 --- Ship it! Master (5183fe0) is green with this patch.

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Stephan Erb
> On Dez. 18, 2015, 9:12 nachm., John Sirois wrote: > > > > John Sirois wrote: > I won't be able to get to trials until later this evening. Ideally > though this doc could be tied to the test/ provision.sh's somehow for more > assured working-ness. I have no useful suggestion though atm

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Bill Farner
> On Dec. 18, 2015, 4:43 p.m., John Sirois wrote: > > docs/installing.md, line 126 > > > > > > Had to s/ mesos// and then: > > ``` > > wget -c > >

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/#review111354 --- Ship it! Master (5183fe0) is green with this patch.

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Bill Farner
> On Dec. 18, 2015, 3:45 p.m., John Sirois wrote: > > docs/installing.md, line 111 > > > > > > Invalid URL - I updated to > > http://people.apache.org/~wfarner/aurora/distributions/0.10.0-1/rpm/centos-7/ > > to

Re: Review Request 41564: Add a guide for installing Aurora.

2015-12-18 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41564/ --- (Updated Dec. 18, 2015, 7:03 p.m.) Review request for Aurora, Joshua Cohen,