Loadbalancer in autoscalinggroup delete members incorrectly

Bug #1310602 reported by Zhang Yang
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Zane Bitter
Juno
Fix Released
Medium
Dirk Mueller

Bug Description

some members in loadbalancer couldn't be deleted, because when deleting a loadbalancer, the loadbalancer resource want to get all members from tmplate, which is incorrect in autoscaling.
the same bug is https://bugs.launchpad.net/heat/+bug/1248218, but the patch just fix the bug in update.

Changed in heat:
assignee: nobody → Zhang Yang (neil-zhangyang)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to heat (master)

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

Changed in heat:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to heat (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/104061

Zane Bitter (zaneb)
Changed in heat:
importance: Undecided → Medium
milestone: none → kilo-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Changed in heat:
assignee: Zhang Yang (neil-zhangyang) → Zane Bitter (zaneb)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/128729
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=3f33130b89b96c6fcc1dace12498f691413e4713
Submitter: Jenkins
Branch: master

commit 3f33130b89b96c6fcc1dace12498f691413e4713
Author: Zane Bitter <email address hidden>
Date: Wed Oct 15 12:08:13 2014 -0400

    Ensure all Neutron LoadBalancer members are deleted

    Previously we only deleted the ones listed in the template, even though
    members can be added by autoscaling that don't appear in the template. In
    theory this is OK if we're deleting the autoscaling group too, and since
    it's not possible to update the LoadBalancer property on an autoscaling
    group I can't find a way to actually trigger any bug (hence no new test).
    However, it's much safer to ensure we delete all of the remaining members
    when we delete the LoadBalancer resource.

    Change-Id: I6f1c02ae408e9169a23ae32e7fce5ea0d2239add
    Closes-Bug: #1310602

Changed in heat:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in heat:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/juno)

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/151179

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/juno)

Reviewed: https://review.openstack.org/151179
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=42b032f8bd3680f3bdfff78fe6152d6243549d54
Submitter: Jenkins
Branch: stable/juno

commit 42b032f8bd3680f3bdfff78fe6152d6243549d54
Author: Zane Bitter <email address hidden>
Date: Wed Oct 15 12:08:13 2014 -0400

    Ensure all Neutron LoadBalancer members are deleted

    Previously we only deleted the ones listed in the template, even though
    members can be added by autoscaling that don't appear in the template. In
    theory this is OK if we're deleting the autoscaling group too, and since
    it's not possible to update the LoadBalancer property on an autoscaling
    group I can't find a way to actually trigger any bug (hence no new test).
    However, it's much safer to ensure we delete all of the remaining members
    when we delete the LoadBalancer resource.

    Change-Id: I6f1c02ae408e9169a23ae32e7fce5ea0d2239add
    Closes-Bug: #1310602
    (cherry picked from commit 3f33130b89b96c6fcc1dace12498f691413e4713)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on heat (master)

Change abandoned by Angus Salkeld (<email address hidden>) on branch: master
Review: https://review.openstack.org/89359
Reason: adandoning old reviews, feel free to repost this if you want to get it merged.

Thierry Carrez (ttx)
Changed in heat:
milestone: kilo-1 → 2015.1.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Qiming Teng (<email address hidden>) on branch: master
Review: https://review.openstack.org/104061
Reason: no activity over 18 months.

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.