-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Issue with non 'username' defined #2373
Comments
@beltran-rubo I have a fix for this on a local branch. Having gotten this a little bit wrong the first time (as proved by your bug report), I am trying very hard to arrive at a design that is rock solid and with good code patterns. That said, the simplest fix would be for Once I am happy with the design (I think that can be today) I will post it as a PR. But if posting it right away as a Work-In-Progress PR would be helpful, I can do that as well. |
@beltran-rubo here is the Pull Request with my proposed fix: #2375 |
Awesome thanks! |
Related to this, still an issue when the username is not defined, only the user ID e.g. that is common in 'non-root' containers for OpenShift https://docs.openshift.com/container-platform/3.3/creating_images/guidelines.html#openshift-container-platform-specific-guidelines.
Python is in the path so a good option could be to add a fallback or not return an error for the line
Originally posted by @beltran-rubo in #2333 (comment)
The text was updated successfully, but these errors were encountered: