Skip to content

fix: useradd in Running step-ca as a Daemon#405

Merged
tashian merged 2 commits into
smallstep:mainfrom
z63d:fix/running-step-ca-as-a-daemon
Jun 11, 2026
Merged

fix: useradd in Running step-ca as a Daemon#405
tashian merged 2 commits into
smallstep:mainfrom
z63d:fix/running-step-ca-as-a-daemon

Conversation

@z63d

@z63d z63d commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

Describe your changes:

The original useradd command does not create the /etc/step-ca directory.
sudo mv $(step path)/* /etc/step-ca gives an error.

@z63d z63d requested a review from a team as a code owner April 9, 2025 13:18

@tashian tashian left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Just a quick suggestion.

@@ -366,7 +366,7 @@ Note: _This section requires a Linux OS running `systemd` version 245 or greater
The service user will only be used by `systemd` to manage the CA. Run:

<CodeBlock language="shell-session" copytext="sudo useradd --user-group --system --home /etc/step-ca --shell /bin/false step">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<CodeBlock language="shell-session" copytext="sudo useradd --user-group --system --home /etc/step-ca --shell /bin/false step">
<CodeBlock language="shell-session" copytext="sudo useradd --user-group --system --create-home --home /etc/step-ca --shell /bin/false step">

Signed-off-by: Kaita Nakamura <kaita.nakamura0830@gmail.com>
@z63d z63d force-pushed the fix/running-step-ca-as-a-daemon branch from d35828e to 217c022 Compare June 18, 2025 23:10
@z63d z63d requested a review from tashian June 18, 2025 23:11
@CLAassistant

CLAassistant commented Oct 12, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@tashian tashian enabled auto-merge June 11, 2026 21:39
@tashian tashian merged commit e31796d into smallstep:main Jun 11, 2026
4 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