<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 4/23/15 1:16 PM, Dan Smith wrote:<br>
    <blockquote cite="mid:55386418.7020404@danplanet.com" type="cite">
      <blockquote type="cite">
        <pre wrap="">If I selected all the instance_type_id's from the system-metadata table
and used those uuid's to load the object with something like:
        instance = objects.Instance.get_by_uuid(
            context, instance_uuid,
            expected_attrs=['system_metadata', 'flavor'])

The tests would fail at that point when trying to read in the flavor as
json. <a class="moz-txt-link-freetext" href="http://paste.openstack.org/show/205158/">http://paste.openstack.org/show/205158/</a>

Basically without digging further it seems like I should be able to load
an instance by uuid regardless of the state of my flavor(s). Since this
fails it seems like there is something wrong with the flavor handling on
the objects.
</pre>
      </blockquote>
      <pre wrap="">
You should. The above is a reasonable result to get without the fix to
ensure that we create instance_extra records for instances missing it.

Do you still see the above with

  <a class="moz-txt-link-freetext" href="https://review.openstack.org/#/c/176387/">https://review.openstack.org/#/c/176387/</a>

applied?</pre>
    </blockquote>
    <br>
    That change works on the dataset. However I was referring to the
    db/object api (which I have no real knowledge of) that it should be
    able to get_by_uuid unmigrated instances and in my case I got the
    traceback given in that paste. It's possible I'm just using the API
    incorrectly.<br>
    <br>
    <blockquote cite="mid:55386418.7020404@danplanet.com" type="cite">
      <pre wrap="">

</pre>
      <blockquote type="cite">
        <pre wrap="">Another interesting thing is that migrate_flavor_data avoids migrating
instances that are in the middle of an operation. The snapshot of
databases we have include instances in this state. Since turbo-hipster
is just testing against a snapshot in time there is no way for those
instances to leave their working state and hence migrate_flavor_data can
never finish (every run will leave instances undone). This therefore
blocks the migrations from ever finishing.
</pre>
      </blockquote>
      <pre wrap="">
Ah, yeah, that's interesting, but I think it's a restriction we have to
make for sanity.</pre>
    </blockquote>
    <br>
    Oh yes, we want that restriction, but a way around it for instances
    that may be stuck or just testing purposes could be useful.<br>
    <br>
    Cheers,<br>
    Josh<br>
    <br>
    <blockquote cite="mid:55386418.7020404@danplanet.com" type="cite">
      <pre wrap="">

--Dan

</pre>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: <a class="moz-txt-link-abbreviated" href="mailto:OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a>
<a class="moz-txt-link-freetext" href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>