)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7c2f4d97a4ce8e9c45731f6555c416ed5cdaa9d4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vladik Romanovsky \u003cvromanso@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-09-08 14:24:00 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"cleanup: sparating the creation of a local root to it\u0027s own method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A trivial cleanup that speparates the creation of a local root disk"},{"line_number":10,"context_line":"and the injection of data into it to it\u0027s own method for easier futuer testing."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a89bdaa_92652f1f","line":7,"range":{"start_line":7,"start_character":9,"end_line":7,"end_character":18},"updated":"2016-09-08 18:54:38.000000000","message":"separate","commit_id":"b3db6445df2d209190f819d5996f36b1888c17ed"},{"author":{"_account_id":8802,"name":"Vladik Romanovsky","email":"vromanso@redhat.com","username":"vladikr"},"change_message_id":"171a5cc2e913f70976b933f6dcee27b4bd4461b9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vladik Romanovsky \u003cvromanso@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-09-08 14:24:00 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"cleanup: sparating the creation of a local root to it\u0027s own method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A trivial cleanup that speparates the creation of a local root disk"},{"line_number":10,"context_line":"and the injection of data into it to it\u0027s own method for easier futuer testing."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a89bdaa_2df6c68e","line":7,"range":{"start_line":7,"start_character":9,"end_line":7,"end_character":18},"in_reply_to":"9a89bdaa_92652f1f","updated":"2016-09-08 19:10:27.000000000","message":"Done","commit_id":"b3db6445df2d209190f819d5996f36b1888c17ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7c2f4d97a4ce8e9c45731f6555c416ed5cdaa9d4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"cleanup: sparating the creation of a local root to it\u0027s own method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A trivial cleanup that speparates the creation of a local root disk"},{"line_number":10,"context_line":"and the injection of data into it to it\u0027s own method for easier futuer testing."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ifea7f9f61ad9fd1a5e1995074790ec4ab63fd0ed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a89bdaa_f26e7bfe","line":10,"range":{"start_line":10,"start_character":64,"end_line":10,"end_character":70},"updated":"2016-09-08 18:54:38.000000000","message":"future","commit_id":"b3db6445df2d209190f819d5996f36b1888c17ed"},{"author":{"_account_id":8802,"name":"Vladik Romanovsky","email":"vromanso@redhat.com","username":"vladikr"},"change_message_id":"171a5cc2e913f70976b933f6dcee27b4bd4461b9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"cleanup: sparating the creation of a local root to it\u0027s own method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A trivial cleanup that speparates the creation of a local root disk"},{"line_number":10,"context_line":"and the injection of data into it to it\u0027s own method for easier futuer testing."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ifea7f9f61ad9fd1a5e1995074790ec4ab63fd0ed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a89bdaa_6dec4ebb","line":10,"range":{"start_line":10,"start_character":64,"end_line":10,"end_character":70},"in_reply_to":"9a89bdaa_f26e7bfe","updated":"2016-09-08 19:10:27.000000000","message":"Done","commit_id":"b3db6445df2d209190f819d5996f36b1888c17ed"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"8a42a62881303078e7107b3c7b2639754e06b460","unresolved":false,"context_lines":[{"line_number":7,"context_line":"cleanup: separate the creation of a local root to it\u0027s own method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A trivial cleanup that separates the creation of a local root disk"},{"line_number":10,"context_line":"and the injection of data into it to it\u0027s own method for easier future testing."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ifea7f9f61ad9fd1a5e1995074790ec4ab63fd0ed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a89bdaa_ed02eb6e","line":10,"updated":"2016-09-09 14:45:54.000000000","message":"I\u0027d really expect to see at least one test case added for this new method right now. The next change in this series is not adding a general test for this method - its just testing one specific scenario, so I think we need more in this commit.","commit_id":"f0bcc0de21443e349b25e1d5f3ca826a2c65696c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c8a9ca9f3dfa94f137af853c87b17410706e5616","unresolved":false,"context_lines":[{"line_number":7,"context_line":"cleanup: separate the creation of a local root to it\u0027s own method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A trivial cleanup that separates the creation of a local root disk"},{"line_number":10,"context_line":"and the injection of data into it to it\u0027s own method for easier future testing."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ifea7f9f61ad9fd1a5e1995074790ec4ab63fd0ed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a89bdaa_acbb6574","line":10,"in_reply_to":"9a89bdaa_ed02eb6e","updated":"2016-09-10 22:15:35.000000000","message":"This is just a clean refactor to pull the block out into it\u0027s own method to make testing the fix in the next patch in the series easier, I\u0027m not sure why we\u0027d need to add new tests for this. I actually prefer seeing the existing tests unchanged to show that there isn\u0027t a regression in the refactor.","commit_id":"f0bcc0de21443e349b25e1d5f3ca826a2c65696c"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2ee882dca00416af948e92fab351e81b31596e1c","unresolved":false,"context_lines":[{"line_number":3056,"context_line":"            if size \u003d\u003d 0 or suffix \u003d\u003d \u0027.rescue\u0027:"},{"line_number":3057,"context_line":"                size \u003d None"},{"line_number":3058,"context_line":""},{"line_number":3059,"context_line":"            backend \u003d self.image_backend.image(instance, \u0027disk\u0027 + suffix,"},{"line_number":3060,"context_line":"                                               CONF.libvirt.images_type)"},{"line_number":3061,"context_line":"            if instance.task_state \u003d\u003d task_states.RESIZE_FINISH:"},{"line_number":3062,"context_line":"                backend.create_snap(libvirt_utils.RESIZE_SNAPSHOT_NAME)"},{"line_number":3063,"context_line":"            if backend.SUPPORTS_CLONE:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a89bdaa_976b4f85","line":3060,"range":{"start_line":3059,"start_character":12,"end_line":3060,"end_character":72},"updated":"2016-09-09 02:30:56.000000000","message":"This is the only difference but it\u0027s the same as what was in the inline image method on L2938.","commit_id":"f0bcc0de21443e349b25e1d5f3ca826a2c65696c"}]}