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

Move systemd files from /etc to /usr/lib #4029

Closed
wants to merge 1 commit into from
Closed

Move systemd files from /etc to /usr/lib #4029

wants to merge 1 commit into from

Conversation

Dridi
Copy link
Contributor

@Dridi Dridi commented Oct 31, 2013

As documented in systemd's manual pages tmpfiles.d(5) and systemd.unit(5),
a package should install its default configuration in /usr/lib, which can
be overridden by system administrators in /etc.

New locations in the rpm:
/usr/lib/systemd/system/elasticsearch.service
/usr/lib/tmpfiles.d/elasticsearch.conf

The man pages are available online:
http://www.freedesktop.org/software/systemd/man/tmpfiles.d.html
http://www.freedesktop.org/software/systemd/man/systemd.unit.html#Unit%20Load%20Path

As documented in systemd's manual pages tmpfiles.d(5) and systemd.unit(5),
a package should install its default configuration in /usr/lib, which can
be overriden by system administrators in /etc.

New locations in the rpm:
/usr/lib/systemd/system/elasticsearch.service
/usr/lib/tmpfiles.d/elasticsearch.conf
@obourgain
Copy link
Contributor

+1

@ghost ghost assigned spinscale Jan 3, 2014
@Dridi
Copy link
Contributor Author

Dridi commented Jan 13, 2014

Hi,

In case you'd be wondering, I have signed the CLA before sending this PR and you don't need to be a systemd expert to merge it.

I've noticed a new systemd file in 81e13a8, and this time it lands in /usr/lib/sysctl.d as expected and can be overridden by sysadmins in /etc/sysctl.d. This is the same idea for this PR.

If you don't feel comfortable with systemd, I'll gladly answer your questions.

@drewr
Copy link
Contributor

drewr commented Mar 7, 2014

👍

@spinscale
Copy link
Contributor

I will get this in next week, just need to test with opensuse/redhat to make sure everything works (yes I know, it should work flawless)

@s1monw
Copy link
Contributor

s1monw commented Mar 17, 2014

LGTM

@s1monw
Copy link
Contributor

s1monw commented Mar 17, 2014

as discussed I guess this should go to all branches?

@spinscale
Copy link
Contributor

merged via 9500ddd

thanks for your patience!

@spinscale spinscale closed this Mar 17, 2014
@Dridi
Copy link
Contributor Author

Dridi commented Mar 17, 2014

Glad I could help, even with a trivial fix :)

@clintongormley clintongormley added the :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts label Jun 7, 2015
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >enhancement Team:Delivery Meta label for Delivery team v0.90.13 v1.0.2 v1.1.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants