<div dir="ltr"><div dir="ltr">I tested a code change that essentially reverts <a href="https://review.openstack.org/#/c/276861/1/nova/api/metadata/base.py">https://review.openstack.org/#/c/276861/1/nova/api/metadata/base.py</a></div><div dir="ltr"><br></div><div>In other words, with this change metadata tables are not fetched by default in API requests. If I understand correctly, metadata is fetched in separate queries as the instance object is created. Everything seems to work just fine, and I've considerably reduced the amount of data fetched from the database, as well as reduced the average response time of API requests.</div><div><br></div><div>Given how simple it is and the results I'm getting, I don't see any reason not to patch my clusters with this change.</div><div><br></div><div>Do you guys see any other impact this change could have? Anything that it could potentially break?</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 22, 2018 at 10:05 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"><br><div><a href="https://bugs.launchpad.net/nova/+bug/1799298" target="_blank">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" target="_blank">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>
</blockquote></div>