Skip to content
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

Docs: fix path to nologin shell #1610

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Docs: fix path to nologin shell #1610

merged 1 commit into from
Feb 15, 2024

Conversation

exsplashit
Copy link
Contributor

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

Just small fix to copy-paste tutorial deploy

@ohdearaugustin
Copy link
Collaborator

ohdearaugustin commented Feb 8, 2024

Several locations for nologin do exist:

  • /usr/bin/nologin
  • /usr/sbin/nologin
  • /sbin/nologin

This is mainly dependend on the OS you are using.

Arch: has all three
Ubuntu:

  • /usr/sbin/nologin
  • /sbin/nologin

Rockylinux (RedHat):

  • /usr/sbin/nologin
  • /sbin/nologin

So I guess the change to /usr/sbin/nologin should make it work on more systems out of the box.

@kradalby kradalby closed this Feb 15, 2024
@kradalby kradalby reopened this Feb 15, 2024
@kradalby kradalby merged commit 82c64f6 into juanfont:main Feb 15, 2024
50 of 51 checks passed
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.

3 participants