)]}'
{"tempest/tests/compute/test_live_block_migration.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"2c073d59be45082f8d78a7a7ed47ae33be49187e","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            for (key, value) in self._get_server_details(server_id).items()"},{"line_number":138,"context_line":"            if key.endswith(\u0027}host\u0027)]"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"        assert 1 \u003d\u003d len(hosts)"},{"line_number":141,"context_line":"        return hosts[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOH%2F%2FtBA%3D","line":140,"updated":"2013-04-02 11:13:42.000000000","message":"we don\u0027t use raw asserts, if you need to check something here use the test assert, otherwise you blow up randomly in the middle of a test.","commit_id":"7c25de54a367fb1d81e2cb91a948b4995a31a2d4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5d0658cd49acc00f02e66d76bf05dfb243bae4b9","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        _resp, body \u003d self.admin_servers_client.get_server(server_id)"},{"line_number":58,"context_line":"        return body"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def _get_host_for_server(self, server_id):"},{"line_number":61,"context_line":"        return self._get_server_details(server_id)[\u0027OS-EXT-SRV-ATTR:host\u0027]"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _migrate_server_to(self, server_id, dest_host):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOX%2F%2F%2B6o%3D","line":60,"updated":"2013-04-03 16:29:02.000000000","message":"Could we not just make this key a variable that is given a different value in the XML case?","commit_id":"39bc2472d5de7f073bcc7183b3b910d7b61313af"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"c92f8c43cd18e1661ce017800d140ef2f359c3df","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        _resp, body \u003d self.admin_servers_client.get_server(server_id)"},{"line_number":58,"context_line":"        return body"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def _get_host_for_server(self, server_id):"},{"line_number":61,"context_line":"        return self._get_server_details(server_id)[\u0027OS-EXT-SRV-ATTR:host\u0027]"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def _migrate_server_to(self, server_id, dest_host):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOX%2F%2F%2B4c%3D","line":60,"in_reply_to":"AAAAOX%2F%2F%2B6o%3D","updated":"2013-04-03 16:53:41.000000000","message":"The problem is, that I am not sure, whether this key is stable enough, in terms of API versions. So in the XML case, the key would be:\n\n    {http://docs.openstack.org/compute/ext/extended_status/api/v1.1}host\n\nWhich depends on the API revision number (it is actually an XML schema uri, as I understood)\n\nSo that\u0027s why I picked the other route. Maybe that\u0027s an overkill. I am still not sure, if seeing the XML namespace here is not a bug.","commit_id":"39bc2472d5de7f073bcc7183b3b910d7b61313af"}]}