Re: Review Request 43027: Optionally enable setuid inside Docker containers

2016-01-30 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43027/#review117139 --- Master (de0029b) is green with this patch. ./build-support/jenki

Re: Review Request 34568: run tasks as user that match aurora-role

2016-01-30 Thread Benjamin Staffin
> On Jan. 9, 2016, 4:43 p.m., Benjamin Staffin wrote: > > Any chance of getting this merged? It might be a surprise to some users, > > since right now there's no requirement for matching role accounts to exist > > inside the docker images, but with the right documentation about how to > > ret

Re: Review Request 43027: Optionally enable setuid inside Docker containers

2016-01-30 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43027/#review117137 --- Tests for this are not written yet. - Benjamin Staffin On Jan.

Review Request 43027: Optionally enable setuid inside Docker containers

2016-01-30 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43027/ --- Review request for Aurora. Bugs: AURORA-1237 https://issues.apache.org/jira

Re: Review Request 42953: Improve --read-json to handle multi-job files

2016-01-30 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42953/#review117128 --- Ship it! Master (de0029b) is green with this patch. ./build-s

Re: Review Request 42953: Improve --read-json to handle multi-job files

2016-01-30 Thread Benjamin Staffin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42953/ --- (Updated Jan. 30, 2016, 3:08 p.m.) Review request for Aurora. Bugs: AURORA-15

Re: Review Request 42953: Improve --read-json to handle multi-job files

2016-01-30 Thread Benjamin Staffin
> On Jan. 29, 2016, 6:51 p.m., Bill Farner wrote: > > Can you add a test case for the new behavior? Also, an 0.13.0 NEWS entry > > to call out the new feature. done. - Benjamin --- This is an automatically generated e-mail. To reply,

Re: Review Request 43013: Move lifecycle documentation into separate file

2016-01-30 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43013/#review117121 --- Ship it! Master (de0029b) is green with this patch. ./build-s

Review Request 43013: Move lifecycle documentation into separate file

2016-01-30 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43013/ --- Review request for Aurora and John Sirois. Bugs: AURORA-1068, AURORA-1262, AURO