)]}'
{"nova/tests/unit/virt/ironic/test_driver.py":[{"author":{"_account_id":13295,"name":"Mario Villaplana","email":"mario.villaplana@gmail.com","username":"mariojv"},"change_message_id":"4b666ce0b0b40710ffd0ce6b4afca701c192dc41","unresolved":false,"context_lines":[{"line_number":1068,"context_line":"    @mock.patch.object(ironic_driver.IronicDriver, \u0027_cleanup_deploy\u0027)"},{"line_number":1069,"context_line":"    def test_spawn_node_configdrive_fail(self, mock_cleanup_deploy,"},{"line_number":1070,"context_line":"                                         mock_pvifs, mock_sf, mock_configdrive,"},{"line_number":1071,"context_line":"                                         mock_node, mock_save,"},{"line_number":1072,"context_line":"                                         mock_required_by):"},{"line_number":1073,"context_line":"        mock_required_by.return_value \u003d True"},{"line_number":1074,"context_line":"        node_uuid \u003d \u0027aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_277f2367","line":1071,"updated":"2016-03-03 15:43:43.000000000","message":"mock_save isn\u0027t  defined; the extra parameter is causing the test to fail.","commit_id":"adac960acd37213249d0cc790ff2859564f7fb95"},{"author":{"_account_id":13295,"name":"Mario Villaplana","email":"mario.villaplana@gmail.com","username":"mariojv"},"change_message_id":"4b666ce0b0b40710ffd0ce6b4afca701c192dc41","unresolved":false,"context_lines":[{"line_number":1093,"context_line":"                                               flavor\u003dflavor)"},{"line_number":1094,"context_line":""},{"line_number":1095,"context_line":"    @mock.patch.object(configdrive, \u0027required_by\u0027)"},{"line_number":1096,"context_line":"    @mock.patch.object(objects.Instance, \u0027save\u0027)"},{"line_number":1097,"context_line":"    @mock.patch.object(FAKE_CLIENT, \u0027node\u0027)"},{"line_number":1098,"context_line":"    @mock.patch.object(ironic_driver.IronicDriver, \u0027_start_firewall\u0027)"},{"line_number":1099,"context_line":"    @mock.patch.object(ironic_driver.IronicDriver, \u0027_plug_vifs\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_a7fe93ab","line":1096,"updated":"2016-03-03 15:43:43.000000000","message":"This doesn\u0027t have a corresponding parameter.","commit_id":"adac960acd37213249d0cc790ff2859564f7fb95"}],"nova/virt/ironic/driver.py":[{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"10fd3418190e04412ec3d5198e7fafdd8f40ba12","unresolved":false,"context_lines":[{"line_number":683,"context_line":"                    cdb.make_drive(uncompressed.name)"},{"line_number":684,"context_line":"            except Exception as e:"},{"line_number":685,"context_line":"                with excutils.save_and_reraise_exception():"},{"line_number":686,"context_line":"                    LOG.error(_LE(\"Creating config drive failed with \""},{"line_number":687,"context_line":"                                  \"error: %s\"), e, instance\u003dinstance)"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"            with tempfile.NamedTemporaryFile() as compressed:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_5cc18f53","line":686,"updated":"2016-03-03 03:50:31.000000000","message":"The clean up is necessary, but there is a very similar log here, it\u0027s kind of duplicate. do we need both of them?","commit_id":"adac960acd37213249d0cc790ff2859564f7fb95"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"b5e1cfaea5425e852b1a5d139a029150b2f12319","unresolved":false,"context_lines":[{"line_number":683,"context_line":"                    cdb.make_drive(uncompressed.name)"},{"line_number":684,"context_line":"            except Exception as e:"},{"line_number":685,"context_line":"                with excutils.save_and_reraise_exception():"},{"line_number":686,"context_line":"                    LOG.error(_LE(\"Creating config drive failed with \""},{"line_number":687,"context_line":"                                  \"error: %s\"), e, instance\u003dinstance)"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"            with tempfile.NamedTemporaryFile() as compressed:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_4de7e23f","line":686,"in_reply_to":"7af24918_5cc18f53","updated":"2016-03-03 20:30:54.000000000","message":"I removed this try;except in favor of the higher level one in spawn()","commit_id":"adac960acd37213249d0cc790ff2859564f7fb95"}]}