-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bower asking for allow-root option #4
Comments
Interesting, I haven't seen that before. I'm guessing you're using the script from the bin directory, that executes a script in the container that uses sudo to run as someone other than root. Based on this thread I'm guessing it's a relatively recent change. I'll take a look but I may just add |
@mkenney I'll try it out on a few different environments / operating systems. I do not recall this happening every time, in-fact I believe it only happened once. I'll see what I can find. But yeah maybe adding the --allow-root might be a viable option. Can you think of any side-effects from that? |
Worked without asking for the --allow-root. This was on a web server hosted at DigitalOcean. Pretty fresh setup of Ubuntu 16.04. Let me know if any other information is needed to be helpful. I am still pretty new to docker. |
Here is that same info from my OS X install (which is still complaining about --allow-root, just tested again, same output).
|
Just touching base, I haven't had a chance to dig in yet (I broke my wrist pretty badly so everything I type is with one hand) but I did try to replicate it on my CentOS dev server and it seemed to work fine, and I triggered a successful build in TravisCI (debian). The TravisCI build was for the master (debian) version, not the alpine version so that may be related. The alpine build stopped working recently (can't find the |
Ahh, sorry to hear about your wrist. That sounds like a major road block in production all around. Yeah, on my ubuntu droplet it worked without a problem. I am only seeing an issue on OS X currently. Applying the --allow-root is obviously no big deal to do and I haven't seen any side-effects from it but I'm not entirely sure how the container works so I can't really comment intelligently. Did you have anything in-mind for how to rework the /run-as-user script? |
Docker is a pain on my Mac but I'm testing it now. So far it has worked for me with the current hub image
So I'm at a loss. The way the container works is it mounts the current directory into the container when the I've tested on these environments:
Anyway, because it's using sudo to switch away from root to the dev user, I can't think of any negative side-effects to the |
OS X 10.10.5
Well... I am at a loss as well. Do you know how an we verify that our containers are the same? |
Here is my script as well. |
I haven't forgotten about this. I'm going to try look some more this weekend. I've also solved the |
@bretmette So I went ahead and added the --allow-root option in cf3ef60 I don't see any reason not to have it, but I wasn't able to reproduce the issue either. I made a few other changes too, but the biggest was switching the Let me know if it gives you any trouble! |
@mkenney Been swamped these past few days. Also regarding the switch to Debian. Did you mean to say you switched from Ubuntu to Debian? Looks like the two options now are Alpine and Debian, correct? I will pull down the Debian one and give it a go and provide feedback as needed. |
Hey @bretmette, No, the Debian image hasn't changed. I just pointed the Mainly because the |
@mkenney Oooooh. Gotcha! Sorry long week, brain is fried :) |
Not sure if I accidentally modified something or if this is a known issue. Have you seen this before? For now I am just executing with --allow-root and it is not effecting the permissions on the host. What are your thoughts @mkenney ?
The text was updated successfully, but these errors were encountered: