-
Notifications
You must be signed in to change notification settings - Fork 62
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
Renovate image contains files with extraneous UID and GID #258
Comments
I'm pretty sure these are somehow being set by |
this is probably caused by npm v9, which comes with latest node lts |
Is this something that could be addressed in the Renovate image? I found localstack/localstack#7626 with a similar problem and it does appear to be caused by npm v9 not changing the file permissions any more. |
we can add a workaround inside containerbase install tool but in general it's not a renovate fault. the workaround will also slowdown dynamic installs. 😕 |
Can this be opted into or out of via an env variable? i.e. only do the extra work and slow down in environments where it matters |
should be possible, please open an issue at containerbase repo to track. |
FYI. I opened this issue with npm and it seems there's a fix in the works: npm/cli#6173 |
This has been fixed in npm 9.5.1. Before 9.5.1root@6a51ad3b2ca1:/# npm -v9.5.0 @6a51ad3b2ca1:/tmp/test-install# npm install [email protected] added 2 packages, and audited 3 packages in 652ms found 0 vulnerabilities @6a51ad3b2ca1:/tmp/test-install# ls -al node_modules/tmp/ total 32 With 9.5.1root@6a51ad3b2ca1:/tmp/test-install# npm -v9.5.1 root@6a51ad3b2ca1:/tmp/test-install# npm install [email protected] added 2 packages, and audited 3 packages in 425ms found 0 vulnerabilities root@6a51ad3b2ca1:/tmp/test-install# ls -al node_modules/tmp/ |
Verified this is no longer an issue in 3.1.0. |
The following files in the Renovate image have a very large UID (2119470584) and GID (2042662593). This is preventing us from being able to build a docker image with this one as the parent in docker rootless mode due to limited subordinate IDs. We only started seeing this issue in the latest release (3.0.0). 2.6.0 did not have this issue.
The text was updated successfully, but these errors were encountered: