nova.compute.manager calls methods in nova.volume.manager directly

[Nachi Ueno]

nova.compute.manager calls methods in nova.volume.manager directly.
This must be via api module.

146: self.volume_manager = utils.import_object(FLAGS.volume_manager)
1230: dev_path = self.volume_manager.setup_compute_volume(context, volume_id)
1241: dev_path = self.volume_manager.setup_compute_volume(context,
1267: self.volume_manager.remove_compute_volume(context,
1289: self.volume_manager.remove_compute_volume(context, volume_id)
1306: self.volume_manager.remove_compute_volume(context, volume_id)
1401: self.volume_manager.setup_compute_volume(context, v['id'])
1511: self.volume_manager.remove_compute_volume(ctxt, vol['id'])

[Nirmal Ranganathan (rnirmal)]

I think this just got merged in by Vishy in one of his volume cleanup branches

[Nachi Ueno (nati-ueno)]

>Nirmal Ranganathan
Thanks
I confirmed it is merged to Essex branch.
But I think it is also needed for Diablo Stable Branch.

https://github.com/openstack/nova/blob/2011.3/nova/compute/manager.py

[Nirmal Ranganathan (rnirmal)]

Ok cool.. yeah not sure what all was going to get backported to Diablo.

On Wed, Oct 12, 2011 at 3:09 PM, Nachi Ueno <email address hidden>wrote:

> >Nirmal Ranganathan
> Thanks
> I confirmed it is merged to Essex branch.
> But I think it is also needed for Diablo Stable Branch.
>
> https://github.com/openstack/nova/blob/2011.3/nova/compute/manager.py
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/872445
>
> Title:
> nova.compute.manager calls methods in nova.volume.manager directly
>
> Status in OpenStack Compute (Nova):
> New
> Status in OpenStack QA:
> New
>
> Bug description:
> nova.compute.manager calls methods in nova.volume.manager directly.
> This must be via api module.
>
> 146: self.volume_manager =
> utils.import_object(FLAGS.volume_manager)
> 1230: dev_path = self.volume_manager.setup_compute_volume(context,
> volume_id)
> 1241: dev_path = self.volume_manager.setup_compute_volume(context,
> 1267: self.volume_manager.remove_compute_volume(context,
> 1289: self.volume_manager.remove_compute_volume(context, volume_id)
> 1306: self.volume_manager.remove_compute_volume(context, volume_id)
> 1401: self.volume_manager.setup_compute_volume(context,
> v['id'])
> 1511: self.volume_manager.remove_compute_volume(ctxt,
> vol['id'])
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/nova/+bug/872445/+subscriptions
>