get_info and list_instances to be made async

Bug #686772 reported by Armando Migliaccio
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Won't Fix
Undecided
Armando Migliaccio

Bug Description

I can see from the code in virt/libvirt_conn.py that most of the LibvirtConnection's methods are asynchronous. However list_instances and get_info are not.

Based on Ewan's comment at revision 229 (http://bazaar.launchpad.net/~ewanmellor/nova/xenapi-concurrency-model/revision/229) it seems that these methods were meant to be made async at some point. It is not a bug per se, but it becomes an issue when compute->get_info compute->and list_instances are backed by xenapi.

Changed in nova:
assignee: nobody → Armando Migliaccio (armando-migliaccio)
Revision history for this message
Soren Hansen (soren) wrote :

We're supposed to move away from Twisted, so nothing gets to be async anymore.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Thanks for the heads-up! Can you fill me in? Is there a etherpad page that describes thia, and how the move is going to impact current code?

Revision history for this message
Eric Day (eday) wrote : Re: [Bug 686772] Re: get_info and list_instances to be made async

Well, things still need to be async, they just do it in a different
way. :)

On Tue, Dec 07, 2010 at 09:53:58PM -0000, Soren Hansen wrote:
> We're supposed to move away from Twisted, so nothing gets to be async
> anymore.
>
> --
> You received this bug notification because you are a member of Nova Bug
> Team, which is subscribed to OpenStack Compute (nova).
> https://bugs.launchpad.net/bugs/686772
>
> Title:
> get_info and list_instances to be made async
>
> Status in OpenStack Compute (Nova):
> New
>
> Bug description:
> I can see from the code in virt/libvirt_conn.py that most of the LibvirtConnection's methods are asynchronous. However list_instances and get_info are not.
>
> Based on Ewan's comment at revision 229 (http://bazaar.launchpad.net/~ewanmellor/nova/xenapi-concurrency-model/revision/229) it seems that these methods were meant to be made async at some point. It is not a bug per se, but it becomes an issue when compute->get_info compute->and list_instances are backed by xenapi.
>

Revision history for this message
Eric Day (eday) wrote :

https://blueprints.launchpad.net/nova/+spec/unified-service-architecture

On Tue, Dec 07, 2010 at 09:58:18PM -0000, Armando Migliaccio wrote:
> Thanks for the heads-up! Can you fill me in? Is there a etherpad page
> that describes thia, and how the move is going to impact current code?
>
> --
> You received this bug notification because you are a member of Nova Bug
> Team, which is subscribed to OpenStack Compute (nova).
> https://bugs.launchpad.net/bugs/686772
>
> Title:
> get_info and list_instances to be made async
>
> Status in OpenStack Compute (Nova):
> New
>
> Bug description:
> I can see from the code in virt/libvirt_conn.py that most of the LibvirtConnection's methods are asynchronous. However list_instances and get_info are not.
>
> Based on Ewan's comment at revision 229 (http://bazaar.launchpad.net/~ewanmellor/nova/xenapi-concurrency-model/revision/229) it seems that these methods were meant to be made async at some point. It is not a bug per se, but it becomes an issue when compute->get_info compute->and list_instances are backed by xenapi.
>

Changed in nova:
status: New → Confirmed
status: Confirmed → New
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

I was trying to do a bit of housekeeping and wanted to mark this bug 'Won't fix' but it is greyed out (like Triaged). Is that because 'Importance' has not been set yet?

Revision history for this message
Soren Hansen (soren) wrote :

Maybe you don't have privileges to do it. Hmmm...

I just set it up Won't Fix for you for now.

Changed in nova:
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.