purge_deleted doesn't handle prev_raw_template_id

Bug #1506079 reported by Thomas Herve
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Thomas Herve

Bug Description

We added storage of previous template in the prev_raw_template_id column of the stack db for convergence. We also use it in the non-convergence case.

Unfortunately we delete templates which are not referenced by raw_template_id in purge_deleted. We should check this column as well.

Revision history for this message
Thomas Herve (therve) wrote :

It was the other way around, we delete the ones even on non-deleted stack.

description: updated
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/235132

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

Reviewed: https://review.openstack.org/235132
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=78d5c425d0a5573de187c83705f4b6c034fc441d
Submitter: Jenkins
Branch: master

commit 78d5c425d0a5573de187c83705f4b6c034fc441d
Author: Thomas Herve <email address hidden>
Date: Thu Oct 15 09:31:30 2015 +0200

    Fix purging of raw_template rows

    Templates referenced in the prev_raw_template_id column of the stack
    table are not filtered by the purge_deleted script and end up being
    removed. This fixes it by making sure that we consider them when
    filtering.

    Change-Id: I8bab657dfe77b7cef5971e370f6b4f9825a22ba8
    Closes-Bug: #1506079

Changed in heat:
status: In Progress → Fix Committed
tags: added: liberty-backport-potential
Changed in heat:
importance: Low → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/242288

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

Reviewed: https://review.openstack.org/242288
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=3000b92004ec2744fc10c84c410b2d4fbb8902e6
Submitter: Jenkins
Branch: stable/liberty

commit 3000b92004ec2744fc10c84c410b2d4fbb8902e6
Author: Thomas Herve <email address hidden>
Date: Thu Oct 15 09:31:30 2015 +0200

    Fix purging of raw_template rows

    Templates referenced in the prev_raw_template_id column of the stack
    table are not filtered by the purge_deleted script and end up being
    removed. This fixes it by making sure that we consider them when
    filtering.

    Change-Id: I8bab657dfe77b7cef5971e370f6b4f9825a22ba8
    Closes-Bug: #1506079
    (cherry picked from commit 78d5c425d0a5573de187c83705f4b6c034fc441d)

tags: added: in-stable-liberty
tags: removed: liberty-backport-potential
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/heat 6.0.0.0b1

This issue was fixed in the openstack/heat 6.0.0.0b1 development milestone.

Changed in heat:
status: Fix Committed → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/heat 5.0.1

This issue was fixed in the openstack/heat 5.0.1 release.

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.