Re: [libvirt] [PATCH 1/3] Change VM cgroup suffix from '{lxc, qemu}.libvirt' to 'libvirt-{lxc, qemu}'

From: "Daniel P. Berrange" <berrange(a)redhat.com&gt; Recently we changed to create VM cgroups with the naming pattern $VMNAME.$DRIVER.libvirt. Following discussions with the systemd community it was decided that only having a single '.' in the names is preferrable. So this changes the naming scheme to be $VMNAME.libvirt-$DRIVER. eg for LXC 'mycontainer.libvirt-lxc' or for KVM 'myvm.libvirt-qemu'.

Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com&gt; --- src/util/vircgroup.c | 2 +- tests/vircgrouptest.c | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-)

+++ b/tests/vircgrouptest.c @@ -362,13 +362,13 @@ static int testCgroupNewForPartitionDomain(const void *args ATTRIBUTE_UNUSED) int ret = -1; int rv; const char *placement[VIR_CGROUP_CONTROLLER_LAST] = { - [VIR_CGROUP_CONTROLLER_CPU] = "/production/foo.lxc.libvirt", - [VIR_CGROUP_CONTROLLER_CPUACCT] = "/production/foo.lxc.libvirt", - [VIR_CGROUP_CONTROLLER_CPUSET] = "/production/foo.lxc.libvirt", - [VIR_CGROUP_CONTROLLER_MEMORY] = "/production/foo.lxc.libvirt", + [VIR_CGROUP_CONTROLLER_CPU] = "/production/foo.libvirt-lxc",