Skip to content

mongos: add RuntimeDirectory to systemd unit file#793

Open
schlitzered wants to merge 1 commit intovoxpupuli:masterfrom
schlitzered:fix_systemd_unit_file
Open

mongos: add RuntimeDirectory to systemd unit file#793
schlitzered wants to merge 1 commit intovoxpupuli:masterfrom
schlitzered:fix_systemd_unit_file

Conversation

@schlitzered
Copy link
Copy Markdown

This adds RuntimeDirectory and RuntimeDirectoryMode to the mongos systemd unit file template to ensure /run/mongodb is created with the correct permissions.

Also fixes a bug in the mongos service manifest where service_group was incorrectly passed as service_user to the unit file template.

@bastelfreak
Copy link
Copy Markdown
Member

@schlitzered thanks for the PR, can you please rebase and take a look at the CI failures?

@schlitzered
Copy link
Copy Markdown
Author

@schlitzered thanks for the PR, can you please rebase and take a look at the CI failures?

done

@bastelfreak
Copy link
Copy Markdown
Member

as said above, please rebase. a merge isn't a rebase :)

@smortex
Copy link
Copy Markdown
Member

smortex commented Mar 16, 2026

From your working directory, you can rebase and squash with:

git fetch origin            # Download the latest code we have here
git rebase -i origin/master # Interactively rewrite history moving your commits on top of the main branch

This will bring your editor listing all commits in your branch. Keep the first line unchanged, and replace pick with squash for all subsequent lines. Save and quit, your editor will pop again and allow you to combine all the commit messages in a single one.

Then push your updated branch:

git push -f            # Send the changes (-f is required because we re-wrote history)

@schlitzered schlitzered force-pushed the fix_systemd_unit_file branch from 028252e to ae7da99 Compare March 16, 2026 20:12
@schlitzered schlitzered force-pushed the fix_systemd_unit_file branch from ae7da99 to bb21611 Compare March 16, 2026 20:15
@schlitzered
Copy link
Copy Markdown
Author

as said above, please rebase. a merge isn't a rebase :)

not sure if i did the right thing now... but now there is only one commit, and i do not see any conflicts.

if not, please try to explain in detail what you expect, with the assumption in mind that i am a 3 year old that likes to eat clay ;)

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