)]}'
{"tempest/services/compute/json/version_client.py":[{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"5ae0c150ee7304f4776b3d82d78b06a79936fec6","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    def get_version(self):"},{"line_number":31,"context_line":"        resp, body \u003d self.get(\u0027/\u0027)"},{"line_number":32,"context_line":"        body \u003d json.loads(body)"},{"line_number":33,"context_line":"        return resp, body[\u0027version\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUX%2F%2FGBI%3D","line":32,"updated":"2014-02-17 07:12:54.000000000","message":"Body of this method can be next:\n\nresp, body \u003d self.get(\u0027/\u0027)\nrestun resp, self._parse_resp(body)\n\nAfter this, xml rest client can just inherit this class and redefine self.TYPE to \"xml\"","commit_id":"3685f3ccf0dedc35fa2e3ab6ee9fe860b1857bc7"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"237a9f192428db5e1dfad65caef612b7f60d3bfb","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import json"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest.common import rest_client"},{"line_number":19,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUX%2F%2FFu4%3D","line":16,"updated":"2014-02-17 07:48:36.000000000","message":"Oh, one thins else, it is redundant with current code.","commit_id":"73c4d8cc549b9e8d938e0c94c46cc95d82fff31c"}],"tempest/services/compute/xml/version_client.py":[{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"b7fb11f299f0a4d07b33e675fd1834ec98febd59","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        return json"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def get_version(self):"},{"line_number":36,"context_line":"        resp, body \u003d self.get(\u0027\u0027, self.headers)"},{"line_number":37,"context_line":"        body \u003d self._parse_array(etree.fromstring(body))"},{"line_number":38,"context_line":"        return resp, body"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUX%2F%2FGDw%3D","line":36,"updated":"2014-02-17 07:04:54.000000000","message":"There is no need to use self.headers anymore.","commit_id":"4043d7d37c7aeaa0fa3ce01a12c9bfe542f4aa37"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"5ae0c150ee7304f4776b3d82d78b06a79936fec6","unresolved":false,"context_lines":[{"line_number":22,"context_line":"CONF \u003d config.CONF"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class VersionClientXML(rest_client.RestClientXML):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def __init__(self, auth_provider):"},{"line_number":28,"context_line":"        super(VersionClientXML, self).__init__(auth_provider)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUX%2F%2FGAE%3D","line":25,"updated":"2014-02-17 07:12:54.000000000","message":"See comment in \ntempest/services/compute/json/version_client.py\n\nwe can have here next:\n\nfrom tempest.services.compute.json import version_client\n\n\nclass VersionClientXML(version_client.VersionClientJSON):\n    TYPE \u003d \"xml\"","commit_id":"3685f3ccf0dedc35fa2e3ab6ee9fe860b1857bc7"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1463728a163dee7826331770123a800a8437524f","unresolved":false,"context_lines":[{"line_number":22,"context_line":"CONF \u003d config.CONF"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class VersionClientXML(rest_client.RestClientXML):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def __init__(self, auth_provider):"},{"line_number":28,"context_line":"        super(VersionClientXML, self).__init__(auth_provider)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUX%2F%2FF34%3D","line":25,"in_reply_to":"AAAAUX%2F%2FGAE%3D","updated":"2014-02-17 07:25:12.000000000","message":"cool, good point.","commit_id":"3685f3ccf0dedc35fa2e3ab6ee9fe860b1857bc7"}]}