test_notifications.TestMeterProcessing.test_fallback_meter_path fails if meter.yaml files exists under /etc directory

Bug #1486869 reported by Lianhao Lu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
Lianhao Lu

Bug Description

If meter.yaml files exists under /etc/ceilometer directory, the test_notifications.TestMeterProcessing.test_fallback_meter_path test case will fail:
======================================================================
FAIL: test_notifications.TestMeterProcessing.test_fallback_meter_path
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test_notifications.py", line 162, in test_fallback_meter_path
    self.assertIn("meter/data/meters.yaml", fall_bak_path)
  File "/opt/stack/ceilometer/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 356, in assertIn
    self.assertThat(haystack, Contains(needle), message)
  File "/opt/stack/ceilometer/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 435, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: 'meter/data/meters.yaml' not in '/etc/ceilometer/meters.yaml'

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/215001

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

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

commit 329b986f13d9c9127e9b8b46eb26d66f6807eb12
Author: Lianhao Lu <email address hidden>
Date: Thu Aug 20 15:32:31 2015 +0800

    Corrected test_fallback_meter_path test case

    Make sure the test_fallback_meter_path test case won't fail if
    /etc/ceilometer/meters.yaml file exists or not.

    Closes-Bug: #1486869
    Change-Id: I630aa6a2fca48d87fcd7f9584701e461222a9bed

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-3 → 5.0.0
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.