)]}'
{"nova/tests/unit/virt/libvirt/test_firewall.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d92f9951f923f48e57db029255e2a120e3c12111","unresolved":false,"context_lines":[{"line_number":742,"context_line":"            self.fw._get_filter_uuid(\u0027test\u0027)"},{"line_number":743,"context_line":"        self.assertEqual(2, debug.call_count)"},{"line_number":744,"context_line":"        self.assertEqual(u\"Cannot find UUID for filter \u0027test\u0027: \u0027\\U0001F4A9\u0027\","},{"line_number":745,"context_line":"                         debug.call_args_list[0][0][0])"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_edd6efc2","line":745,"updated":"2015-03-10 20:38:02.000000000","message":"I don\u0027t think the test will work with the way you\u0027re passing the args dict for replacement variables in the message, it worked in my change since I was doing % with the message and vars.","commit_id":"889f5407c4c6df23a4bcd2daa31898a931585e7c"}],"nova/virt/libvirt/firewall.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"52598d5ea28062fb6e544bf3742131860ba98af1","unresolved":false,"context_lines":[{"line_number":251,"context_line":"            doc \u003d etree.fromstring(xml)"},{"line_number":252,"context_line":"            u \u003d doc.find(\"./uuid\").text"},{"line_number":253,"context_line":"        except Exception as e:"},{"line_number":254,"context_line":"            LOG.debug(u\"Cannot find UUID for filter \u0027%(name)s: \u0027%(e)s\u0027\","},{"line_number":255,"context_line":"                {\u0027name\u0027: name, \u0027e\u0027: e})"},{"line_number":256,"context_line":"            u \u003d uuid.uuid4().hex"},{"line_number":257,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_a73e46d8","line":254,"updated":"2015-03-07 00:07:57.000000000","message":"I learned by asking Joe this u\" is required because e is a libvirt error which may be unicode because it is translated. I don\u0027t know if we should have a comment here (or is it common knowledge?) It also makes me wonder if there are any other libvirt e\u0027s that need u\" in the log (wish we could detect it automatically somehow)","commit_id":"be623fdf0a6c2f89c10ac415947bd25b271771bc"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"895e86031e59e02ebf36b33726540fa77f180048","unresolved":false,"context_lines":[{"line_number":251,"context_line":"            doc \u003d etree.fromstring(xml)"},{"line_number":252,"context_line":"            u \u003d doc.find(\"./uuid\").text"},{"line_number":253,"context_line":"        except Exception as e:"},{"line_number":254,"context_line":"            LOG.debug(u\"Cannot find UUID for filter \u0027%(name)s: \u0027%(e)s\u0027\","},{"line_number":255,"context_line":"                {\u0027name\u0027: name, \u0027e\u0027: e})"},{"line_number":256,"context_line":"            u \u003d uuid.uuid4().hex"},{"line_number":257,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_e523cca4","line":254,"in_reply_to":"9a80dd14_a73e46d8","updated":"2015-03-07 22:47:54.000000000","message":"Melanie, being hotly debated in oslo.log migration notes - https://review.openstack.org/#/c/147312/","commit_id":"be623fdf0a6c2f89c10ac415947bd25b271771bc"}]}