)]}'
{"tempest/services/compute/json/servers_client.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"6e71a7af9cda172f9f2084bddb84f401b8b64b01","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        post_body \u003d json.dumps({action_name: kwargs})"},{"line_number":204,"context_line":"        resp, body \u003d self.post(\u0027servers/%s/action\u0027 % str(server_id),"},{"line_number":205,"context_line":"                               post_body)"},{"line_number":206,"context_line":"        if response_key is not None:"},{"line_number":207,"context_line":"            body \u003d json.loads(body)"},{"line_number":208,"context_line":"            # Check for Schema as \u0027None\u0027 because if we donot have any server"},{"line_number":209,"context_line":"            # action schema implemented yet then they can pass \u0027None\u0027 to skip"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2Fi0E%3D","line":206,"updated":"2014-04-13 06:44:33.000000000","message":"if there isn\u0027t any response, we also need checking the status code? But it doesn\u0027t look like this patch scope.","commit_id":"d6d30404020cb5db5cb55bb7a836971c3a750d0b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"597a802dacf6be85b8d386c8b3e23f20395338b4","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        post_body \u003d json.dumps({action_name: kwargs})"},{"line_number":204,"context_line":"        resp, body \u003d self.post(\u0027servers/%s/action\u0027 % str(server_id),"},{"line_number":205,"context_line":"                               post_body)"},{"line_number":206,"context_line":"        if response_key is not None:"},{"line_number":207,"context_line":"            body \u003d json.loads(body)"},{"line_number":208,"context_line":"            # Check for Schema as \u0027None\u0027 because if we donot have any server"},{"line_number":209,"context_line":"            # action schema implemented yet then they can pass \u0027None\u0027 to skip"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2FiiA%3D","line":206,"in_reply_to":"AAAAW3%2F%2Fi0E%3D","updated":"2014-04-13 11:04:39.000000000","message":"Thanks for the review. yes, in case of no response key passed, we should validate the status code and it is being released in another patch - https://review.openstack.org/#/c/84380/ (which needs rebase and restore.)","commit_id":"d6d30404020cb5db5cb55bb7a836971c3a750d0b"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"ab84a16957db6c86ee0f189e79927b1cce665eaf","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        post_body \u003d json.dumps({action_name: kwargs})"},{"line_number":204,"context_line":"        resp, body \u003d self.post(\u0027servers/%s/action\u0027 % str(server_id),"},{"line_number":205,"context_line":"                               post_body)"},{"line_number":206,"context_line":"        if response_key is not None:"},{"line_number":207,"context_line":"            body \u003d json.loads(body)"},{"line_number":208,"context_line":"            # Check for Schema as \u0027None\u0027 because if we donot have any server"},{"line_number":209,"context_line":"            # action schema implemented yet then they can pass \u0027None\u0027 to skip"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAW3%2F%2FgmA%3D","line":206,"in_reply_to":"AAAAW3%2F%2FiiA%3D","updated":"2014-04-14 05:17:58.000000000","message":"thanks for the explaination","commit_id":"d6d30404020cb5db5cb55bb7a836971c3a750d0b"}]}