<div dir="ltr">When I initially spoke to the infra team regarding this problem, they suggested that "just fixing migrations" so that the job could run was not a real option.<div>I tend to agree with this statement.</div>
<div>However, I'm open to options for getting grenade going until the migration problem is solved. Ugly workarounds might be fine, as long as we won't be doing anything that a real deployer would never do.</div><div>
<br></div><div>Personally, I still think the best way for getting grenade to work is to ensure previos_rev and current_rev have the same configuration. For the havana/icehouse upgrade, this will mean that devstack for icehouse should not add the metering plugin. I and Jakub are overdue a discussion on whether this would be feasible or not.</div>
<div><br></div><div>Change 87935 is acceptable as a fix for that specific migration.</div><div>However it does not fix the general issue, where the root cause is that currently the state of the neutron database depends on configuration settings, and therefore migrations are idempotent as long as the plugin configuration is not changed, which is not the case.</div>
<div><br></div><div>Salvatore</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 22 April 2014 11:14, Jakub Libosvar <span dir="ltr"><<a href="mailto:libosvar@redhat.com" target="_blank">libosvar@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 04/22/2014 10:53 AM, Anna Kamyshnikova wrote:<br>
> Hello everyone!<br>
><br>
> I'm working on fixing bug 1307344. I found out solution that will fix<br>
> Grenade jobs and will work for online and offline migartions.<br>
> <a href="https://review.openstack.org/87935" target="_blank">https://review.openstack.org/87935</a> But I faced the problem that Metering<br>
> usage won't be fixed as we need to create 2 tables (meteringlabels,<br>
> meteringlabelrules). I tried to create both in patch set #7 but it won't<br>
> work for offline migration. In fact to fix Grenade it is enough to<br>
> create meteringlabels table, that is done in my change in the last patch<br>
> set #8. I want to ask reviewers to take a look at this change and<br>
> suggest something or approve it. I'm available on IRC(akamyshnikova) or<br>
> by email.<br>
><br>
> Regards<br>
> Ann<br>
</div></div>Hi Ann,<br>
<br>
Good suggestion how to get out of failing job but I don't think it<br>
should go to 33c3db036fe4_set_length_of_description_field_metering.py<br>
because this failure is grenade specific while the real issue is a fact<br>
that we're not able to add new service plugin to already deployed Neutron.<br>
<br>
I think the same workaround you proposed in the 87935 review should go<br>
to grenade itself (from-havana/upgrade-neutron script) just to have the<br>
job working on havana->icehouse upgrade. It's a bit of ugly workaround<br>
though but imho so far the best solution to reach stable job in a short<br>
time.<br>
<br>
Kuba<br>
<br>
><br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>