Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 475194 - sys-apps/openrc: Filter the systemd cgroup from the current process
Summary: sys-apps/openrc: Filter the systemd cgroup from the current process
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: OpenRC Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 439098 463784 systemd-love
  Show dependency tree
 
Reported: 2013-06-29 10:09 UTC by Pacho Ramos
Modified: 2013-09-16 01:51 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pacho Ramos gentoo-dev 2013-06-29 10:09:52 UTC
As discussed in bug 463784, we need the following patch to get systemd cgroup handled properly in openrc side to let logind work:
https://github.com/Sabayon/systemd-love/blob/master/sys-apps/openrc/files/0001-Filter-the-systemd-cgroup-from-the-current-process.patch

Reproducible: Always
Comment 1 Fabio Erculiani (RETIRED) gentoo-dev 2013-06-30 17:41:47 UTC
Or this:
https://github.com/qnikst/openrc/commit/7ed2c619598caedb2502b77f921404c0abfe02df

Also see my comment there.
And yes, it works.
Comment 2 Alexander Vershilov (RETIRED) gentoo-dev 2013-07-01 05:26:43 UTC
patch is fixed and pull request is done.

https://github.com/OpenRC/openrc/pull/3

I can attach the code to the bug if needed.

Thanks for testing.
Comment 3 Pacho Ramos gentoo-dev 2013-09-12 20:25:01 UTC
How did this end finally? I guess not needed :/
Comment 4 Alexander Vershilov (RETIRED) gentoo-dev 2013-09-13 08:31:29 UTC
(In reply to Pacho Ramos from comment #3)
> How did this end finally? I guess not needed :/

This issue is fixed in openrc-0.12 by:

https://github.com/OpenRC/openrc/commit/992807de55bd9b0e8cbb569a86bb84b75fe9482e
https://github.com/OpenRC/openrc/commit/2391ddff772ce541bc2237c89a2792a0b85e076c

both are commits are redundant and I think we don't need second commit, as first solves this issue without hardcoding. 

We may provide more general solution but I don't think we need it at this moment.
Comment 5 Pacho Ramos gentoo-dev 2013-09-13 16:22:46 UTC
OK then :)