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
_______________________________________________
OpenStack-dev mailing list
[email protected]
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
