Instance object pci_devices and security_groups can't be None

Bug #1228402 reported by Chris Behrens
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Chris Behrens

Bug Description

The Instance object defines security_groups and pci_devices as being able to be None, however they should never be None. They are lists and should just be empty if assigned, never None.

The Instance object has code for pci_devices to account for the DB layer possibly returning None. However, it should never return None.

Chris Behrens (cbehrens)
Changed in nova:
status: New → In Progress
importance: Undecided → Low
summary: - Instance object pcI_devices and security_groups can't be None
+ Instance object pci_devices and security_groups can't be None
Chris Behrens (cbehrens)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/47647

Changed in nova:
assignee: nobody → Chris Behrens (cbehrens)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/47647
Committed: http://github.com/openstack/nova/commit/39d117904783ed5cc3137b4371c087f33077a553
Submitter: Jenkins
Branch: master

commit 39d117904783ed5cc3137b4371c087f33077a553
Author: Chris Behrens <email address hidden>
Date: Fri Sep 20 21:47:09 2013 +0000

    Don't allow pci_devices/security_groups to be None

    These should never be able to be None in the Instance object. They are
    lists and can be empty, however.

    Fixes bug 1228402

    Change-Id: Id250f8921926f906e858a09759bdab888bd597b0

Changed in nova:
status: In Progress → Fix Committed
Changed in nova:
milestone: none → havana-rc1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-rc1 → 2013.2
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.