<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 15, 2016 at 10:01 AM, Michal Sekletar <span dir="ltr"><<a href="mailto:msekleta@redhat.com" target="_blank">msekleta@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Mar 15, 2016 at 5:42 PM, Mrunal Patel <<a href="mailto:mpatel@redhat.com">mpatel@redhat.com</a>> wrote:<br>
><br>
><br>
><br>
>> ><br>
>> > <a href="http://docker.io" rel="noreferrer" target="_blank">docker.io</a> did that too three weeks ago:<br>
>> ><br>
>> >   <a href="https://github.com/docker/docker/commit/65820132" rel="noreferrer" target="_blank">https://github.com/docker/docker/commit/65820132</a><br>
>><br>
>> This seems odd. I thought you primarily want to enable delegation for<br>
>> scope units wrapping processes which constitute your container and not<br>
>> for the container manager itself. For example, on Fedora 23<br>
>> libvirtd.service doesn't have Delegate=yes. However libvirtd registers<br>
>> VMs as machines in machined. And machined will take care for creating<br>
>> scope unit and it also sets Delegate=yes for the scope.<br>
>><br>
> Yes, this is the model we have. So, it looks like we should set Delegate=yes<br>
> for the scope. Last time I tried, I got some errors from the API.<br>
> I will try again and respond back on this thread.<br>
<br>
</span>You can have a look at our C implementation and re-implement it in Go.<br>
<br>
<a href="https://github.com/systemd/systemd/blob/master/src/machine/machined-dbus.c#L1277" rel="noreferrer" target="_blank">https://github.com/systemd/systemd/blob/master/src/machine/machined-dbus.c#L1277</a><br>
<br></blockquote><div>Thanks Michal,<br></div><div>I created a patch setting Delegate to true and it seems to be working. We will need to see how the new version behaves on Openshift builds where this problem was originally reported.<br><br></div></div><br></div></div>