)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"37067758f652d2f5e4b2f628b84290c456ed564c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-04 00:01:24 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added missing raise statement when checking the config driver format"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-bug: 1275772"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F9dPI%3D","line":7,"updated":"2014-02-06 19:38:23.000000000","message":"nit: should have hyper-v in the commit title for context","commit_id":"c6aba9b1fc415bd1c8d589a84a356234549f60ae"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"38fb905539bb2e32e81b2464c06dd63d41874052","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     7d95d729 (Merge \"Correct host managers free disk calculation\")"},{"line_number":2,"context_line":"Author:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2014-02-03 11:42:22 -0800"},{"line_number":4,"context_line":"Commit:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-14 07:25:01 -0800"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAU3%2F%2FerY%3D","line":2,"updated":"2014-02-26 11:45:01.000000000","message":"Please set your real name","commit_id":"7d206b05c821723f62fc98a6b0b93527cdded801"},{"author":{"_account_id":3185,"name":"Alessandro Pilotti","email":"apilotti@cloudbasesolutions.com","username":"alexpilotti"},"change_message_id":"e20d72fdf190e11c12a68b55d3348a61a25b4e59","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     7d95d729 (Merge \"Correct host managers free disk calculation\")"},{"line_number":2,"context_line":"Author:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2014-02-03 11:42:22 -0800"},{"line_number":4,"context_line":"Commit:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-14 07:25:01 -0800"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAU3%2F%2FeoU%3D","line":2,"in_reply_to":"AAAAU3%2F%2FerY%3D","updated":"2014-02-26 11:49:36.000000000","message":"Good catch, tx!","commit_id":"7d206b05c821723f62fc98a6b0b93527cdded801"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"38fb905539bb2e32e81b2464c06dd63d41874052","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     7d95d729 (Merge \"Correct host managers free disk calculation\")"},{"line_number":2,"context_line":"Author:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2014-02-03 11:42:22 -0800"},{"line_number":4,"context_line":"Commit:     root \u003cavladu@cloudbasesolutions.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-14 07:25:01 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added missing raise statement when checking the config driver format"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAU3%2F%2FerQ%3D","line":4,"updated":"2014-02-26 11:45:01.000000000","message":"And again","commit_id":"7d206b05c821723f62fc98a6b0b93527cdded801"}],"nova/tests/virt/hyperv/test_hypervapi.py":[{"author":{"_account_id":8430,"name":"Christopher Lefelhocz","email":"christopher.lefelhoc@rackspace.com","username":"leif"},"change_message_id":"9f69f27c5a2c53afd5642002892ef983c1d01dd6","unresolved":false,"context_lines":[{"line_number":431,"context_line":"            expected_ide_disks \u003d 2"},{"line_number":432,"context_line":"            expected_ide_dvds \u003d 0"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"        self._test_spawn_instance(expected_ide_disks\u003dexpected_ide_disks,"},{"line_number":435,"context_line":"                                  expected_ide_dvds\u003dexpected_ide_dvds,"},{"line_number":436,"context_line":"                                  config_drive\u003dTrue,"},{"line_number":437,"context_line":"                                  use_cdrom\u003duse_cdrom,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUX%2F%2FWhw%3D","line":434,"updated":"2014-02-13 19:43:26.000000000","message":"It seems like having the assertRaises here makes more sense.","commit_id":"283a180d6bbbcc520b2da85113f6b025d2184a3f"},{"author":{"_account_id":8430,"name":"Christopher Lefelhocz","email":"christopher.lefelhoc@rackspace.com","username":"leif"},"change_message_id":"9f69f27c5a2c53afd5642002892ef983c1d01dd6","unresolved":false,"context_lines":[{"line_number":870,"context_line":"                        instance\u003dNone):"},{"line_number":871,"context_line":"        self.flags(use_cow_images\u003dcow)"},{"line_number":872,"context_line":""},{"line_number":873,"context_line":"        if not config_drive_error:"},{"line_number":874,"context_line":"            self._instance_data \u003d self._get_instance_data()"},{"line_number":875,"context_line":"        else:"},{"line_number":876,"context_line":"            self._instance_data \u003d instance"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUX%2F%2FWhU%3D","line":873,"updated":"2014-02-13 19:43:26.000000000","message":"Why is instance_data different here?  It\u0027s not what causes the error right?","commit_id":"283a180d6bbbcc520b2da85113f6b025d2184a3f"},{"author":{"_account_id":8430,"name":"Christopher Lefelhocz","email":"christopher.lefelhoc@rackspace.com","username":"leif"},"change_message_id":"9f69f27c5a2c53afd5642002892ef983c1d01dd6","unresolved":false,"context_lines":[{"line_number":890,"context_line":"                             network_info\u003dnetwork_info,"},{"line_number":891,"context_line":"                             block_device_info\u003dblock_device_info)"},{"line_number":892,"context_line":"        else:"},{"line_number":893,"context_line":"            self.assertRaises(vmutils.HyperVException, self._conn.spawn,"},{"line_number":894,"context_line":"                              self._context, instance, image, [], None,"},{"line_number":895,"context_line":"                              network_info, block_device_info)"},{"line_number":896,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUX%2F%2FWfg%3D","line":893,"updated":"2014-02-13 19:43:26.000000000","message":"If you move the assertRaises up, you should be able to not pass down config_driver_error and have if/else block?","commit_id":"283a180d6bbbcc520b2da85113f6b025d2184a3f"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"1a8ac56d68c9acd4e8274a8b671b5c5da1186e7a","unresolved":false,"context_lines":[{"line_number":432,"context_line":"            expected_ide_dvds \u003d 0"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"        if format_error:"},{"line_number":435,"context_line":"            self.assertRaises(vmutils.HyperVException,"},{"line_number":436,"context_line":"                              self._test_spawn_instance,"},{"line_number":437,"context_line":"                              with_exception\u003dTrue,"},{"line_number":438,"context_line":"                              config_drive\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAWH%2F%2FegU%3D","line":435,"updated":"2014-03-17 17:14:01.000000000","message":"Shouldn\u0027t you check that the proper (new) exception is raised?","commit_id":"98942a72c08611f3527ce490aed7d384fb819f3f"},{"author":{"_account_id":9649,"name":"Adrian Vladu","email":"avladu@cloudbasesolutions.com","username":"avladu"},"change_message_id":"713f729c895452a6d2b5c44609e020545a723885","unresolved":false,"context_lines":[{"line_number":432,"context_line":"            expected_ide_dvds \u003d 0"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"        if format_error:"},{"line_number":435,"context_line":"            self.assertRaises(vmutils.HyperVException,"},{"line_number":436,"context_line":"                              self._test_spawn_instance,"},{"line_number":437,"context_line":"                              with_exception\u003dTrue,"},{"line_number":438,"context_line":"                              config_drive\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAWH%2F%2FeNo%3D","line":435,"in_reply_to":"AAAAWH%2F%2FegU%3D","updated":"2014-03-17 17:52:28.000000000","message":"Done.","commit_id":"98942a72c08611f3527ce490aed7d384fb819f3f"},{"author":{"_account_id":9649,"name":"Adrian Vladu","email":"avladu@cloudbasesolutions.com","username":"avladu"},"change_message_id":"7aa72af9074503db555db9eee706159fbc0d6a06","unresolved":false,"context_lines":[{"line_number":436,"context_line":"            expected_ide_dvds \u003d 0"},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"        if format_error:"},{"line_number":439,"context_line":"            self.assertRaises(vmutils.UnsupportedConfigDriveFormatException,"},{"line_number":440,"context_line":"                              self._test_spawn_instance,"},{"line_number":441,"context_line":"                              with_exception\u003dTrue,"},{"line_number":442,"context_line":"                              config_drive\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAWH%2F%2FeK8%3D","line":439,"updated":"2014-03-17 17:50:07.000000000","message":"Done.","commit_id":"378c28a8c05d3943324b5a39edef947ed7a9f52a"}],"nova/virt/hyperv/vmops.py":[{"author":{"_account_id":3185,"name":"Alessandro Pilotti","email":"apilotti@cloudbasesolutions.com","username":"alexpilotti"},"change_message_id":"8c235a648bb78eaf2e2c784fa6f8d732ebae5de0","unresolved":false,"context_lines":[{"line_number":278,"context_line":""},{"line_number":279,"context_line":"    def _create_config_drive(self, instance, injected_files, admin_password):"},{"line_number":280,"context_line":"        if CONF.config_drive_format !\u003d \u0027iso9660\u0027:"},{"line_number":281,"context_line":"            raise vmutils.HyperVException("},{"line_number":282,"context_line":"                _(\u0027Invalid config_drive_format \"%s\"\u0027) %"},{"line_number":283,"context_line":"                CONF.config_drive_format)"},{"line_number":284,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUX%2F%2FQg0%3D","line":281,"updated":"2014-02-14 13:41:28.000000000","message":"Please define a more specialized Exception class for this inheriting from HyperVException.","commit_id":"283a180d6bbbcc520b2da85113f6b025d2184a3f"}]}