)]}'
{"nova/virt/libvirt/driver.py":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"5b8459824bd896ee44d8ce969ad2703829cd9ce1","unresolved":false,"context_lines":[{"line_number":869,"context_line":"                self.firewall_driver.unfilter_instance("},{"line_number":870,"context_line":"                    instance,"},{"line_number":871,"context_line":"                    network_info\u003dnetwork_info"},{"line_number":872,"context_line":"                )"},{"line_number":873,"context_line":"            except libvirt.libvirtError as e:"},{"line_number":874,"context_line":"                try:"},{"line_number":875,"context_line":"                    state \u003d self.get_info(instance)[\u0027state\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2F1uM%3D","line":872,"updated":"2013-06-20 01:49:23.000000000","message":"This is an unrelated formatting change.  Please revert this (and anything else that\u0027s unrelated if there\u0027s more).","commit_id":"7ac3465bc49d92b1ba8133a4d75e6fea9208a91a"},{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"32ce3f845425275fb139911cddf1164fbbbe9754","unresolved":false,"context_lines":[{"line_number":868,"context_line":"            try:"},{"line_number":869,"context_line":"                self.firewall_driver.unfilter_instance(instance,"},{"line_number":870,"context_line":"                                                    network_info\u003dnetwork_info)"},{"line_number":871,"context_line":""},{"line_number":872,"context_line":"            except libvirt.libvirtError as e:"},{"line_number":873,"context_line":"                try:"},{"line_number":874,"context_line":"                    state \u003d self.get_info(instance)[\u0027state\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2F0b0%3D","line":871,"updated":"2013-06-20 14:06:43.000000000","message":"unrelated formatting change here","commit_id":"17f010666b64533478a52b835d2497c6ce6d9448"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"4f3433c64254ed7094dc1a282e79b745076146b2","unresolved":false,"context_lines":[{"line_number":868,"context_line":"            try:"},{"line_number":869,"context_line":"                self.firewall_driver.unfilter_instance(instance,"},{"line_number":870,"context_line":"                                                    network_info\u003dnetwork_info)"},{"line_number":871,"context_line":""},{"line_number":872,"context_line":"            except libvirt.libvirtError as e:"},{"line_number":873,"context_line":"                try:"},{"line_number":874,"context_line":"                    state \u003d self.get_info(instance)[\u0027state\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2F0X0%3D","line":871,"in_reply_to":"AAAAPn%2F%2F0b0%3D","updated":"2013-06-20 14:57:28.000000000","message":"oops, missed that one","commit_id":"17f010666b64533478a52b835d2497c6ce6d9448"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7791c15c1493388ff56730066ea723da2868f539","unresolved":false,"context_lines":[{"line_number":919,"context_line":"                try:"},{"line_number":920,"context_line":"                    shutil.rmtree(target)"},{"line_number":921,"context_line":"                except OSError as e:"},{"line_number":922,"context_line":"                    LOG.error(_(\u0027Failed to cleanup directory %(targ)s: %(e)s\u0027),"},{"line_number":923,"context_line":"                              {\u0027targ\u0027: target, \u0027e\u0027: e})"},{"line_number":924,"context_line":""},{"line_number":925,"context_line":"            #NOTE(bfilippov): destroy all LVM disks for this instance"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F0NQ%3D","line":922,"updated":"2013-06-20 15:22:55.000000000","message":"Why did this change from target to targ?  Was the line too long?","commit_id":"de3259c48a63719a0110bda6844280efaad5a1d6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4944aa5f4c96aa59372babd59914edc219d9b279","unresolved":false,"context_lines":[{"line_number":919,"context_line":"                try:"},{"line_number":920,"context_line":"                    shutil.rmtree(target)"},{"line_number":921,"context_line":"                except OSError as e:"},{"line_number":922,"context_line":"                    LOG.error(_(\u0027Failed to cleanup directory %(targ)s: %(e)s\u0027),"},{"line_number":923,"context_line":"                              {\u0027targ\u0027: target, \u0027e\u0027: e})"},{"line_number":924,"context_line":""},{"line_number":925,"context_line":"            #NOTE(bfilippov): destroy all LVM disks for this instance"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FztI%3D","line":922,"in_reply_to":"AAAAPn%2F%2F0II%3D","updated":"2013-06-20 19:03:52.000000000","message":"Personally I would have probably left the original names and split it to a second line.","commit_id":"de3259c48a63719a0110bda6844280efaad5a1d6"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"8f56fcbf371d8ec97fc801cf4033b80901729338","unresolved":false,"context_lines":[{"line_number":919,"context_line":"                try:"},{"line_number":920,"context_line":"                    shutil.rmtree(target)"},{"line_number":921,"context_line":"                except OSError as e:"},{"line_number":922,"context_line":"                    LOG.error(_(\u0027Failed to cleanup directory %(targ)s: %(e)s\u0027),"},{"line_number":923,"context_line":"                              {\u0027targ\u0027: target, \u0027e\u0027: e})"},{"line_number":924,"context_line":""},{"line_number":925,"context_line":"            #NOTE(bfilippov): destroy all LVM disks for this instance"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2F0II%3D","line":922,"in_reply_to":"AAAAPn%2F%2F0NQ%3D","updated":"2013-06-20 15:43:40.000000000","message":"yes. Do you think it worth splitting the message in two lines to preserve the placeholder name?","commit_id":"de3259c48a63719a0110bda6844280efaad5a1d6"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"bc6f284ae609ed6169dc981a6270fa987aa92a9d","unresolved":false,"context_lines":[{"line_number":919,"context_line":"                try:"},{"line_number":920,"context_line":"                    shutil.rmtree(target)"},{"line_number":921,"context_line":"                except OSError as e:"},{"line_number":922,"context_line":"                    LOG.error(_(\u0027Failed to cleanup directory %(targ)s: %(e)s\u0027),"},{"line_number":923,"context_line":"                              {\u0027targ\u0027: target, \u0027e\u0027: e})"},{"line_number":924,"context_line":""},{"line_number":925,"context_line":"            #NOTE(bfilippov): destroy all LVM disks for this instance"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2Fy2I%3D","line":922,"in_reply_to":"AAAAPn%2F%2FztI%3D","updated":"2013-06-21 04:03:08.000000000","message":"Done","commit_id":"de3259c48a63719a0110bda6844280efaad5a1d6"}]}