Open Stack

Manjeet, 

Tony has some issues moving model classes to other location. Given that some class models are used by other neutron services, Ihar suggest to use debtcollector to make this transition smoothly.  Can we include that solution as part of this movement?

Thanks
Victor Morales



On 7/28/16, 12:19 PM, "Bhatia, Manjeet S" <manjeet.s.bhatia at intel.com> wrote:

>
>Ihar Hrachyshka <ihrachys at redhat.com> wrote:
>> Manjeet S <manjeet.s.bhatia at intel.com> wrote:
>> 
>>> Hello Team,
>>>
>>> I have a question regarding centralizing all db models in neutron. As 
>>> you all know Oslo versioned object work is under progress and I also 
>>> had a ticket opened for refactoring Db models. 
>>> (https://bugs.launchpad.net/neutron/+bug/1597913). There are three 
>>> way I can do this, 1, move all models to db/models_v2.py 2, create a 
>>> new dir db/models/ and move whatever models are giving issue Of 
>>> cyclic import to db_models.py under db/models/ tree but all in same 
>>> file, 3rd is move into different files under Same tree db/models. I 
>>> liked second way better, please let me know which one according to 
>>> experienced developers is better, I’ll do that way.
>> 
>> I don’t think 2. is the best way forward because it still keeps all 
>> models in a single file with no classification. I prefer we split 
>> models by topic, so option 3.
>> 
>> I took the approach for security groups here:  
>> https://review.openstack.org/#/c/284738/49/neutron/db/models/securityg
>> roup.py
>
>>I also prefer this organization (option 3).
>
>Ok thanks will follow 3.
>
>
>__________________________________________________________________________
>OpenStack Development Mailing List (not for usage questions)
>Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>__________________________________________________________________________
>OpenStack Development Mailing List (not for usage questions)
>Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev