powervm driver needs to pass on unsupported aggregate operations

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

Bug Description

The powervm driver doesn't support aggregate host resource pools (currently the xenapi driver is the only one that does). The base compute driver class raises NotImplementedError on aggregate operations to the virt driver if the subclass doesn't pass on the method. The libvirt driver passes on the unsupported aggregate operations, the powervm driver should do the same.

This was seen in the logs from some tempest runs against the powervm driver:

http://paste.openstack.org/show/42631/

Tags: powervm
Matt Riedemann (mriedem)
Changed in nova:
assignee: nobody → Matt Riedemann (mriedem)
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/39297

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/39297
Committed: http://github.com/openstack/nova/commit/bb75291a37c910f38b1aaf6a9ca41dcf9eee7b44
Submitter: Jenkins
Branch: master

commit bb75291a37c910f38b1aaf6a9ca41dcf9eee7b44
Author: Matt Riedemann <email address hidden>
Date: Tue Jul 30 07:30:14 2013 -0700

    powervm: pass on unimplemented aggregate operations

    The powervm driver does not implement the add/remove/undo aggregate
    operations defined in the base ComputeDriver class which can cause
    issues when running Tempest with the powervm driver. This patch simply
    overrides the methods in the powervm driver and does a pass like in the
    libvirt driver.

    Fixes bug 1206528

    Change-Id: Ieac227f321339a4069b015303b6e713ab9fad7b7

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