)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"8d8a87b3d17705ded948e99430c8a784e28dca19","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sean M. Collins \u003csean@coreitpro.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2012-12-20 11:35:05 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove instance_name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Portions of the code use instance_name, while others use"},{"line_number":10,"context_line":"instance[\u0027name\u0027]. In some cases, within the same function. See"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAM3%2F%2F6n4%3D","line":7,"updated":"2012-12-20 23:40:16.000000000","message":"Please clarify commit header line. Perhaps something like:\n\nRemove inconsistent usage of instance_name from hyperv.","commit_id":"c9ecfb3166fd9aa4a51f5816c4c040869a79f796"}],"nova/virt/hyperv/vmops.py":[{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"2c65c84e928fd2c5b0b97aa560c4a23b4d4169f9","unresolved":false,"context_lines":[{"line_number":496,"context_line":"            vhdfile \u003d self._conn_cimv2.query("},{"line_number":497,"context_line":"            \"Select * from CIM_DataFile where Name \u003d \u0027\" +"},{"line_number":498,"context_line":"                disk.replace(\"\u0027\", \"\u0027\u0027\") + \"\u0027\")[0]"},{"line_number":499,"context_line":"            LOG.debug(_(\"Del: disk {vhdfile!s} vm {instance[\u0027name\u0027]}\"))"},{"line_number":500,"context_line":"            vhdfile.Delete()"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"    def pause(self, instance):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2F6IM%3D","line":499,"updated":"2012-12-21 16:43:53.000000000","message":"It looks like you removed the variable substitution, which should be outside of i18n anyway.\n\n  LOG.debug(_(\"Del: disk %(vhdfile)s vm %(name)s\")\n    % {\u0027vhdfile\u0027:vhdfile, \u0027name\u0027:instance[\u0027name\u0027]})","commit_id":"4cfafcbb37200c5b93827827d889b926f9cfae59"},{"author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"change_message_id":"46af46d09a8ad32cbdfe8f9d6ef53e1eebfbb898","unresolved":false,"context_lines":[{"line_number":496,"context_line":"            vhdfile \u003d self._conn_cimv2.query("},{"line_number":497,"context_line":"            \"Select * from CIM_DataFile where Name \u003d \u0027\" +"},{"line_number":498,"context_line":"                disk.replace(\"\u0027\", \"\u0027\u0027\") + \"\u0027\")[0]"},{"line_number":499,"context_line":"            LOG.debug(_(\"Del: disk {vhdfile!s} vm {instance[\u0027name\u0027]}\"))"},{"line_number":500,"context_line":"            vhdfile.Delete()"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"    def pause(self, instance):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2F6Ec%3D","line":499,"in_reply_to":"AAAAM3%2F%2F6IM%3D","updated":"2012-12-21 16:59:10.000000000","message":"Good point. This part of the patch was the piece I was the least certain about. I think your solution is far better.","commit_id":"4cfafcbb37200c5b93827827d889b926f9cfae59"}]}