)]}'
{"tempest/api_schema/compute/servers.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"9e4a351a6bb34abd38a939c5192b3821d4580dd0","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    }"},{"line_number":24,"context_line":"}"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"resp_202_schema \u003d {"},{"line_number":27,"context_line":"    \u0027status_code\u0027: [202]"},{"line_number":28,"context_line":"}"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAXH%2F%2Fyj4%3D","line":26,"updated":"2014-04-17 13:00:54.000000000","message":"it is better that the schema name shows its owner.\n\nfor example, server_actions_without_confirmed_resize and confirmed_resize are better as v2 schemas. If doing it, we can know what schema is used for each API just by seeing schema file.","commit_id":"f476e8049fc1d2ccfb52d953e7b51954b36c4a69"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"bed8fbcbc1f7a130fac5c1afcdc65a12d05c49c0","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    }"},{"line_number":24,"context_line":"}"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"resp_202_schema \u003d {"},{"line_number":27,"context_line":"    \u0027status_code\u0027: [202]"},{"line_number":28,"context_line":"}"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAXH%2F%2Fs4Q%3D","line":26,"in_reply_to":"AAAAXH%2F%2Fyj4%3D","updated":"2014-04-18 04:57:53.000000000","message":"Yes, That will be better.\nActually 204 is being used for delete/change password also for v2 and v3. I will define their name accordingly but for all other action how about \"server_actions\"\nFor example\n- confirm_resize - \"server_actions_confirm_resize\" -\u003ev2 (status - 204)\n- delete_password - \"server_actions_delete_password\" -\u003ecommon in v2 \u0026 v3 (status - 204)\n- change_password - \"server_actions_change_password\" -\u003e v3 (status - 204)\n- For all other actions- \"server_actions\" -\u003e common in v2 \u0026 v3 (status - 202)","commit_id":"f476e8049fc1d2ccfb52d953e7b51954b36c4a69"}],"tempest/api_schema/compute/v2/servers.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"fe2958925cf1cb216ed31f09569a2638ba59cd88","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest.api_schema.compute import parameter_types"},{"line_number":18,"context_line":"from tempest.api_schema.compute import servers"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"9ad9bd40_401e3ebc","side":"PARENT","line":16,"updated":"2014-05-07 02:38:29.000000000","message":"this new line is necessary for representing python library types.\nPython libraries should be imported with the following order.\n\n standard libs\n [new line]\n 3rd-party libs\n [new line]\n project libs\n\ncopy is standard libs and tempest.api_schema.. is project libs so a new line is necessary.","commit_id":"92d3d864945cc44b6bd155db44fc8a491d4b09a4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a8819bada974bd12d9efb5308ecc9cae345cfdfe","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest.api_schema.compute import parameter_types"},{"line_number":18,"context_line":"from tempest.api_schema.compute import servers"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"9ad9bd40_4301e8ad","side":"PARENT","line":16,"in_reply_to":"9ad9bd40_401e3ebc","updated":"2014-05-07 03:27:24.000000000","message":"Thanks. \nDone.","commit_id":"92d3d864945cc44b6bd155db44fc8a491d4b09a4"}],"tempest/api_schema/compute/v3/servers.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"fe2958925cf1cb216ed31f09569a2638ba59cd88","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest.api_schema.compute import parameter_types"},{"line_number":18,"context_line":"from tempest.api_schema.compute import servers"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"9ad9bd40_a080eac0","side":"PARENT","line":16,"updated":"2014-05-07 02:38:29.000000000","message":"ditto","commit_id":"92d3d864945cc44b6bd155db44fc8a491d4b09a4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a8819bada974bd12d9efb5308ecc9cae345cfdfe","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest.api_schema.compute import parameter_types"},{"line_number":18,"context_line":"from tempest.api_schema.compute import servers"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"9ad9bd40_63feecaf","side":"PARENT","line":16,"in_reply_to":"9ad9bd40_a080eac0","updated":"2014-05-07 03:27:24.000000000","message":"Done.","commit_id":"92d3d864945cc44b6bd155db44fc8a491d4b09a4"}],"tempest/services/compute/json/servers_client.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"4675be0e6518e2c052bacfd84115c4a9f9d1e4cb","unresolved":false,"context_lines":[{"line_number":415,"context_line":""},{"line_number":416,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":417,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":418,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027adminPass\u0027, **kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":421,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F9gTY%3D","line":418,"updated":"2014-04-04 03:43:42.000000000","message":"I think we need a response schema for this case because it does return information in the body.","commit_id":"5b6d2a070d82e75b47510f80f4f45415cc915aaa"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a46a7e66637bff0a07e9a256d769956aa87bf24c","unresolved":false,"context_lines":[{"line_number":415,"context_line":""},{"line_number":416,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":417,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":418,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027adminPass\u0027, **kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":421,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F9gAc%3D","line":418,"in_reply_to":"AAAAWH%2F9gRg%3D","updated":"2014-04-04 04:37:27.000000000","message":"Released in https://review.openstack.org/#/c/85234/ \nRelease in different patch to have action with response body in separate commit which will be easy to review.","commit_id":"5b6d2a070d82e75b47510f80f4f45415cc915aaa"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c4e1afbe70dc9b31e066cdc50ad176ea04f847d0","unresolved":false,"context_lines":[{"line_number":415,"context_line":""},{"line_number":416,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":417,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":418,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027adminPass\u0027, **kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":421,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F9gRg%3D","line":418,"in_reply_to":"AAAAWH%2F9gTY%3D","updated":"2014-04-04 03:47:21.000000000","message":"Yes Chris you are right. I am in progress of adding schema for this. As it has response body so I planned to release that in different patch.","commit_id":"5b6d2a070d82e75b47510f80f4f45415cc915aaa"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"446a3d68a231ad5951118ef25a106f49c7157d4d","unresolved":false,"context_lines":[{"line_number":439,"context_line":""},{"line_number":440,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":441,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":442,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027adminPass\u0027, None, **kwargs)"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":445,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":11,"id":"7adec928_36f59783","line":442,"updated":"2014-05-08 06:37:08.000000000","message":"Is this change required ?","commit_id":"4ec18f84877dcea6ea8cd62cdb8368712dd6846e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a2155510e4d11e478b5ed81b18a70ad2d33fac83","unresolved":false,"context_lines":[{"line_number":439,"context_line":""},{"line_number":440,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":441,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":442,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027adminPass\u0027, None, **kwargs)"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":445,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":11,"id":"7adec928_b6c1a7e6","line":442,"in_reply_to":"7adec928_36f59783","updated":"2014-05-08 06:51:22.000000000","message":"Thanks afazekas for review. Actually if we donot pass this then, it will validate this API response with common_schema.server_actions_common_schema (L218). So passing the response key as this API response has the one and implementing its validation in https://review.openstack.org/#/c/85234/","commit_id":"4ec18f84877dcea6ea8cd62cdb8368712dd6846e"}],"tempest/services/compute/v3/json/servers_client.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"4675be0e6518e2c052bacfd84115c4a9f9d1e4cb","unresolved":false,"context_lines":[{"line_number":404,"context_line":""},{"line_number":405,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":406,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":407,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027admin_password\u0027, **kwargs)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":410,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F9gSc%3D","line":407,"updated":"2014-04-04 03:43:42.000000000","message":"this one needs a response schema too for the body response","commit_id":"5b6d2a070d82e75b47510f80f4f45415cc915aaa"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a46a7e66637bff0a07e9a256d769956aa87bf24c","unresolved":false,"context_lines":[{"line_number":404,"context_line":""},{"line_number":405,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":406,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":407,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027admin_password\u0027, **kwargs)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":410,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F9gAY%3D","line":407,"in_reply_to":"AAAAWH%2F9gRY%3D","updated":"2014-04-04 04:37:27.000000000","message":"Released in https://review.openstack.org/#/c/85234/ \nRelease in different patch to have action with response body in separate commit which will be easy to review.","commit_id":"5b6d2a070d82e75b47510f80f4f45415cc915aaa"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c4e1afbe70dc9b31e066cdc50ad176ea04f847d0","unresolved":false,"context_lines":[{"line_number":404,"context_line":""},{"line_number":405,"context_line":"    def rescue_server(self, server_id, **kwargs):"},{"line_number":406,"context_line":"        \"\"\"Rescue the provided server.\"\"\""},{"line_number":407,"context_line":"        return self.action(server_id, \u0027rescue\u0027, \u0027admin_password\u0027, **kwargs)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def unrescue_server(self, server_id):"},{"line_number":410,"context_line":"        \"\"\"Unrescue the provided server.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F9gRY%3D","line":407,"in_reply_to":"AAAAWH%2F9gSc%3D","updated":"2014-04-04 03:47:21.000000000","message":"Yes Chris you are right. I am in progress of adding schema for this. As it has response body so I planned to release that in different patch.","commit_id":"5b6d2a070d82e75b47510f80f4f45415cc915aaa"}]}