Add docker group IDs for boot2docker and Docker for Mac Beta

[l0rd]

This is related to #1482. When running Che server inside a container, adding user to the local docker group (gid=999) is not enough. User should be added to the Docker host group too.

And in Docker for Mac Beta VM docker gid is 50, in boot2docker 100.

It worked accidentally on boot2docker because container user group and Docker host docker group had the same gid. But it did not work on Docker for Mac Beta.

Signed-off-by: Mario Loriedo mario.loriedo@gmail.com

[codenvy-ci]

Can one of the admins verify this patch?

[TylerJewell]

lgtm

@l0rd - can you confirm you have a CLA on file with Eclipse foundation? I have to assume you do. But the GitHub / Eclipse integration regularly seems to report users as not having a valid CLA. We can ignore it if you do.

[benoitf]

https://dev.eclipse.org/site_login/web-api/cla_decorator.php?email=mario.loriedo@gmail.com

(no checkmark)

[l0rd]

I'll do that as soon as eclipse servers get back online. I can't log in right now.

[l0rd]

done 😃

[TylerJewell]

@eivantsov @riuvshin - please review.

[garagatyi]

Looks like #1576 has part of changes from this PR. Why do you need both?

[l0rd]

That's a rebase issue. I've rebased that commit on top #1576 to generate
the Docker image. I'll fix that.

On Friday, 24 June 2016, Alexander Garagatyi notifications@github.com
wrote:

Looks like #1576 #1576 has part of
changes from this PR. Why do you need both?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1490 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAlC73jvGYDMaKS8juATMjjeTMzWATERks5qPAnvgaJpZM4I0WxS
.

[l0rd]

@garagatyi should be ok now

[garagatyi]

LGTM