ceilometer devstack plugin should remove /etc/ceilometer and /var/log/ceilometer-api directory when clean is called

Bug #1509253 reported by ZhiQiang Fan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
ZhiQiang Fan

Bug Description

if we don't clean those directory and files, then next run to test devstack plugin may succeed even with a broken change, which is not good for developers

ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
gordon chung (chungg)
Changed in ceilometer:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

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

Reviewed: https://review.openstack.org/249114
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=af85965f555928e6d9897b69ab8de687008b08eb
Submitter: Jenkins
Branch: master

commit af85965f555928e6d9897b69ab8de687008b08eb
Author: ZhiQiang Fan <email address hidden>
Date: Wed Nov 25 01:45:20 2015 +0800

    remove config files when run clean.sh

    Currently we don't remove Ceilometer config files when ./clean.sh
    is run, which should be fixed because it will bite developers when
    they test change related to devstack script.

    Note, this patch also removes ceilometer api log files when WSGI
    is not enabled.

    Change-Id: I3fd62787708cf64d1860b2b64d25773e79b17fe9
    Closes-Bug: #1509253

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/ceilometer 6.0.0.0b1

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

Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Liusheng (liusheng)
Changed in ceilometer:
milestone: none → mitaka-1
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.