)]}'
{"nova/tests/unit/virt/libvirt/test_config.py":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"987ea6babbb1c1ba07734eebe37e07f346f3a914","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        if type(actualXmlstr) is not six.text_type:"},{"line_number":28,"context_line":"            actualXmlstr \u003d actualXmlstr.decode()"},{"line_number":29,"context_line":"        if type(expectedXmlstr) is not six.text_type:"},{"line_number":30,"context_line":"            expectedXmlstr \u003d expectedXmlstr.decode()"},{"line_number":31,"context_line":"        self.assertThat(actualXmlstr, matchers.XMLMatches(expectedXmlstr))"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a061dce_ddd51bba","line":30,"range":{"start_line":30,"start_character":50,"end_line":30,"end_character":52},"updated":"2016-04-04 08:41:43.000000000","message":"This is not the right place to fix this - it must be done in the XMLMatches class itself as I\u0027v done here https://review.openstack.org/#/c/300509/1","commit_id":"bb8e5d2893202518b4a27d2a2dcf0e729726519e"},{"author":{"_account_id":18777,"name":"Jinxing Fang","email":"fang.jinxing@99cloud.net","username":"jinxingfang"},"change_message_id":"a71774da00bdaca420780b08e309c2188e7bfa52","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        if type(actualXmlstr) is not six.text_type:"},{"line_number":28,"context_line":"            actualXmlstr \u003d actualXmlstr.decode()"},{"line_number":29,"context_line":"        if type(expectedXmlstr) is not six.text_type:"},{"line_number":30,"context_line":"            expectedXmlstr \u003d expectedXmlstr.decode()"},{"line_number":31,"context_line":"        self.assertThat(actualXmlstr, matchers.XMLMatches(expectedXmlstr))"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_880be729","line":30,"range":{"start_line":30,"start_character":50,"end_line":30,"end_character":52},"in_reply_to":"9a061dce_ddd51bba","updated":"2016-04-14 02:30:09.000000000","message":"Agree with Daniel Berrange!","commit_id":"bb8e5d2893202518b4a27d2a2dcf0e729726519e"}],"nova/virt/libvirt/storage/rbd_utils.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"03b4adfbd3eff0bff3dd37c2a159e6e8b099e78d","unresolved":false,"context_lines":[{"line_number":167,"context_line":"        if not url.startswith(prefix):"},{"line_number":168,"context_line":"            reason \u003d _(\u0027Not stored in rbd\u0027)"},{"line_number":169,"context_line":"            raise exception.ImageUnacceptable(image_id\u003durl, reason\u003dreason)"},{"line_number":170,"context_line":"        pieces \u003d list(map(urllib.parse.unquote, url[len(prefix):].split(\u0027/\u0027)))"},{"line_number":171,"context_line":"        if \u0027\u0027 in pieces:"},{"line_number":172,"context_line":"            reason \u003d _(\u0027Blank components\u0027)"},{"line_number":173,"context_line":"            raise exception.ImageUnacceptable(image_id\u003durl, reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":20,"id":"bacf61ea_f141c80c","line":170,"updated":"2016-07-28 20:35:16.000000000","message":"I\u0027d suggest converting this to a list comprehension, rather than wrapping a map() in a list().","commit_id":"e7b0862e0e2c67ec05f3a4dde52c07509295f909"}]}