test_setup_events_default_config fails

[gordon chung]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)