Solum

add-check-to-delete-logs-from-swift

Bug #1578275 reported by Devdatta Kulkarni on 2016-05-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
​ Solum
Fix Released
Undecided
Devdatta Kulkarni
Also affects project (?) Also affects distribution/package Nominate for series

Bug Description

In heat.py when destroy_assembly is called, we are calling delete_app_artifacts_from_swift

https://github.com/openstack/solum/blob/master/solum/deployer/handlers/heat.py#L240

But we should call delete from swift only if we are using swift to store the logs. If we are using local filesystem, then we should delete the logs from the local fs.

Devdatta Kulkarni (devdatta-kulkarni) on 2016-05-04
Changed in solum:
assignee: nobody → Devdatta Kulkarni (devdatta-kulkarni)
OpenStack Infra (hudson-openstack) wrote : Fix proposed to solum (master)

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

Changed in solum:
status: New → In Progress
OpenStack Infra (hudson-openstack) wrote : Fix merged to solum (master)

Reviewed: https://review.openstack.org/312695
Committed: https://git.openstack.org/cgit/openstack/solum/commit/?id=95e5f97589fe02d2a21c4398670564f3d12a391d
Submitter: Jenkins
Branch: master

commit 95e5f97589fe02d2a21c4398670564f3d12a391d
Author: Devdatta Kulkarni <email address hidden>
Date: Wed May 4 14:02:06 2016 -0500

    Add checks to delete app artifacts from appropriate backends

    Change-Id: If38eba5cccfa7ae05db553c11aba86c1d0edbb38
    Closes-Bug: #1578275

Changed in solum:
status: In Progress → Fix Released
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/solum 5.0.0

This issue was fixed in the openstack/solum 5.0.0 release.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.
Subscribing...

Other bug subscribers