Move Mattermost systemd unit file from /lib/systemd/system to /etc/systemd/system - #9129
Conversation
…stemd/system /lib/systemd/system is reserved for package-manager-owned unit files. Since our guides have the admin create this file by hand, /etc/systemd/system is the correct location. Updates the install guides and downstream references to the unit file path for consistency.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThe documentation now uses ChangesSystemd unit path documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Reviewed changes, looks great. Thanks for implementing. |
shiftZZnet
left a comment
There was a problem hiding this comment.
All looks fine, thanks for implementing, changes approved.
Summary
/lib/systemd/system(aliased to/usr/lib/systemd/systemon merged-usr systems) is reserved for unit files installed and owned by a package manager. Our tarball and RHEL install guides have the admin create the Mattermost unit file by hand, so/etc/systemd/systemis the correct location./lib/systemd/system./etc/systemd/systemfor this file, so this also removes an existing inconsistency rather than introducing one.