)]}'
{"ceilometer/tests/tempest/scenario/test_telemetry_integration.py":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"5d9e338e49cdf1149e7581f5fe291f205d928613","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        with file(os.path.join(test_dir, name)) as scenario_file:"},{"line_number":89,"context_line":"            p \u003d subprocess.Popen([\"gabbi-run\", \"-x\", \"-v\", \"all\"],"},{"line_number":90,"context_line":"                                 cwd\u003dtest_dir,"},{"line_number":91,"context_line":"                                 stdin\u003dscenario_file,"},{"line_number":92,"context_line":"                                 stdout\u003dsubprocess.PIPE,"},{"line_number":93,"context_line":"                                 stderr\u003dsubprocess.STDOUT)"},{"line_number":94,"context_line":"            stderr, stdout \u003d p.communicate()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_071be4ca","line":91,"updated":"2017-03-22 19:15:00.000000000","message":"not sure it would be useful here but do you know about gabbi-run being able to handle files instead of stdin:\n\ngabbi-run http://host:port -- /my/test.yaml /my/other.yaml","commit_id":"3b6b9304b8a912753738e29fa0494425b8d991c1"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"0604811a67fa508e2a1206ab41cad8cfb240a841","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        with file(os.path.join(test_dir, name)) as scenario_file:"},{"line_number":89,"context_line":"            p \u003d subprocess.Popen([\"gabbi-run\", \"-x\", \"-v\", \"all\"],"},{"line_number":90,"context_line":"                                 cwd\u003dtest_dir,"},{"line_number":91,"context_line":"                                 stdin\u003dscenario_file,"},{"line_number":92,"context_line":"                                 stdout\u003dsubprocess.PIPE,"},{"line_number":93,"context_line":"                                 stderr\u003dsubprocess.STDOUT)"},{"line_number":94,"context_line":"            stderr, stdout \u003d p.communicate()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_a1e4a9ab","line":91,"in_reply_to":"1a1ced50_071be4ca","updated":"2017-03-23 08:07:50.000000000","message":"Thx I have used that instead.","commit_id":"3b6b9304b8a912753738e29fa0494425b8d991c1"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"601273540da98f0c696008793777f4e7dd108331","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            \"GLANCE_IMAGE_NAME\": self.glance_image_create(),"},{"line_number":90,"context_line":"        })"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        p \u003d subprocess.Popen([\"gabbi-run\", \"-x\", \"--\", filename],"},{"line_number":93,"context_line":"                             cwd\u003dTEST_DIR,"},{"line_number":94,"context_line":"                             stdout\u003dsubprocess.PIPE,"},{"line_number":95,"context_line":"                             stderr\u003dsubprocess.STDOUT)"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a1ced50_7700aa34","line":92,"updated":"2017-03-23 10:37:08.000000000","message":"isn\u0027t there a way to call it without Popen? I\u0027m sure it could be imported in some way :)","commit_id":"1f3c11c1fb53553e31cbbb650eaaf222fb7b9ede"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"367c1b9ad0fb47dd24956d3d05a85e5d7083a714","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            \"GLANCE_IMAGE_NAME\": self.glance_image_create(),"},{"line_number":90,"context_line":"        })"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        p \u003d subprocess.Popen([\"gabbi-run\", \"-x\", \"--\", filename],"},{"line_number":93,"context_line":"                             cwd\u003dTEST_DIR,"},{"line_number":94,"context_line":"                             stdout\u003dsubprocess.PIPE,"},{"line_number":95,"context_line":"                             stderr\u003dsubprocess.STDOUT)"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a1ced50_601e220d","line":92,"in_reply_to":"1a1ced50_7700aa34","updated":"2017-03-23 11:33:15.000000000","message":"Yes, but but that need some work to create a testrunner that\nuse the logger instead of stderr.","commit_id":"1f3c11c1fb53553e31cbbb650eaaf222fb7b9ede"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"ee6375be72b1f0021c6ac8037c723317ef120e70","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from ceilometer.tests.tempest.scenario import manager"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import logging"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"ffe62b97_c8705f0b","line":23,"updated":"2017-03-28 14:13:58.000000000","message":"should be part of first grouping","commit_id":"239f4f811a50c9ba161103c3a42b2ebe89b10227"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"7b086061f6beaed43875d5e1ebe0b3e594f70c0e","unresolved":false,"context_lines":[{"line_number":118,"context_line":"                    test, bt \u003d failures[0]"},{"line_number":119,"context_line":"                    name \u003d test.test_data.get(\u0027name\u0027, test.id())"},{"line_number":120,"context_line":"                    msg \u003d \u0027From test \"%s\" :\\n%s\u0027 % (name, bt)"},{"line_number":121,"context_line":"                    self.fail(msg)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"            self.assertTrue(result.wasSuccessful())"},{"line_number":124,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a1ced50_d09a3f13","line":121,"updated":"2017-03-23 13:27:06.000000000","message":"Hmm. I guess this will work, but does it swallow any \u0027verbose: True\u0027 output that a test might have. Does it matter?","commit_id":"239f4f811a50c9ba161103c3a42b2ebe89b10227"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"c2488a2a215cd7a3150d487571356a4fa975dfcf","unresolved":false,"context_lines":[{"line_number":118,"context_line":"                    test, bt \u003d failures[0]"},{"line_number":119,"context_line":"                    name \u003d test.test_data.get(\u0027name\u0027, test.id())"},{"line_number":120,"context_line":"                    msg \u003d \u0027From test \"%s\" :\\n%s\u0027 % (name, bt)"},{"line_number":121,"context_line":"                    self.fail(msg)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"            self.assertTrue(result.wasSuccessful())"},{"line_number":124,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a1ced50_8e8d08d5","line":121,"in_reply_to":"1a1ced50_d09a3f13","updated":"2017-03-23 14:04:23.000000000","message":"I don\u0027t think so. To me verbose shall be global or for punctual debugging, that\u0027s it.","commit_id":"239f4f811a50c9ba161103c3a42b2ebe89b10227"}]}