)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9e27f1ac7d3fcfb4a176e9dac3f2047785e21d2d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Yuriy Nesenenko \u003cynesenenko@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-14 19:44:38 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix incorrect work with host"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"String for host could include host@backend info. The service"},{"line_number":10,"context_line":"cinder-volume has got the info for host in this format and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"ba63e191_8fdda772","line":7,"range":{"start_line":7,"start_character":24,"end_line":7,"end_character":28},"updated":"2016-01-19 01:58:15.000000000","message":"This is actually skip the tests to get cinder changes merged.\n\nCommit title and message is confusing.Please change the same and Also mention the cinder patch which is changing this API for getting more clear view about those skip.\n\nAdding about new config option will be more better. Something that this config will be enable(True) for master and False for kilo and Liberty","commit_id":"50bb54a305dd0c7b93472432c24abd9e8aea6702"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8cb954cb4189aa8b980b37b0aa4707638e846cb7","unresolved":false,"context_lines":[{"line_number":12,"context_line":"This fix related with the incorrect work of service-list with"},{"line_number":13,"context_line":"\u0027--host\u0027 argument. The filter should ignore backend name in the"},{"line_number":14,"context_line":"host name. This fix related with bug #1530144 in service-list filter."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Depends-On: Ifdc0bab819c05a00cd0c20316bd81961cf6aeb88"},{"line_number":17,"context_line":"Change-Id: I4044ab15078ecf54447e1c6e67c27fc8d7c9d6f7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"da6ed579_dff00ccc","line":15,"updated":"2016-01-18 10:07:37.000000000","message":"We can put a tag for related bug like this. \n\n Related-Bug: #1530144\n\nIt\u0027s not a mandatory and just FYI, though.","commit_id":"50bb54a305dd0c7b93472432c24abd9e8aea6702"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"22c599e55d5259050c4784105b0e4200e568c299","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Skip the test_get_service_by_host_name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch skips the test_get_service_by_host_name to get cinder"},{"line_number":10,"context_line":"changes merged [1]. String for host could include host@backend info. "},{"line_number":11,"context_line":"The service cinder-volume has got the info for host in this format and"},{"line_number":12,"context_line":"therefore it is missing in the service list. "},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"ba63e191_8e1a7816","line":10,"updated":"2016-01-19 10:15:25.000000000","message":"one tailing space","commit_id":"fc734d688652a596df5800a93b5146b41c7bf625"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"22c599e55d5259050c4784105b0e4200e568c299","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch skips the test_get_service_by_host_name to get cinder"},{"line_number":10,"context_line":"changes merged [1]. String for host could include host@backend info. "},{"line_number":11,"context_line":"The service cinder-volume has got the info for host in this format and"},{"line_number":12,"context_line":"therefore it is missing in the service list. "},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] I21775106693176ca128dbfd9db0d43cfc58de00a"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"ba63e191_4e5ec0ea","line":12,"updated":"2016-01-19 10:15:25.000000000","message":"ditto","commit_id":"fc734d688652a596df5800a93b5146b41c7bf625"}],"tempest/api/volume/admin/test_volume_services.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8cb954cb4189aa8b980b37b0aa4707638e846cb7","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        for service in services:"},{"line_number":53,"context_line":"            self.assertEqual(self.binary_name, service[\u0027binary\u0027])"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @decorators.skip_because(bug\u003d\"1530144\")"},{"line_number":56,"context_line":"    @test.idempotent_id(\u0027178710e4-7596-4e08-9333-745cb8bc4f8d\u0027)"},{"line_number":57,"context_line":"    @testtools.skipUnless(CONF.volume_feature_enabled.volume_services,"},{"line_number":58,"context_line":"                          \"Skip. Not implemented on Liberty and Kilo.\")"}],"source_content_type":"text/x-python","patch_set":11,"id":"da6ed579_7f71e0df","line":55,"updated":"2016-01-18 10:07:37.000000000","message":"So why do you want to skip this?\nI think you already submitted a patch for closing it.\nI21775106693176ca128dbfd9db0d43cfc58de00a\n\nSo I think you can remove this skip_because decorator with adding more dependency on it.\n\nhttp://docs.openstack.org/infra/manual/developers.html#multiple-changes","commit_id":"50bb54a305dd0c7b93472432c24abd9e8aea6702"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"21af2df9e131a364fb288d8e66c381596a83735e","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        for service in services:"},{"line_number":53,"context_line":"            self.assertEqual(self.binary_name, service[\u0027binary\u0027])"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @decorators.skip_because(bug\u003d\"1530144\")"},{"line_number":56,"context_line":"    @test.idempotent_id(\u0027178710e4-7596-4e08-9333-745cb8bc4f8d\u0027)"},{"line_number":57,"context_line":"    @testtools.skipUnless(CONF.volume_feature_enabled.volume_services,"},{"line_number":58,"context_line":"                          \"Skip. Not implemented on Liberty and Kilo.\")"}],"source_content_type":"text/x-python","patch_set":11,"id":"da6ed579_7286f714","line":55,"in_reply_to":"da6ed579_7f71e0df","updated":"2016-01-18 10:16:42.000000000","message":"ah, the reason is the \"one-way dependencies\".\nMore comments help us :) But it\u0027s OK for me now.","commit_id":"50bb54a305dd0c7b93472432c24abd9e8aea6702"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"83a655902b433bf829f82c83af4d6b60eaf921ea","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    @decorators.skip_because(bug\u003d\"1530144\")"},{"line_number":56,"context_line":"    @test.idempotent_id(\u0027178710e4-7596-4e08-9333-745cb8bc4f8d\u0027)"},{"line_number":57,"context_line":"    @testtools.skipUnless(CONF.volume_feature_enabled.volume_services,"},{"line_number":58,"context_line":"                          \"Skip. Not implemented on Liberty and Kilo.\")"},{"line_number":59,"context_line":"    def test_get_service_by_host_name(self):"},{"line_number":60,"context_line":"        def get_host(host):"},{"line_number":61,"context_line":"            return host.split(\u0027@\u0027)[0]"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a57f1b5_642d5494","line":58,"updated":"2016-02-12 14:57:52.000000000","message":"I didn\u0027t think there was any expectation that you could run a newer tempest against an older stack?","commit_id":"9b14cecb881b992fc453fabf87f31bff25efce5b"},{"author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"change_message_id":"c809c37c3a8369f2d3c9f03383882c4ccd8427e7","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    @decorators.skip_because(bug\u003d\"1530144\")"},{"line_number":56,"context_line":"    @test.idempotent_id(\u0027178710e4-7596-4e08-9333-745cb8bc4f8d\u0027)"},{"line_number":57,"context_line":"    @testtools.skipUnless(CONF.volume_feature_enabled.volume_services,"},{"line_number":58,"context_line":"                          \"Skip. Not implemented on Liberty and Kilo.\")"},{"line_number":59,"context_line":"    def test_get_service_by_host_name(self):"},{"line_number":60,"context_line":"        def get_host(host):"},{"line_number":61,"context_line":"            return host.split(\u0027@\u0027)[0]"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a57f1b5_7160b1cf","line":58,"in_reply_to":"3a57f1b5_642d5494","updated":"2016-02-12 15:28:20.000000000","message":"If you look at jenkins jobs you\u0027ll see gate-tempest-dsvm-full-liberty and gate-tempest-dsvm-full-kilo and so on. As tempest don\u0027t have stable branches, they run tests on older stacks.","commit_id":"9b14cecb881b992fc453fabf87f31bff25efce5b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"574a8bed7e7add40c7601df8b1e2d6977e18cd26","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    @test.idempotent_id(\u0027178710e4-7596-4e08-9333-745cb8bc4f8d\u0027)"},{"line_number":56,"context_line":"    def test_get_service_by_host_name(self):"},{"line_number":57,"context_line":"        def get_host(host):"},{"line_number":58,"context_line":"            if CONF.volume_feature_enabled.volume_services:"},{"line_number":59,"context_line":"                host \u003d host.split(\u0027@\u0027)[0]"},{"line_number":60,"context_line":"            return host"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"        services_on_host \u003d [service for service in self.services if"}],"source_content_type":"text/x-python","patch_set":15,"id":"dae33548_08326710","line":59,"range":{"start_line":58,"start_character":0,"end_line":59,"end_character":41},"updated":"2016-02-16 05:52:37.000000000","message":"There is one issue with this fix. If first service has host name with @backendname then it will fail as you are fetching only first part always.\n\nBut here ok as this test is skipped in this patch. lets fix that in https://review.openstack.org/#/c/267765/4","commit_id":"86d90f2d4c5d037e69b63dde04e0d5c86a361ad1"}],"tempest/config.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"1f5a4b3905de5dfb5a0a03cc805853a25ff40a58","unresolved":false,"context_lines":[{"line_number":791,"context_line":"                deprecated_for_removal\u003dTrue),"},{"line_number":792,"context_line":"    cfg.BoolOpt(\u0027volume_services\u0027,"},{"line_number":793,"context_line":"                default\u003dFalse,"},{"line_number":794,"context_line":"                help\u003d\u0027Extract correct host info from host@backend\u0027)"},{"line_number":795,"context_line":"]"},{"line_number":796,"context_line":""},{"line_number":797,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"9a68dd71_5a432a6c","line":794,"updated":"2016-01-25 12:49:39.000000000","message":"Please add a line about when this feature flag could be deleted.","commit_id":"a3271fefd44a8e9cc0dce7aeb8a6555fd270a0ae"}]}