test_post_samples_scenarios fail when devstack enable ceilometer services

[ZhiQiang Fan]

after install devstack with keystone and ceilometer service enabled, and then run tox -epy27 under ceilometer clone, it get the following error, and cause 42 failure

Traceback (most recent call last):
  File "ceilometer/tests/base.py", line 75, in skip_if_not_implemented
    return func(*args, **kwargs)
  File "/home/zqfan/openstack/ceilometer-master/ceilometer/tests/api/v2/test_post_samples_scenarios.py", line 57, in setUp
    self.patch_publishers()
  File "/home/zqfan/openstack/ceilometer-master/ceilometer/tests/api/v2/test_post_samples_scenarios.py", line 43, in patch_publishers
    pipeline_hook = self.app.app.application.app.hooks[2]
AttributeError: 'StaticFileMiddleware' object has no attribute 'application'

after some debug, I found that, the /etc/ceilometer/ceilometer.conf has set debug to true, which cause ceilometer.tests.api.FunctionalTest().app to be (app=webtest.app.TestApp, app=pecan.middleware.static.StaticFileMiddleware, app=pecan.middleware.debug.DebugMiddleware, app=pecan.middleware.recursive.RecursiveMiddleware), which instead is expected to be (app=webtest.app.TestApp, app=pecan.middleware.recursive.RecursiveMiddleware, application=ceilometer.api.middleware.ParsableErrorMiddleware, app=pecan.core.Pecan)

mock the debug value to false can fix this problem, but further dig is needed too

[ZhiQiang Fan (aji-zqfan)]

will upload a patch soon

[ZhiQiang Fan (aji-zqfan)]

according to https://github.com/openstack/ceilometer/blob/master/ceilometer/api/app.py#L74, ceilometer.api will use cfg.CONF.debug to invoke pecan.make_app

and according to https://github.com/stackforge/pecan/blob/master/pecan/__init__.py#L101, pecan will wrap app with DebugMiddleware and StaticFileMiddleware if debug is true, so it explains the behavior

I think mock the debug value can eventually fix the problem

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by ZhiQiang Fan (<email address hidden>) on branch: master
Review: https://review.openstack.org/97099
Reason: llu's patch is better than this one, and that one has already been merged

[ZhiQiang Fan (aji-zqfan)]

the final solution is here: https://review.openstack.org/#/c/98295/

and this one has been already released