Re: [libvirt] [PATCH] cgroup: Drop resource partition from virSystemdMakeScopeName

2015-07-21 Thread Peter Krempa
On Tue, Jul 21, 2015 at 16:12:09 +0100, Daniel Berrange wrote: > On Mon, Jul 20, 2015 at 11:10:18AM +0200, Peter Krempa wrote: > > On Fri, Jul 17, 2015 at 15:40:31 +0100, Daniel Berrange wrote: > > > On Thu, Jul 16, 2015 at 04:18:08PM +0200, Peter Krempa wrote: ... > > The patch as-is removes the

Re: [libvirt] [PATCH] cgroup: Drop resource partition from virSystemdMakeScopeName

2015-07-21 Thread Daniel P. Berrange
On Mon, Jul 20, 2015 at 11:10:18AM +0200, Peter Krempa wrote: > On Fri, Jul 17, 2015 at 15:40:31 +0100, Daniel Berrange wrote: > > On Thu, Jul 16, 2015 at 04:18:08PM +0200, Peter Krempa wrote: > > > The scope name, even according to our docs is > > > "machine-$DRIVER\x2d$VMNAME.scope" virSystemdMak

Re: [libvirt] [PATCH] cgroup: Drop resource partition from virSystemdMakeScopeName

2015-07-20 Thread Peter Krempa
On Fri, Jul 17, 2015 at 15:40:31 +0100, Daniel Berrange wrote: > On Thu, Jul 16, 2015 at 04:18:08PM +0200, Peter Krempa wrote: > > The scope name, even according to our docs is > > "machine-$DRIVER\x2d$VMNAME.scope" virSystemdMakeScopeName would use the > > resource partition name instead of "machi

Re: [libvirt] [PATCH] cgroup: Drop resource partition from virSystemdMakeScopeName

2015-07-17 Thread Daniel P. Berrange
On Thu, Jul 16, 2015 at 04:18:08PM +0200, Peter Krempa wrote: > The scope name, even according to our docs is > "machine-$DRIVER\x2d$VMNAME.scope" virSystemdMakeScopeName would use the > resource partition name instead of "machine-" if it was specified thus > creating invalid scope paths. > > This

[libvirt] [PATCH] cgroup: Drop resource partition from virSystemdMakeScopeName

2015-07-16 Thread Peter Krempa
The scope name, even according to our docs is "machine-$DRIVER\x2d$VMNAME.scope" virSystemdMakeScopeName would use the resource partition name instead of "machine-" if it was specified thus creating invalid scope paths. This makes libvirt drop cgroups for a VM that uses custom resource partition u