Bug 1001304 - Remove obsolete ControlGroup option from beah service file
Summary: Remove obsolete ControlGroup option from beah service file
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: beah
Version: 0.14
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 0.14.4
Assignee: Amit Saha
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-26 20:54 UTC by Bill Peck
Modified: 2018-02-06 00:41 UTC (History)
12 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-12-19 05:09:09 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 773259 0 high CLOSED tests don't run in root cpu cgroup with systemd 2021-02-22 00:41:40 UTC

Internal Links: 773259

Description Bill Peck 2013-08-26 20:54:50 UTC
Description of problem:
beah-srv.service file has ControlGroup in the service file which systemd does not support any more.

Version-Release number of selected component (if applicable):
0.6.45-1.fc17

How reproducible:
Everytime on Fedora20 or Rawhide

Additional info:
Please see the following article for background
https://lwn.net/Articles/555922/

Comment 2 Nick Coghlan 2013-08-27 03:08:28 UTC
This sounds like it needs to be filed as a bug against systemd, not against Beaker. Silently breaking every unit file that contains a CGroup definition instead of emitting a warning that the definition is being ignored is extraordinarily user hostile.

Comment 3 Nick Coghlan 2013-08-27 03:41:23 UTC
Then again...

Bill, Amit tried starting beah on the next-to-latest rawhide systemd (systemd-206-6.fc20.x86_64) and couldn't reproduce the report failure. It appeared to start fine for him:

=========================================
# systemctl status beah-srv
beah-srv.service - The Beaker Harness server.
   Loaded: loaded (/usr/lib/systemd/system/beah-srv.service; disabled)
   Active: active (running) since Mon 2013-08-26 21:21:25 MDT; 7s ago
 Main PID: 7373 (beah-srv)
   CGroup: /system.slice/beah-srv.service
           └─7373 /usr/bin/python /usr/bin/beah-srv
 
Aug 26 21:21:25 localhost.localdomain systemd[1]: Starting The Beaker Harness server....
Aug 26 21:21:25 localhost.localdomain systemd[1]: Started The Beaker Harness server..
[root@localhost yum.repos.d]#
=========================================

Starting it as shown also created /sys/fs/cgroup/systemd/system.slice/beah-srv.service/ as expected.

What are the failure symptoms you're seeing?

Independent of any direct impact on beah, this systemd change may have flow on effects to the use cases that led to the ControlGroup setting being added in bug 773259 in the first place.

Comment 5 Bill Peck 2013-08-27 13:17:52 UTC
Hi Nick,

It looks like there is just a significant delay before systemd tries to launch beah.  I am doing this in my home lab so maybe I don't have time services configured properly and chrony is timing out?

In any event, this is what I see in the journal on a fresh install of Fedora-20-20130827 fedora x86_64

Aug 27 13:00:17 localhost.localdomain systemd[1]: [/usr/lib/systemd/system/beah-srv.service:4] Failed to add dependency on beah-beaker-backend, ignoring: Invalid argument
Aug 27 13:00:17 localhost.localdomain systemd[1]: [/usr/lib/systemd/system/beah-srv.service:4] Failed to add dependency on beah-fwd-backend, ignoring: Invalid argument
Aug 27 13:00:17 localhost.localdomain systemd[1]: [/usr/lib/systemd/system/beah-srv.service:9] Unknown lvalue 'ControlGroup' in section 'Service'

So I don't think this is a blocker but I do think we will need to remove this option from the services file eventually.

Thanks

Comment 6 Amit Saha 2013-11-20 05:33:52 UTC
http://gerrit.beaker-project.org/#/c/2515/

Comment 7 Amit Saha 2013-11-26 08:12:18 UTC
Verification tip:

Note that this affects RHEL 7 and Fedora systems only.

Messages such as "[/usr/lib/systemd/system/beah-srv.service:9] Unknown lvalue 'ControlGroup' in section 'Service'" should not be present for RHEL 7 in /var/log/messages.


Note You need to log in before you can comment on or make changes to this bug.