)]}'
{"ironic_lib/disk_utils.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"ba2e3f51c4e033577cd615a3b97bcf66289d1b20","unresolved":false,"context_lines":[{"line_number":356,"context_line":"                      run_as_root\u003dTrue,"},{"line_number":357,"context_line":"                      use_standard_locale\u003dTrue)"},{"line_number":358,"context_line":"    except processutils.ProcessExecutionError as e:"},{"line_number":359,"context_line":"        # NOTE(zhenguo): Check if --force option is supported for wipefs,"},{"line_number":360,"context_line":"        # if not, we should try without it."},{"line_number":361,"context_line":"        if \u0027--force\u0027 in str(e):"},{"line_number":362,"context_line":"            utils.execute(\u0027wipefs\u0027, \u0027--all\u0027, dev,"},{"line_number":363,"context_line":"                          run_as_root\u003dTrue,"},{"line_number":364,"context_line":"                          use_standard_locale\u003dTrue)"},{"line_number":365,"context_line":"        else:"},{"line_number":366,"context_line":"            raise"},{"line_number":367,"context_line":"    LOG.info(_LI(\"Disk metadata on %(dev)s successfully destroyed for node \""},{"line_number":368,"context_line":"                 \"%(node)s\"), {\u0027dev\u0027: dev, \u0027node\u0027: node_uuid})"},{"line_number":369,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_4efac2be","line":366,"range":{"start_line":359,"start_character":8,"end_line":366,"end_character":17},"updated":"2016-12-02 15:55:23.000000000","message":"Actually, this bit needs to use excutils.save_and_reraise_exception, to properly log an exception that occured if wipefs fails too.","commit_id":"c5ad7ebc0c1c9ad5a58094677a977f706cf20f48"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"7a3d2dfa9cde99d24adc43f8249987fe8778f216","unresolved":false,"context_lines":[{"line_number":356,"context_line":"                      run_as_root\u003dTrue,"},{"line_number":357,"context_line":"                      use_standard_locale\u003dTrue)"},{"line_number":358,"context_line":"    except processutils.ProcessExecutionError as e:"},{"line_number":359,"context_line":"        # NOTE(zhenguo): Check if --force option is supported for wipefs,"},{"line_number":360,"context_line":"        # if not, we should try without it."},{"line_number":361,"context_line":"        if \u0027--force\u0027 in str(e):"},{"line_number":362,"context_line":"            utils.execute(\u0027wipefs\u0027, \u0027--all\u0027, dev,"},{"line_number":363,"context_line":"                          run_as_root\u003dTrue,"},{"line_number":364,"context_line":"                          use_standard_locale\u003dTrue)"},{"line_number":365,"context_line":"        else:"},{"line_number":366,"context_line":"            raise"},{"line_number":367,"context_line":"    LOG.info(_LI(\"Disk metadata on %(dev)s successfully destroyed for node \""},{"line_number":368,"context_line":"                 \"%(node)s\"), {\u0027dev\u0027: dev, \u0027node\u0027: node_uuid})"},{"line_number":369,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a6eadb0_55bc3105","line":366,"range":{"start_line":359,"start_character":8,"end_line":366,"end_character":17},"in_reply_to":"3a71b18c_4efac2be","updated":"2016-12-13 15:08:33.000000000","message":"I agree...\n\nThis patch is hanging for about 11 days now. I wonder if we should submit a new patch-set ourselves.","commit_id":"c5ad7ebc0c1c9ad5a58094677a977f706cf20f48"}]}