<div dir="ltr">Morning<br><br><div class="gmail_quote"><div dir="ltr">On Thu, 27 Jul 2017 at 23:58 Michael Still <<a href="mailto:mikal@stillhq.com">mikal@stillhq.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>I'm cc'ing openstack-dev because your email is the same as the comment you made on the relevant review, and I think getting visibility with the wider Nova team is a good idea.<br><div><br></div><div>Unfortunately this is a risk of having an out of tree Nova driver, which has never been the recommended path for implementing drivers for Nova. Being out of tree isn't forbidden, but it does come with the cost of staying up to date with Nova and handling changes as they occur. </div></div></div></blockquote><div><br></div><div>Agreed - and we've felt this from time-to-time since the nova-lxd driver was started.</div><div><br></div><div>Maybe the Queens cycle might be a good time to review our out-of-tree-ness and see whether the Nova team would be prepared to accept the LXD driver in-tree.</div><div><br></div><div>I'm more comfortable now with where we are with regards to devstack tempest testing in the gate for the LXD driver, although we do still have some feature gaps compared to the libvirt driver (specifically boot from volume) which results in a large-ish skip list.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>In this case, if you look at the review chain you'll see that the move is a pre-cursor to moving this code to use oslo.privsep. Unless lxd is going to move to privsep in lockstep with nova, your best bet would be to duplicate this utility method in the nova-lxd codebase.</div></div></div></blockquote><div><br></div><div>We'll duplicate right now (see [0]) to unblock the current problem and take a look at the work to move lockstep with nova to use oslo.privsep.</div><div><br></div><div>Cheers</div><div><br></div><div>James </div><div><br></div><div>[0] <a href="https://review.openstack.org/#/c/488254/">https://review.openstack.org/#/c/488254/</a></div><div><br></div><div><br></div><div> </div><div><br></div><div> </div></div></div>