ci gate fail because depended module upgrade

Bug #1460869 reported by ZhiQiang Fan
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
ZhiQiang Fan

Bug Description

Traceback (most recent call last):
  File "ceilometer/tests/base.py", line 97, in skip_if_not_implemented
    return func(*args, **kwargs)
  File "ceilometer/tests/api/v2/test_alarm_scenarios.py", line 785, in test_post_invalid_alarm_input_enabled_int
    resp.json['error_message']['faultstring'])
KeyError: 'error_message'

Traceback (most recent call last):
  File "ceilometer/tests/base.py", line 97, in skip_if_not_implemented
    return func(*args, **kwargs)
  File "ceilometer/tests/api/v2/test_alarm_scenarios.py", line 761, in test_post_invalid_alarm_input_enabled_str
    resp.json['error_message']['faultstring'])
  File "/home/jenkins/workspace/gate-ceilometer-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 350, 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 435, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: !=:
reference = "Invalid input for field/attribute enabled. Value: 'bad_enabled'. Wrong type. Expected '<type 'bool'>', got '<type 'str'>'"
actual = u"Invalid input for field/attribute data. Value: '{'name': 'alarm5', 'state': 'ok', 'enabled': 'bad_enabled', 'type': 'threshold', 'threshold_rule': {'meter_name': 'ameter', 'comparison_operator': 'gt', 'threshold': 50.0}}'. Value not an unambiguous boolean: bad_enabled"

ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
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/187425

Changed in ceilometer:
status: New → In Progress
Revision history for this message
Stéphane Bisinger (kjir) wrote :

By looking at those unit test I can't help but wonder: is it useful to test WSME behaviour in Ceilometer? Shouldn't those kind of tests be in WSME rather that Ceilometer?

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

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

commit a5b04650c8735551320176c5822d021d562412b5
Author: ZhiQiang Fan <email address hidden>
Date: Tue Jun 2 10:05:18 2015 +0800

    adjust alarm post ut code to adapt to upstream wsme

    wsme has upgraded from 0.6.4 to 0.7.0, and now it supports convert
    int and str to boolean, which causes our unit test code failed.

    This change adjust the affected unit test code to adapt to such
    changes.

    Change-Id: Ic0d098e8e689e2421f261a72744abc3da576d8b2
    Closes-Bug: #1460869

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/187969

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ceilometer (stable/kilo)

Change abandoned by Lan Qi Song (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/187969
Reason: Got it, thanks gordc, abandon this patch.

Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-1 → 5.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.