)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"a958b42face0fe3d4a269c2f1ef25c1454363b86","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"syntax err on wrong_tests arithmetic"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TrivialFix"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ib1c927faf86fd7555c2f703968fd9b3a4dcd74cb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa0719c6_59903c95","line":9,"updated":"2016-03-24 07:30:52.000000000","message":"To understand what is going on here I need to read not only you patch but also tempest stuff(To read the code of this function https://github.com/openstack/rally/blob/master/rally/verification/tempest/tempest.py#L359) It would be very helpful if you\u0027d show us the trace with error before diving in the code. Also lets specify in commit msg that this is TypeError http://paste.openstack.org/show/491667/\nThe title should be more clear. For example \"Fix TypeError in tempest_tests_exists validator\" and the paste with the trace in the commit msg.","commit_id":"d50a1da6ab79bd1380a0a91d73149f47d60b6d0c"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"bebac96250a9f1c681446830b92f47cbe7934c46","unresolved":false,"context_lines":[{"line_number":11,"context_line":"list into a set, without that you get \"TypeError\","},{"line_number":12,"context_line":"when doing arithmetic on the list."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"TrivialFix"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ib1c927faf86fd7555c2f703968fd9b3a4dcd74cb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"7a1b2928_44400157","line":14,"updated":"2016-04-11 12:39:46.000000000","message":"Please don\u0027t write \"TrivialFix\" its totally redundant\u003d)","commit_id":"d3b96bfa9939bf0461104f996aebec65f3b5c7e3"}],"rally-jobs/unstable-neutron.yaml":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"bebac96250a9f1c681446830b92f47cbe7934c46","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        failure_rate:"},{"line_number":389,"context_line":"          max: 20"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"  TempestScenario.list_of_tests:"},{"line_number":392,"context_line":"    -"},{"line_number":393,"context_line":"      args:"},{"line_number":394,"context_line":"        tempest_conf: \"/etc/tempest/tempest.conf\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7a1b2928_4472c1e6","line":391,"updated":"2016-04-11 12:39:46.000000000","message":"http://logs.openstack.org/79/296379/6/check/gate-rally-dsvm-neutron-unstable/888ced4/console.html.gz#_2016-04-08_18_59_45_864 Here is an error","commit_id":"d3b96bfa9939bf0461104f996aebec65f3b5c7e3"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"bb52817a1d4e4d8014dd52d07cc0df069ed8f0f1","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        failure_rate:"},{"line_number":389,"context_line":"          max: 20"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"  TempestScenario.list_of_tests:"},{"line_number":392,"context_line":"    -"},{"line_number":393,"context_line":"      args:"},{"line_number":394,"context_line":"        tempest_conf: \"/etc/tempest/tempest.conf\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1a122d0e_0741060c","line":391,"in_reply_to":"7a1b2928_4472c1e6","updated":"2016-04-15 10:51:16.000000000","message":"Yes, i saw that. It cannot find the tempest tests:\n\n\u0027tempest.api.identity.v2.test_tokens.TokensTest.test_create_token\u0027, \u0027tempest.api.image.v2.test_images.ListImagesTest.test_index_no_params\u0027\n\nWhich tests should i use here?","commit_id":"d3b96bfa9939bf0461104f996aebec65f3b5c7e3"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"fe31a99cb8a9f61386f99f176a520ea0e1179e80","unresolved":false,"context_lines":[{"line_number":264,"context_line":"        failure_rate:"},{"line_number":265,"context_line":"          max: 0"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"  NeutronNetworks.create_and_update_ports:"},{"line_number":268,"context_line":"    -"},{"line_number":269,"context_line":"      args:"},{"line_number":270,"context_line":"        network_create_args: {}"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"1a122d0e_e026ebcc","line":267,"updated":"2016-04-26 12:13:08.000000000","message":"How this scenario \"NeutronNetworks.create_and_update_ports\" is related to the patch?","commit_id":"2bc746891ecac955694ed1a3ac6fc9d151534f2a"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"3856ed0250201d715b915c3f8503e7292796b6cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9abb7d3a_a874fd9b","line":277,"updated":"2016-05-30 11:29:59.000000000","message":"Missed \u0027sla\u0027 section:\n\n      sla:\n        failure_rate:\n          max: 0","commit_id":"8a117b80bb94946d16f3f428c1dd032fab06f284"}],"rally/common/utils.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"1aed87fd5712fe17a8d0a1ee853faa424c61d450","unresolved":false,"context_lines":[{"line_number":310,"context_line":"        task_id \u003d None"},{"line_number":311,"context_line":"        if hasattr(self, \"task\"):"},{"line_number":312,"context_line":"            task_id \u003d self.task[\"uuid\"]"},{"line_number":313,"context_line":"        elif getattr(self, \"verification\", None):"},{"line_number":314,"context_line":"            task_id \u003d self.verification[\"uuid\"]"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        match \u003d self._resource_name_placeholder_re.match("}],"source_content_type":"text/x-python","patch_set":16,"id":"9abb7d3a_039882bf","line":313,"updated":"2016-05-30 15:05:12.000000000","message":"We could get an error here https://github.com/openstack/rally/blob/master/rally/common/utils.py#L268 If task_id will stay None","commit_id":"40b94d6a5cb12533be750d719a4ccbf36274b114"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"934113c0eb3e6e844244e9b5379b6d44b32b1864","unresolved":false,"context_lines":[{"line_number":310,"context_line":"        task_id \u003d None"},{"line_number":311,"context_line":"        if hasattr(self, \"task\"):"},{"line_number":312,"context_line":"            task_id \u003d self.task[\"uuid\"]"},{"line_number":313,"context_line":"        elif getattr(self, \"verification\", None):"},{"line_number":314,"context_line":"            task_id \u003d self.verification[\"uuid\"]"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        match \u003d self._resource_name_placeholder_re.match("}],"source_content_type":"text/x-python","patch_set":16,"id":"9abb7d3a_ec91eb53","line":313,"in_reply_to":"9abb7d3a_039882bf","updated":"2016-05-30 16:03:20.000000000","message":"Interesting that locally all fine for me. Hm...","commit_id":"40b94d6a5cb12533be750d719a4ccbf36274b114"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"eea14c375a6bf1c11e19eb54a0c3ada670580a27","unresolved":false,"context_lines":[{"line_number":310,"context_line":"        task_id \u003d None"},{"line_number":311,"context_line":"        if hasattr(self, \"task\"):"},{"line_number":312,"context_line":"            task_id \u003d self.task[\"uuid\"]"},{"line_number":313,"context_line":"        elif getattr(self, \"verification\", None):"},{"line_number":314,"context_line":"            task_id \u003d self.verification[\"uuid\"]"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        match \u003d self._resource_name_placeholder_re.match("}],"source_content_type":"text/x-python","patch_set":16,"id":"9abb7d3a_b2245aaf","line":313,"in_reply_to":"9abb7d3a_ec91eb53","updated":"2016-05-30 16:45:06.000000000","message":"I can\u0027t get how it was working previously but https://github.com/openstack/rally/blob/master/rally/plugins/openstack/context/not_for_production/tempest.py#L45 In context we don\u0027t pass verification","commit_id":"40b94d6a5cb12533be750d719a4ccbf36274b114"}],"rally/task/validation.py":[{"author":{"_account_id":20196,"name":"Piyush","email":"piyush2011257@gmail.com","username":"pirsriva"},"change_message_id":"a75af6a03fbe89eca52e63e9815a6e16c60397c0","unresolved":false,"context_lines":[{"line_number":441,"context_line":"        if not test.startswith(\"tempest.api.\"):"},{"line_number":442,"context_line":"            tests[i] \u003d \"tempest.api.\" + test"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":"    wrong_tests \u003d list(set(tests) - set(allowed_tests))"},{"line_number":445,"context_line":""},{"line_number":446,"context_line":"    if wrong_tests:"},{"line_number":447,"context_line":"        message \u003d (_(\"One or more tests not found: \u0027%s\u0027\") %"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_fc067a1a","line":444,"updated":"2016-03-23 17:58:31.000000000","message":"Just a minor change to keep it simple. I don\u0027t think we need extra list() wrapper on top. Otherwise LGTM.","commit_id":"1501e750b78b637916b4151801335a7ab838fe7a"}]}