<div dir="ltr"><div>I mean the case:</div><div>- an image has long property AAA and non inheritable property BBB;</div><div>- an instance based on the image stores truncated AAA and full BBB;</div><div>- utils.get_image_from_system_metadata returns truncated AAA but not BBB;</div><div>- compute.utils.get_image_metadata updates the result of utils.get_image_from_system_metadata by the image metadata and returns full AAA and added BBB.</div><div><br></div><div>This is the difference i mean.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 28, 2015 at 11:59 AM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@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 Thu, May 28, 2015 at 11:55:59AM +0300, Feodor Tersin wrote:<br>
> +1<br>
> But we should answer the following question before switching libvirt to<br>
> utils.get_image_from_system_metadata.<br>
><br>
> Some differences of utils.get_image_from_system_metadata from<br>
> compute.utils.get_image_metadata are:<br>
> 1) Instance metadata has a restriction of property length: long data is<br>
> stripped to 255 chars.<br>
> This may cause a fault on loading serialized structure from a metadata<br>
> property.<br>
><br>
> 2) utils.get_image_from_system_metadata skips properties defined by<br>
> CONF.non_inheritable_image_properties<br>
><br>
> So the question is: are these facts important for libvirt?<br>
<br>
</span>compute.utils.get_image_metadata calls utils.get_image_from_system_metadata<br>
so those don't cause any difference in behaviour.<br>
<div class="HOEnZb"><div class="h5"><br>
Regards,<br>
Daniel<br>
--<br>
|: <a href="http://berrange.com" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/~danberr/" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</div></div></blockquote></div><br></div>