Parameter mismatch when calling _create_image() from spawn() in libvirt_conn.py

Bug #764236 reported by Yoshiaki Tamura
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Yoshiaki Tamura

Bug Description

The definition of _create_image() in nova/virt/libvirt_conn.py is:

def _create_image(self, inst, libvirt_xml, suffix='', disk_images=None, network_info=None):

However in spawn(), it's passing network_info to suffix:

def spawn(self, instance, network_info=None):
    ...
    self._create_image(instance, xml, network_info)

It should be fixed to:
self._create_image(instance, xml, network_info=network_info)

Related branches

Changed in nova:
assignee: nobody → Yoshiaki Tamura (tamura-yoshiaki)
status: New → In Progress
Revision history for this message
Yoshiaki Tamura (tamura-yoshiaki) wrote :

I've updated my branch that first introduces a test which checks spawn() with network_info specified, and followed by the fix.

Thierry Carrez (ttx)
Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → diablo-1
Thierry Carrez (ttx)
Changed in nova:
milestone: diablo-1 → 2011.3
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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