Plug/unplug exceptions are a mess

Bug #1629556 reported by Eric Fried
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nova-powervm
Fix Released
Low
Eric Fried

Bug Description

Need some consistency in the VirtualInterface(Un)Plug exceptions in the vif driver.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova-powervm (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/380762

Revision history for this message
Eric Fried (efried) wrote :
Changed in nova-powervm:
assignee: nobody → Eric Fried (efried)
importance: Undecided → Low
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova-powervm (stable/newton)

Change abandoned by Eric Fried (<email address hidden>) on branch: stable/newton
Review: https://review.openstack.org/380762
Reason: In favor of proper cherry-pick: https://review.openstack.org/381836

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova-powervm (stable/newton)

Reviewed: https://review.openstack.org/381836
Committed: https://git.openstack.org/cgit/openstack/nova-powervm/commit/?id=e22a3e823e22657cc725d87a9464c4c41a9c396e
Submitter: Jenkins
Branch: stable/newton

commit e22a3e823e22657cc725d87a9464c4c41a9c396e
Author: Eric Fried <email address hidden>
Date: Fri Sep 2 08:35:20 2016 -0500

    HttpError=>VirtualInterface(Unp|P)lugException

    With this change set, the top-level vif driver plug/unplug methods trap
    HttpError (typically a 400 or 500) and convert it to the standard
    VirtualInterfacePlugException/VirtualInterfaceUnplugException from
    nova.exception.

    This change set also gets rid of the VirtualInterfaceUnplugException in
    vif.py and tasks/network.py, replacing their invocations with the
    corresponding nova.exception class.

    Conflicts:
     nova_powervm/tests/virt/powervm/test_vif.py

    Change-Id: Iad1197218cb8d8a260c00d55c9addd39ac3a6226
    Closes-Bug: #1629556

tags: added: in-stable-newton
Eric Fried (efried)
Changed in nova-powervm:
status: In Progress → 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.