)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"56506892f7aacb5ae438593716c672c0ee47637f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eli Qiao \u003cliyong.qiao@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-02-28 17:29:22 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Libvirt: preallocate_images CONFIG can be arbitrary characters to enable preallocate images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We can now set preallocate_images as any charaters as lone as it is not equal to"},{"line_number":10,"context_line":"\u0027none\u0027 to enable preallocate images function, this is too bad."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba7be1f8_a6f13f6c","line":7,"updated":"2015-02-28 17:19:22.000000000","message":"you should limit the length here","commit_id":"dac273645616966ffd38ecb692c51897f6ac9e1f"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"c7fab750b83e705406716dafb9167852e12ca8b1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eli Qiao \u003cliyong.qiao@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-03-16 12:41:35 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Libvirt: preallocate_images CONFIG can be arbitrary characters to enable preallocate images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We can now set preallocate_images as any charaters as lone as it is not equal to"},{"line_number":10,"context_line":"\u0027none\u0027 to enable preallocate images function, this is too bad."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"da9b358b_3a16baf1","line":7,"updated":"2015-03-31 15:45:08.000000000","message":"Can you wrap these lines please?","commit_id":"cfd0d88b8a70f19b8f8c0f69ef6d0dfca0bb5b83"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"c7fab750b83e705406716dafb9167852e12ca8b1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Libvirt: preallocate_images CONFIG can be arbitrary characters to enable preallocate images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We can now set preallocate_images as any charaters as lone as it is not equal to"},{"line_number":10,"context_line":"\u0027none\u0027 to enable preallocate images function, this is too bad."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"this patch changes the condiation to equal to \u0027space\u0027/\u0027Space\u0027/\u0027SPACE\u0027"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"da9b358b_7ae32206","line":9,"range":{"start_line":9,"start_character":41,"end_line":9,"end_character":50},"updated":"2015-03-31 15:45:08.000000000","message":"\"characters\" is misspelled, and I think you mean \"as long\" not \"as lone\".","commit_id":"cfd0d88b8a70f19b8f8c0f69ef6d0dfca0bb5b83"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"c7fab750b83e705406716dafb9167852e12ca8b1","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We can now set preallocate_images as any charaters as lone as it is not equal to"},{"line_number":10,"context_line":"\u0027none\u0027 to enable preallocate images function, this is too bad."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"this patch changes the condiation to equal to \u0027space\u0027/\u0027Space\u0027/\u0027SPACE\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: 1427092"},{"line_number":15,"context_line":"Change-Id: Ie8f4861fb77d2c3c3a8976fb7768ced12e7d8d5b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"da9b358b_bab76af4","line":12,"range":{"start_line":12,"start_character":23,"end_line":12,"end_character":33},"updated":"2015-03-31 15:45:08.000000000","message":"\"condition\" is misspelled.","commit_id":"cfd0d88b8a70f19b8f8c0f69ef6d0dfca0bb5b83"}],"nova/virt/libvirt/imagebackend.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"56506892f7aacb5ae438593716c672c0ee47637f","unresolved":false,"context_lines":[{"line_number":384,"context_line":"        self.path \u003d (path or"},{"line_number":385,"context_line":"                     os.path.join(libvirt_utils.get_instance_path(instance),"},{"line_number":386,"context_line":"                                  disk_name))"},{"line_number":387,"context_line":"        self.preallocate \u003d("},{"line_number":388,"context_line":"            strutils.to_slug(CONF.preallocate_images) \u003d\u003d \u0027space\u0027)"},{"line_number":389,"context_line":"        self.disk_info_path \u003d os.path.join(os.path.dirname(self.path),"},{"line_number":390,"context_line":"                                           \u0027disk.info\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_06020b3e","line":387,"updated":"2015-02-28 17:19:22.000000000","message":"self.preallocate \u003d( \u003d\u003e self.preallocate \u003d (","commit_id":"dac273645616966ffd38ecb692c51897f6ac9e1f"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"56506892f7aacb5ae438593716c672c0ee47637f","unresolved":false,"context_lines":[{"line_number":385,"context_line":"                     os.path.join(libvirt_utils.get_instance_path(instance),"},{"line_number":386,"context_line":"                                  disk_name))"},{"line_number":387,"context_line":"        self.preallocate \u003d("},{"line_number":388,"context_line":"            strutils.to_slug(CONF.preallocate_images) \u003d\u003d \u0027space\u0027)"},{"line_number":389,"context_line":"        self.disk_info_path \u003d os.path.join(os.path.dirname(self.path),"},{"line_number":390,"context_line":"                                           \u0027disk.info\u0027)"},{"line_number":391,"context_line":"        self.correct_format()"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_46fc033f","line":388,"updated":"2015-02-28 17:19:22.000000000","message":"is there a unit test for this?","commit_id":"dac273645616966ffd38ecb692c51897f6ac9e1f"}]}