FS#58336 - [lxc] [lxcfs] cgmanager should not be a dependency.

Attached to Project: Community Packages
Opened by Edênis Freindorfer Azevedo (denisfa) - Tuesday, 24 April 2018, 16:30 GMT
Last edited by Doug Newgard (Scimmia) - Tuesday, 08 May 2018, 14:07 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Sergej Pupykin (sergej)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 3
Private No

Details

Description:

From the latest lxc update (3.0.0), cgmanager has been deprecated (https://brauner.github.io/2018/02/20/lxc-removes-legacy-cgroup-drivers.html).

cgmanager should be removed from being a dependency of this package.
This task depends upon

Closed by  Doug Newgard (Scimmia)
Tuesday, 08 May 2018, 14:07 GMT
Reason for closing:  Fixed
Comment by Vladimir Lebedev (zeronineseven) - Monday, 30 April 2018, 21:32 GMT
Moreover lxc used to rely on pam_cgm.so which was superseeded by pam_cgfs.so module. As of lxc 3.0 pam_cgfs.so is contained in lxc codebase instead of lxcfs one (https://brauner.github.io/2018/02/28/lxc-includes-cgroup-pam-module.html). I believe arch lxc package should now be configured with "--enable-pam" and "--with-pamdir=/usr/lib/security" in PKGBUILD because currently this options are not there and hence no pam integration and no unprivileged containers (without jumping through a lot of hoops).
Comment by Edênis Freindorfer Azevedo (denisfa) - Friday, 04 May 2018, 00:21 GMT
  • Field changed: Percent Complete (100% → 0%)
This is also a request for removing cgmanager from lxcfs. The same reason applies for it. In this case, It is needed to change the lxcfs.service to not include the cgmanager startup.
Comment by Vladimir Lebedev (zeronineseven) - Monday, 07 May 2018, 14:44 GMT
  • Field changed: Percent Complete (100% → 0%)
There is warning during makepkg:
configure: WARNING: unrecognized options: --enable-cgmanager, --enable-lua

As I've already mentioned, "--enable-cgmanager" should be replaced with "--enable-pam"
Comment by Sergej Pupykin (sergej) - Monday, 07 May 2018, 15:18 GMT
I removed --enable-cgmanager and --enable-lua in svn/trunk but did not rebuild the package.
Comment by Vladimir Lebedev (zeronineseven) - Tuesday, 08 May 2018, 08:40 GMT
I've successfully built from trunk - looks like everything is fine. But what about PAM-module? Will you also add "--enable-pam" and "--with-pamdir=/usr/lib/security"?
Comment by Sergej Pupykin (sergej) - Tuesday, 08 May 2018, 09:10 GMT
Current lxc package should be built with these options:

https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/lxc#n64

Loading...