Re: Review Request 37136: Configure vagrant environment for sudo-less docker.

2015-08-05 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37136/#review94296 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 5, 2015, 7:05 p.m.

Re: Review Request 37136: Configure vagrant environment for sudo-less docker.

2015-08-05 Thread Bill Farner
> On Aug. 5, 2015, 7:55 p.m., Maxim Khutornenko wrote: > > examples/vagrant/provision-dev-cluster.sh, line 99 > > > > > > Don't you want "groupadd docker" and "newgrp docker" here as well? The docker group is already

Re: Review Request 37136: Configure vagrant environment for sudo-less docker.

2015-08-05 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37136/#review94294 --- examples/vagrant/provision-dev-cluster.sh (line 99)

Re: Review Request 37136: Configure vagrant environment for sudo-less docker.

2015-08-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37136/#review94291 --- Ship it! Master (ae6e857) is green with this patch. ./build-suppo