)]}'
{"tempest/api/compute/servers/test_create_server.py":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"4dc5cb18b9195a339c11f6b0dec575f4455c0414","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from tempest.api.compute import base"},{"line_number":24,"context_line":"from tempest.common.utils import data_utils"},{"line_number":25,"context_line":"from tempest.common.utils.linux.remote_client import RemoteClient"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from tempest import config"},{"line_number":28,"context_line":"from tempest import test"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F8og%3D","line":26,"updated":"2014-01-06 11:33:26.000000000","message":"additional line is not needed. Section defines imports from tempest","commit_id":"f1f1d674dd6960cb01db9cfe3be1112d09583143"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"9e043eb05cdc59d64562b405cc7b4a277efbc0c5","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from tempest.api.compute import base"},{"line_number":24,"context_line":"from tempest.common.utils import data_utils"},{"line_number":25,"context_line":"from tempest.common.utils.linux.remote_client import RemoteClient"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from tempest import config"},{"line_number":28,"context_line":"from tempest import test"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F8g8%3D","line":26,"in_reply_to":"AAAAUH%2F%2F8og%3D","updated":"2014-01-06 12:11:59.000000000","message":"Done","commit_id":"f1f1d674dd6960cb01db9cfe3be1112d09583143"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"4dc5cb18b9195a339c11f6b0dec575f4455c0414","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @testtools.skipIf(not run_ssh, \u0027Instance validation tests are disabled.\u0027)"},{"line_number":95,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":96,"context_line":"    def test_can_log_into_created_server(self):"},{"line_number":97,"context_line":"        # Check that the user can authenticate with the generated password"},{"line_number":98,"context_line":"        linux_client \u003d RemoteClient(self.server, self.ssh_user, self.password)"},{"line_number":99,"context_line":"        self.assertTrue(linux_client.can_authenticate())"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F8oc%3D","line":96,"updated":"2014-01-06 11:33:26.000000000","message":"adding a test is little beyond the purpose of the patch. It\u0027s better to add it in the follow up patch.","commit_id":"f1f1d674dd6960cb01db9cfe3be1112d09583143"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"9e043eb05cdc59d64562b405cc7b4a277efbc0c5","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @testtools.skipIf(not run_ssh, \u0027Instance validation tests are disabled.\u0027)"},{"line_number":95,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":96,"context_line":"    def test_can_log_into_created_server(self):"},{"line_number":97,"context_line":"        # Check that the user can authenticate with the generated password"},{"line_number":98,"context_line":"        linux_client \u003d RemoteClient(self.server, self.ssh_user, self.password)"},{"line_number":99,"context_line":"        self.assertTrue(linux_client.can_authenticate())"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F8ig%3D","line":96,"in_reply_to":"AAAAUH%2F%2F8oc%3D","updated":"2014-01-06 12:11:59.000000000","message":"Weird, it appeared after rebase of dependent patch.\nNot mine, and it is absent in master, as I see.\n\nRemoved. Thanks.","commit_id":"f1f1d674dd6960cb01db9cfe3be1112d09583143"}],"tempest/api/compute/v3/admin/test_servers.py":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"4dc5cb18b9195a339c11f6b0dec575f4455c0414","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(ServersAdminTestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        skip_msg \u003d \"Skip until bug #1266050 is fixed\""},{"line_number":36,"context_line":"        raise cls.skipException(skip_msg)"},{"line_number":37,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":38,"context_line":"        cls.non_admin_client \u003d cls.servers_client"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F8n4%3D","line":35,"updated":"2014-01-06 11:33:26.000000000","message":"that seems strange to skip tests in the scope of refactoring effort. could you clarify?","commit_id":"f1f1d674dd6960cb01db9cfe3be1112d09583143"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"9e043eb05cdc59d64562b405cc7b4a277efbc0c5","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(ServersAdminTestJSON, cls).setUpClass()"},{"line_number":35,"context_line":"        skip_msg \u003d \"Skip until bug #1266050 is fixed\""},{"line_number":36,"context_line":"        raise cls.skipException(skip_msg)"},{"line_number":37,"context_line":"        cls.client \u003d cls.os_adm.servers_client"},{"line_number":38,"context_line":"        cls.non_admin_client \u003d cls.servers_client"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2F8hs%3D","line":35,"in_reply_to":"AAAAUH%2F%2F8n4%3D","updated":"2014-01-06 12:11:59.000000000","message":"It was needed, when v3 servers clients were unchanged.\nNow, when v3 methods, that are affected, has been changed in this patch, I agree, it is out of scope for this patch.\n\nRemoved.","commit_id":"f1f1d674dd6960cb01db9cfe3be1112d09583143"}]}