[NEW PATCH] Related to BZ#688616: do not stop cgroup when vdsmd start (via gerrit-bot)

[Dan Kenigsberg]

New patch submitted by Dan Kenigsberg (danken(a)redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/750

commit 06dbebcb1500980b963c7758b71d4ad9ee58e13a
Author: Dan Kenigsberg <danken(a)redhat.com&gt;
Date:   Wed Jul 27 12:52:51 2011 +0300

    Related to BZ#688616: do not stop cgroup when vdsmd start
    
    Oopsy, forgot to revert that one...
    
    Change-Id: I5e709e68e1bcdc3bec0a258ec6344ad30c89c0d3

diff --git a/vdsm/vdsmd b/vdsm/vdsmd
index 0591cb2..46c0f49 100755
--- a/vdsm/vdsmd
+++ b/vdsm/vdsmd
@@ -32,9 +32,6 @@ DOM_METADATA_BACKUP_DIR=/var/log/vdsm/backup
 CORE_PATTERN=/proc/sys/kernel/core_pattern
 NEEDED_SERVICES="iscsid multipathd"
 CONFLICTING_SERVICES="libvirt-guests libvirtd"
-# TODO: Remove cpu cgroup disabling when BZ#623712 is resolved (kernel on
-# massively multicore platforms failing to scale with cgroups turned on)
-CONFLICTING_SERVICES="cgconfig $CONFLICTING_SERVICES"
 
 LCONF=/etc/libvirt/libvirtd.conf
 QCONF=/etc/libvirt/qemu.conf

Change in vdsm[master]: Related to BZ#688616: do not stop cgroup when vdsmd start

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: Related to BZ#688616: do not stop cgroup when vdsmd start
......................................................................


Related to BZ#688616: do not stop cgroup when vdsmd start

Oopsy, forgot to revert that one...

Change-Id: I5e709e68e1bcdc3bec0a258ec6344ad30c89c0d3
---
M vdsm/vdsmd
1 file changed, 0 insertions(+), 3 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Yotam Oron: Looks good to me, but someone else must approve
  Haim Ateya: Verified


--
To view, visit http://gerrit.usersys.redhat.com/750
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5e709e68e1bcdc3bec0a258ec6344ad30c89c0d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Haim Ateya <hateya(a)redhat.com&gt;
Gerrit-Reviewer: Yotam Oron <yoron(a)redhat.com&gt;

Change in vdsm[master]: Related to BZ#688616: do not stop cgroup when vdsmd start

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: Related to BZ#688616: do not stop cgroup when vdsmd start
......................................................................


Patch Set 1: Looks good to me, approved

--
To view, visit http://gerrit.usersys.redhat.com/750
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5e709e68e1bcdc3bec0a258ec6344ad30c89c0d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Haim Ateya <hateya(a)redhat.com&gt;
Gerrit-Reviewer: Yotam Oron <yoron(a)redhat.com&gt;

Change in vdsm[master]: Related to BZ#688616: do not stop cgroup when vdsmd start

[Haim Ateya]

Haim Ateya has posted comments on this change.

Change subject: Related to BZ#688616: do not stop cgroup when vdsmd start
......................................................................


Patch Set 1: Verified

ack. vdsm not disables cgroups on start up.

--
To view, visit http://gerrit.usersys.redhat.com/750
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5e709e68e1bcdc3bec0a258ec6344ad30c89c0d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Haim Ateya <hateya(a)redhat.com&gt;
Gerrit-Reviewer: Yotam Oron <yoron(a)redhat.com&gt;

Change in vdsm[master]: Related to BZ#688616: do not stop cgroup when vdsmd start

[Yotam Oron]

Yotam Oron has posted comments on this change.

Change subject: Related to BZ#688616: do not stop cgroup when vdsmd start
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

--
To view, visit http://gerrit.usersys.redhat.com/750
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5e709e68e1bcdc3bec0a258ec6344ad30c89c0d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Haim Ateya <hateya(a)redhat.com&gt;
Gerrit-Reviewer: Yotam Oron <yoron(a)redhat.com&gt;