> On Aug. 5, 2015, 7:55 p.m., Maxim Khutornenko wrote:
> > examples/vagrant/provision-dev-cluster.sh, line 99
> > <https://reviews.apache.org/r/37136/diff/1/?file=1032812#file1032812line99>
> >
> >     Don't you want "groupadd docker" and "newgrp docker" here as well?

The docker group is already created when docker installed.  `newgrp` is only 
necessary if you want to affect the current session.

man newgrp
```
The newgrp command is used to change the current group ID during a login 
session.
```


- Bill


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37136/#review94294
-----------------------------------------------------------


On Aug. 5, 2015, 7:05 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37136/
> -----------------------------------------------------------
> 
> (Updated Aug. 5, 2015, 7:05 p.m.)
> 
> 
> Review request for Aurora and Maxim Khutornenko.
> 
> 
> Bugs: AURORA-1430
>     https://issues.apache.org/jira/browse/AURORA-1430
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Configure vagrant environment for sudo-less docker.
> 
> 
> Diffs
> -----
> 
>   examples/vagrant/provision-dev-cluster.sh 
> 18d388362b494017439245925080903d18a1424e 
> 
> Diff: https://reviews.apache.org/r/37136/diff/
> 
> 
> Testing
> -------
> 
> ```
> $ vagrant destroy -f
> $ vagrant up
> $ vagrant ssh -c 'docker ps'
> ```
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Reply via email to