<div dir="ltr"><div dir="ltr"><br><div><a href="https://bugs.launchpad.net/nova/+bug/1799298">https://bugs.launchpad.net/nova/+bug/1799298</a><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 22, 2018 at 9:15 PM Sergio A. de Carvalho Jr. <<a href="mailto:scarvalhojr@gmail.com">scarvalhojr@gmail.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"><div dir="ltr"><div dir="ltr"><div dir="ltr">Cool, I'll open a bug then.<div><br></div><div>I was wondering if, before joining the metadata tables with the rest of instance data, we could do a UNION, since both tables are structurally identical.</div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 22, 2018 at 9:04 PM Dan Smith <<a href="mailto:dms@danplanet.com" target="_blank">dms@danplanet.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> Do you guys see an easy fix here?<br>
><br>
> Should I open a bug report?<br>
<br>
Definitely open a bug. IMHO, we should just make the single-instance<br>
load work like the multi ones, where we load the metadata separately if<br>
requested. We might be able to get away without sysmeta these days, but<br>
we needed it for the flavor details back when the join was added. But,<br>
user metadata is controllable by the user and definitely of interest in<br>
that code, so just dropping sysmeta from the explicit required_attrs<br>
isn't enough, IMHO.<br>
<br>
--Dan<br>
</blockquote></div>
</blockquote></div>