Fail to delete a floating IP address that doesn't exist anymore

Bug #1234593 reported by Simon Pasquier
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Xiang Hui

Bug Description

https://github.com/openstack/heat/blob/master/heat/engine/resources/eip.py#L121

There is no try/except to catch a clients.novaclient.exceptions.NotFound exception.

Steven Dake (sdake)
Changed in heat:
importance: Undecided → Low
milestone: none → icehouse-1
assignee: nobody → Steven Dake (sdake)
status: New → Triaged
Revision history for this message
Xiang Hui (xianghui) wrote :

Hi Steven, can I solve this bug? Cause I have committed a patch dup as this bug. https://bugs.launchpad.net/heat/+bug/1237271

Revision history for this message
Steven Dake (sdake) wrote :

Xiang,

Feel free to take the bug.

Changed in heat:
assignee: Steven Dake (sdake) → Xiang Hui (xianghui)
Revision history for this message
Xiang Hui (xianghui) wrote :

Thanks Steve.

Changed in heat:
status: Triaged → In Progress
Steven Hardy (shardy)
Changed in heat:
importance: Low → Medium
Revision history for this message
Steven Hardy (shardy) wrote :
tags: added: havana-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/50577
Committed: http://github.com/openstack/heat/commit/e1c4ab30dcd736e031d784308ae214837566c8cf
Submitter: Jenkins
Branch: master

commit e1c4ab30dcd736e031d784308ae214837566c8cf
Author: Hui HX Xiang <email address hidden>
Date: Wed Oct 9 03:10:36 2013 -0700

    Fix heat deletion failed if floating ip is not found

    Novaclient will throw out a Not Found exception if the
    floating ip not exists. Heat should catch this exception
    to continue to delete the stack.

    Closes-Bug: #1234593

    Change-Id: Ifd9d34de37cc5dd57804787d8b1455f0321fc86f

Changed in heat:
status: In Progress → Fix Committed
Steven Hardy (shardy)
Changed in heat:
milestone: icehouse-1 → havana-rc2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (milestone-proposed)

Fix proposed to branch: milestone-proposed
Review: https://review.openstack.org/51363

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (milestone-proposed)

Reviewed: https://review.openstack.org/51363
Committed: http://github.com/openstack/heat/commit/2aa4186dde97e580694b88ad22dad4db657cc5ad
Submitter: Jenkins
Branch: milestone-proposed

commit 2aa4186dde97e580694b88ad22dad4db657cc5ad
Author: Hui HX Xiang <email address hidden>
Date: Wed Oct 9 03:10:36 2013 -0700

    Fix heat deletion failed if floating ip is not found

    Novaclient will throw out a Not Found exception if the
    floating ip not exists. Heat should catch this exception
    to continue to delete the stack.

    Closes-Bug: #1234593

    Change-Id: Ifd9d34de37cc5dd57804787d8b1455f0321fc86f

Changed in heat:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: havana-rc2 → 2013.2
Steven Hardy (shardy)
tags: added: grizzly-backport-potential
Alan Pevec (apevec)
tags: removed: grizzly-backport-potential havana-rc-potential
no longer affects: heat/grizzly
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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