Ceilometer

test_setup_events_default_config fails

  1. Series kilo
  2. Bug #1449604
Bug #1449604 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
gordon chung
Ceilometer 5.0.0 "liberty"
Juno
Fix Released
Undecided
gordon chung
Ceilometer 2014.2.4
Kilo
Fix Committed
Undecided
Unassigned

Bug Description

when run independently, ie tox -edebug -- test_setup_events_default_config or tox -epy27 -- test_setup_events_default_config

the test will fail

======================================================================
FAIL: ceilometer.tests.event.test_converter.TestNotificationConverter.test_setup_events_default_config
tags: worker-0
----------------------------------------------------------------------
Empty attachments:
  stderr
  stdout

Traceback (most recent call last):
  File "ceilometer/tests/event/test_converter.py", line 776, in test_setup_events_default_config
    c = converter.setup_events(self.fake_plugin_mgr)
  File "ceilometer/event/converter.py", line 404, in setup_events
    add_catchall=not allow_drop)
  File "ceilometer/event/converter.py", line 336, in __init__
    trait_plugin_mgr))
  File "ceilometer/event/converter.py", line 189, in __init__
    trait_plugin_mgr)
  File "ceilometer/event/converter.py", line 114, in __init__
    % dict(jsonpath=fields, trait=name, err=e), self.cfg)
ceilometer.event.converter.EventDefinitionException: EventDefinitionException {'type': 'text', 'fields': 'publisher_id'}: Parse error in JSONPath specification 'publisher_id' for service: source code not available

when run as a group.. tox -epy27 -- TestNotificationConverter it will pass... something is up

gordon chung (chungg)
Changed in ceilometer:
importance: Undecided → Medium
gordon chung (chungg)
summary: - test_setup_events_default_config fails on dependency issue
+ test_setup_events_default_config fails
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/178260

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

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

commit c3b58d7dd2298d343cf88df22e0a60de060393fc
Author: gordon chung <email address hidden>
Date: Tue Apr 28 11:45:29 2015 -0400

    stop mocking os.path in test_setup_events_default_config

    test_setup_events_default_config unnecessarily mocks os.path.exists.
    this seems to be causing failures when running tox -edebug --
    test_setup_events_default_config (and occasionally in gate). it's
    dangerous to mock os.path because other libs might depend on it so
    let's stop mocking it.

    Change-Id: Ibe9027874f597348c5da6c92899cf8740eb678c2
    Closes-Bug: #1449604

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/juno)

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/181877

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

Reviewed: https://review.openstack.org/181877
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=1a5848aab5938c883363f0e533387c86dfae2747
Submitter: Jenkins
Branch: stable/juno

commit 1a5848aab5938c883363f0e533387c86dfae2747
Author: gordon chung <email address hidden>
Date: Mon May 11 08:26:19 2015 -0400

    stop mocking os.path in test_setup_events_default_config

    test_setup_events_default_config unnecessarily mocks os.path.exists.
    this seems to be causing failures when running tox -edebug --
    test_setup_events_default_config (and occasionally in gate). it's
    dangerous to mock os.path because other libs might depend on it so
    let's stop mocking it.

    Change-Id: Ibe9027874f597348c5da6c92899cf8740eb678c2
    Closes-Bug: #1449604
    (cherry picked from commit c3b58d7dd2298d343cf88df22e0a60de060393fc)

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

Reviewed: https://review.openstack.org/181874
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=e7c2286229b47a215e758c56fc7a431ae9b198f4
Submitter: Jenkins
Branch: stable/kilo

commit e7c2286229b47a215e758c56fc7a431ae9b198f4
Author: gordon chung <email address hidden>
Date: Tue Apr 28 11:45:29 2015 -0400

    stop mocking os.path in test_setup_events_default_config

    test_setup_events_default_config unnecessarily mocks os.path.exists.
    this seems to be causing failures when running tox -edebug --
    test_setup_events_default_config (and occasionally in gate). it's
    dangerous to mock os.path because other libs might depend on it so
    let's stop mocking it.

    Change-Id: Ibe9027874f597348c5da6c92899cf8740eb678c2
    Closes-Bug: #1449604
    (cherry picked from commit c3b58d7dd2298d343cf88df22e0a60de060393fc)

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.
Subscribing...

Other bug subscribers

Remote bug watches

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