Skip to content

Ensure that writing to /etc/passwd is no longer required#181

Merged
zlogic merged 3 commits intolatestfrom
fix-random-uids
Oct 18, 2023
Merged

Ensure that writing to /etc/passwd is no longer required#181
zlogic merged 3 commits intolatestfrom
fix-random-uids

Conversation

@zlogic
Copy link
Copy Markdown
Member

@zlogic zlogic commented Sep 28, 2023

  • OpenShift 4 automatically updates /etc/passwd for randomly generated UIDs
    • But something caused CF Buildpack's getpwuid() to fail, most likely something to do with UNIX permissions
    • With default, out-of-the-box permissions for /etc/passwd, OpenShift 4 works correctly - so removed the code to make /etc/passwd writable, as it's not needed anymore.
  • Updated to CF Buildpack v5.0.9

OpenShift already correctly adds an entry to /etc/passwd.
Perhaps the issue was caused by incorrect permissions.
@zlogic zlogic marked this pull request as ready for review October 9, 2023 10:53
@zlogic zlogic merged commit f67107e into latest Oct 18, 2023
@zlogic zlogic deleted the fix-random-uids branch October 30, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants