direct event publisher assumes ordering of events

Bug #1427367 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
High
gordon chung

Bug Description

http://logs.openstack.org/07/159907/2/gate/gate-ceilometer-python27/f16c01d/testr_results.html.gz

Traceback (most recent call last):
_StringException: Empty attachments:
  stderr
  stdout

Traceback (most recent call last):
  File "ceilometer/tests/base.py", line 97, in skip_if_not_implemented
    return func(*args, **kwargs)
  File "ceilometer/tests/publisher/test_direct.py", line 102, in test_direct_publisher
    self.assertEqual(['event_%d' % i for i in range(0, 5)], e_types)
  File "/home/jenkins/workspace/gate-ceilometer-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 348, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/home/jenkins/workspace/gate-ceilometer-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 433, in assertThat
    raise mismatch_error
MismatchError: !=:
reference = ['event_0', 'event_1', 'event_2', 'event_3', 'event_4']
actual = [u'event_3', u'event_1', u'event_0', u'event_2', u'event_4']

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

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

commit 60d41d438bb6164db8eb437dacc0d2a016ec140d
Author: gordon chung <email address hidden>
Date: Mon Mar 2 15:31:43 2015 -0500

    remove event_types ordering assumption

    test occassionally fails because it assumes event_types have an
    order when returned. this is not true, so remove this assumption.

    Change-Id: I6d5ba033b373643a13a214f5efd4ba366e6ae147
    Closes-Bug: #1427367

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: kilo-3 → 2015.1.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.