<p dir="ltr">Thanks for sharing this Ihar!</p>
<p dir="ltr">Thanks<br>
Vikram</p>
<div class="gmail_quote">On Sep 3, 2015 2:13 AM, "Carl Baldwin" <<a href="mailto:carl@ecbaldwin.net">carl@ecbaldwin.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks, I learned a thing or two from the document that you linked.<br>
Thanks for reminding us of that.<br>
<br>
Carl<br>
<br>
On Tue, Sep 1, 2015 at 3:14 AM, Ihar Hrachyshka <<a href="mailto:ihrachys@redhat.com">ihrachys@redhat.com</a>> wrote:<br>
> Hi reviewers,<br>
><br>
> several days ago, a semantically expand-only migration script was merged into contract branch [1]. This is not a disaster, though it would be a tiny one if a contract-only migration script would be merged into expand branch.<br>
><br>
> Please make sure you know the new migration strategy described in [2].<br>
><br>
> Previously, we introduced a check that validates that we don’t mix down_revision heads, linking e.g. expand script to contract revision, or vice versa [3]. Apparently, it’s not enough.<br>
><br>
> Ann is looking into introducing another check for semantical correctness of scripts. I don’t believe it may work for all complex cases we may need to solve manually, but at least it should be able to catch add_* operations in contract scripts, or drop_* operations in expand branch. Since there may be exceptions to general automation, we may also need a mechanism to disable such a sanity check for specific scripts.<br>
><br>
> So all in all, I kindly ask everyone to become aware of how we now manage migration scripts, and what it implies in how we should review code (f.e. looking at paths as well as the code of alembic scripts). That is especially important before the test that Ann is looking to implement is not merged.<br>
><br>
> [1]: <a href="https://bugs.launchpad.net/neutron/+bug/1490767" rel="noreferrer" target="_blank">https://bugs.launchpad.net/neutron/+bug/1490767</a><br>
> [2]: <a href="http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html" rel="noreferrer" target="_blank">http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html</a><br>
> [3]: <a href="https://review.openstack.org/#/c/206746/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/206746/</a><br>
><br>
> Thanks<br>
> Ihar<br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>