Open Stack

Hi all,

rpc module is not imported properly in nova, cinder, neutron projects. It is imported from wrong package.
In oslo-incubator  'rpc' module is used in openstack/common/service.py file and it is present at openstack/common package.
(https://github.com/openstack/oslo-incubator/blob/master/openstack/common/service.py#L48<https://github.com/openstack/oslo-incubator/blob/master/openstack/common/service.py%23L48>)

But this 'rpc' module is present at base package in 'nova' and 'cinder' project while it is present at neutron/common/ package in neutron project.

Nova : https://github.com/openstack/nova/blob/master/nova/rpc.py
Cinder : https://github.com/openstack/cinder/blob/master/cinder/rpc.py
Neutron : https://github.com/openstack/neutron/blob/master/neutron/common/rpc.py

This openstack/common/service.py is synced form oslo-incubator in each project. Because of this if we make change in specific project then these changes will  get removed after re-synced  oslo-incubator code.
The same thing happened in nova project. This patch (https://review.openstack.org/#/c/81833/<https://review.openstack.org/%23/c/81833/>) has merged into nova code, but it is overwritten after syncing oslo-incubator code. There is comment on this patch by 'Mark McLoughlin' regarding the same.

I have filed bug for this issue in oslo : https://bugs.launchpad.net/oslo/+bug/1334661
And also I have pushed patch for same. But this patch will fail for 'Neutron' project.

I think we have to try importing 'rpc' module from all possible places till it gets imported properly
OR we need to change location of 'rpc' module in projects for uniformity. (ie to put 'rpc' module at some common place)

Could you please give me your opinions on the same.

Thanks,
Abhijeet

______________________________________________________________________
Disclaimer:This email and any attachments are sent in strictest confidence for the sole use of the addressee and may contain legally privileged, confidential, and proprietary data.  If you are not the intended recipient, please advise the sender by replying promptly to this email and then delete and destroy this email and any attachments without any further use, copying or forwarding
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140804/99d51f6b/attachment.html>