Open Stack

Anna,

Your second point is going to be a bit of a maintenance headache.  I
was reviewing a patch [1] where you caught the lack of an import in
head.py.  I'm not sure that I can be trusted to check consistently for
this in future reviews.  I'm not sure that we can rely on you catching
them all either.  Jenkins didn't seem to have any problem with the
patch without the import.

So, how can we guarantee good maintenance of this head.py file?

Carl

[1] https://review.openstack.org/#/c/102101/40/neutron/db/migration/models/head.py

On Wed, Jul 16, 2014 at 2:14 AM, Anna Kamyshnikova
<akamyshnikova at mirantis.com> wrote:
> Hello everyone!
>
> I would like to bring the next two points to everybody's attention:
>
> 1) As Henry mentioned if you add new migration you should make it
> unconditional. Conditional migrations should not be merged since now.
>
> 2) If you add some new models you should ensure that module containing it is
> imported in /neutron/db/migration/models/head.py.
>
> The second point in important for testing which I hope will be merged soon:
> https://review.openstack.org/76520.
>
> Regards,
> Ann
>
>
>
> On Wed, Jul 16, 2014 at 5:54 AM, Kyle Mestery <mestery at mestery.com> wrote:
>>
>> On Tue, Jul 15, 2014 at 5:49 PM, Henry Gessau <gessau at cisco.com> wrote:
>> > I am happy to announce that the first (zero'th?) item in the Neutron Gap
>> > Coverage[1] has merged[2]. The Neutron database now contains all tables
>> > for
>> > all plugins, and database migrations are no longer conditional on the
>> > configuration.
>> >
>> > In the short term, Neutron developers who write migration scripts need
>> > to set
>> >   migration_for_plugins = ['*']
>> > but we will soon clean up the template for migration scripts so that
>> > this will
>> > be unnecessary.
>> >
>> > I would like to say special thanks to Ann Kamyshnikova and Jakub
>> > Libosvar for
>> > their great work on this solution. Also thanks to Salvatore Orlando and
>> > Mark
>> > McClain for mentoring this through to the finish.
>> >
>> > [1]
>> >
>> > https://wiki.openstack.org/wiki/Governance/TechnicalCommittee/Neutron_Gap_Coverage
>> > [2] https://review.openstack.org/96438
>> >
>> This is great news! Thanks to everyone who worked on this particular
>> gap. We're making progress on the other gaps identified in that plan,
>> I'll send an email out once Juno-2 closes with where we're at.
>>
>> Thanks,
>> Kyle
>>
>> > _______________________________________________
>> > OpenStack-dev mailing list
>> > OpenStack-dev at lists.openstack.org
>> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>