)]}'
{"nova_powervm/tests/virt/powervm/test_driver.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ef2ba80fada834fa77e160f05ba6b60dddf9bb3d","unresolved":false,"context_lines":[{"line_number":347,"context_line":"                          \u0027remove_optical_time\u0027: 60},"},{"line_number":348,"context_line":"                         mock_pwron.call_args[1][\u0027add_parms\u0027])"},{"line_number":349,"context_line":"        mock_sanitize.assert_called_with("},{"line_number":350,"context_line":"            self.inst.name, prefix\u003dmock.ANY, suffix\u003dmock.ANY, max_len\u003dmock.ANY)"},{"line_number":351,"context_line":"        # Assert that tasks that are not supposed to be called are not called"},{"line_number":352,"context_line":"        self.assertFalse(mock_conn_vol.called)"},{"line_number":353,"context_line":"        self.assertFalse(mock_crt_cfg_drv.called)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_0b2ff6f6","line":350,"range":{"start_line":350,"start_character":28,"end_line":350,"end_character":78},"updated":"2017-01-26 18:05:05.000000000","message":"Why not fill these in?","commit_id":"26d2c865e17f5cbc45619d5455afd74c21c23daa"},{"author":{"_account_id":13557,"name":"Matt Rabe","email":"mdrabe@us.ibm.com","username":"mdrabe"},"change_message_id":"c76420b9a76738fa2983f00a7d6afd0e102f1471","unresolved":false,"context_lines":[{"line_number":347,"context_line":"                          \u0027remove_optical_time\u0027: 60},"},{"line_number":348,"context_line":"                         mock_pwron.call_args[1][\u0027add_parms\u0027])"},{"line_number":349,"context_line":"        mock_sanitize.assert_called_with("},{"line_number":350,"context_line":"            self.inst.name, prefix\u003dmock.ANY, suffix\u003dmock.ANY, max_len\u003dmock.ANY)"},{"line_number":351,"context_line":"        # Assert that tasks that are not supposed to be called are not called"},{"line_number":352,"context_line":"        self.assertFalse(mock_conn_vol.called)"},{"line_number":353,"context_line":"        self.assertFalse(mock_crt_cfg_drv.called)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a461143_c4eebbb3","line":350,"range":{"start_line":350,"start_character":28,"end_line":350,"end_character":78},"in_reply_to":"3a461143_0b2ff6f6","updated":"2017-01-26 19:03:44.000000000","message":"Done","commit_id":"26d2c865e17f5cbc45619d5455afd74c21c23daa"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"b13c5088576150c9b60664b3eec6a68c0799e2c5","unresolved":false,"context_lines":[{"line_number":367,"context_line":"        mock_cfg_drv.return_value \u003d True"},{"line_number":368,"context_line":"        mock_sanitize.return_value \u003d \u0027fake-name\u0027"},{"line_number":369,"context_line":"        vopt_rm_opt_orig \u003d driver.CONF.powervm.remove_vopt_media_on_boot"},{"line_number":370,"context_line":"        driver.CONF.powervm.remove_vopt_media_on_boot \u003d True"},{"line_number":371,"context_line":""},{"line_number":372,"context_line":"        # Invoke the method."},{"line_number":373,"context_line":"        self.drv.spawn(\u0027context\u0027, self.inst, powervm.IMAGE1,"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa31d9ce_b43c50c9","line":370,"updated":"2017-02-13 22:39:47.000000000","message":"We use self.flags() for this.","commit_id":"383adaa0170287c8a91e4d48eb0ed1b069e4b4ed"}],"nova_powervm/virt/powervm/driver.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"b13c5088576150c9b60664b3eec6a68c0799e2c5","unresolved":false,"context_lines":[{"line_number":473,"context_line":"        # Save the slot map information"},{"line_number":474,"context_line":"        flow_spawn.add(tf_slot.SaveSlotStore(instance, slot_mgr))"},{"line_number":475,"context_line":""},{"line_number":476,"context_line":"        pwr_opts \u003d {}"},{"line_number":477,"context_line":"        if CONF.powervm.remove_vopt_media_on_boot:"},{"line_number":478,"context_line":"            # Get the cfgdrive media name for the vopt removal task in PowerOn."},{"line_number":479,"context_line":"            media_name \u003d pvm_util.sanitize_file_name_for_api("}],"source_content_type":"text/x-python","patch_set":5,"id":"fa31d9ce_94710cab","line":476,"updated":"2017-02-13 22:39:47.000000000","message":"Use a power_opts.PowerOnOpts here instead.\n\n pwr_opts \u003d power_opts.PowerOnOpts()\n if CONF....:\n     media_name \u003d ...\n     pwr_opts.remove_optical(media_name, time\u003dCONF...)","commit_id":"383adaa0170287c8a91e4d48eb0ed1b069e4b4ed"}]}