)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"310657b264aa474c8b9f9f5f5a0aa6c9ad13e97d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"396f5e1d_6071f8ff","updated":"2022-02-07 03:16:26.000000000","message":"LGTM, but I think the releasenote is necessary, because you will introduce a new status for ARQ.","commit_id":"2a54697b28b6840004dde73e1475dab9de06d5de"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"1c6707d82a567d2c519189e9d190d1bedacb81aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b3eae111_580c3be9","updated":"2022-02-07 06:29:04.000000000","message":"LGTM, but i have a little question.","commit_id":"2a54697b28b6840004dde73e1475dab9de06d5de"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"162da204b990016aae568f9f9449f5172cb38d6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4726ca65_f8c0aeb7","updated":"2022-02-17 09:55:39.000000000","message":"LGTM","commit_id":"84e030427ca94bdf708a415575ef54c515616005"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"273c8345f38757379281ae316478423bfe002696","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b20da848_4c808dcf","updated":"2022-03-11 09:32:14.000000000","message":"LGTM","commit_id":"84e030427ca94bdf708a415575ef54c515616005"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"3ce2aa59fb0c7e48ab9f17f0cc988bde3ae2d198","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7dfbb9d1_59d781d0","updated":"2022-03-14 07:44:24.000000000","message":"Just update the commit mesaage info, still lgtm.","commit_id":"82255a0e099908b23c71f3f614d1bb1e9275c0a4"}],"cyborg/common/constants.py":[{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"1c6707d82a567d2c519189e9d190d1bedacb81aa","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    \u0027Deleting\u0027)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"ARQ_BIND_STAGE \u003d (ARQ_PRE_BIND, ARQ_FINISH_BIND,"},{"line_number":33,"context_line":"                  ARQ_OUFOF_BIND_FLOW) \u003d ("},{"line_number":34,"context_line":"    [ARQ_INITIAL, ARQ_BIND_STARTED],"},{"line_number":35,"context_line":"    [ARQ_BOUND, ARQ_BIND_FAILED],"}],"source_content_type":"text/x-python","patch_set":2,"id":"ae018a2b_6dcad4e2","line":32,"updated":"2022-02-07 06:29:04.000000000","message":"This variable is not used. Should we remove it?","commit_id":"2a54697b28b6840004dde73e1475dab9de06d5de"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"830b539e3fdc69cc4b485be69840b04c6d313e69","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    \u0027Deleting\u0027)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"ARQ_BIND_STAGE \u003d (ARQ_PRE_BIND, ARQ_FINISH_BIND,"},{"line_number":33,"context_line":"                  ARQ_OUFOF_BIND_FLOW) \u003d ("},{"line_number":34,"context_line":"    [ARQ_INITIAL, ARQ_BIND_STARTED],"},{"line_number":35,"context_line":"    [ARQ_BOUND, ARQ_BIND_FAILED],"}],"source_content_type":"text/x-python","patch_set":2,"id":"8cedb5ea_0df4a4f6","line":32,"in_reply_to":"ae018a2b_6dcad4e2","updated":"2022-02-17 06:32:20.000000000","message":"we can remove in the follow commit.","commit_id":"2a54697b28b6840004dde73e1475dab9de06d5de"}],"cyborg/objects/ext_arq.py":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"3b140565832e742b1bfac70e5a6f133b689b0f55","unresolved":true,"context_lines":[{"line_number":218,"context_line":"        except Exception as e:"},{"line_number":219,"context_line":"            LOG.error(\"Failed to deallocate attach handle %s for ARQ %s.\""},{"line_number":220,"context_line":"                      \"Reason: %s\", ah_id, self.arq.uuid, str(e))"},{"line_number":221,"context_line":"            self.update_check_state("},{"line_number":222,"context_line":"                context, constants.ARQ_UNBIND_FAILED)"},{"line_number":223,"context_line":"            raise"},{"line_number":224,"context_line":"        LOG.info(\u0027Attach handle(%s) deallocate for ARQ(%s) successfully.\u0027,"},{"line_number":225,"context_line":"                 ah_id, self.arq.uuid)"}],"source_content_type":"text/x-python","patch_set":1,"id":"764d3ff1_04592e01","line":222,"range":{"start_line":221,"start_character":12,"end_line":222,"end_character":53},"updated":"2021-03-12 01:50:31.000000000","message":"If we are unbinding arq failed, whether need to keep the state of the attach handle unchanged? My concern is whether we can make the instance fine if unbind ARQ failed.","commit_id":"5f8463598d74d51a66758fe715763e1a17ec9f61"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"1516e7b133685d675b7c65b27bc3fb1a130c3a9c","unresolved":true,"context_lines":[{"line_number":218,"context_line":"        except Exception as e:"},{"line_number":219,"context_line":"            LOG.error(\"Failed to deallocate attach handle %s for ARQ %s.\""},{"line_number":220,"context_line":"                      \"Reason: %s\", ah_id, self.arq.uuid, str(e))"},{"line_number":221,"context_line":"            self.update_check_state("},{"line_number":222,"context_line":"                context, constants.ARQ_UNBIND_FAILED)"},{"line_number":223,"context_line":"            raise"},{"line_number":224,"context_line":"        LOG.info(\u0027Attach handle(%s) deallocate for ARQ(%s) successfully.\u0027,"},{"line_number":225,"context_line":"                 ah_id, self.arq.uuid)"}],"source_content_type":"text/x-python","patch_set":1,"id":"8fd3c53e_345acd3a","line":222,"range":{"start_line":221,"start_character":12,"end_line":222,"end_character":53},"in_reply_to":"764d3ff1_04592e01","updated":"2022-01-29 06:10:34.000000000","message":"i think we need change the state. nowadays, there are no rollback mechanism. we can plan to improve bind and unbind process.","commit_id":"5f8463598d74d51a66758fe715763e1a17ec9f61"}],"cyborg/tests/unit/fake_attach_handle.py":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"310657b264aa474c8b9f9f5f5a0aa6c9ad13e97d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2021 Inspur."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"fb7d2d93_bf39a3b1","line":1,"range":{"start_line":1,"start_character":12,"end_line":1,"end_character":17},"updated":"2022-02-07 03:16:26.000000000","message":"2022","commit_id":"2a54697b28b6840004dde73e1475dab9de06d5de"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"830b539e3fdc69cc4b485be69840b04c6d313e69","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2021 Inspur."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"245b7031_0810e9f3","line":1,"range":{"start_line":1,"start_character":12,"end_line":1,"end_character":17},"in_reply_to":"fb7d2d93_bf39a3b1","updated":"2022-02-17 06:32:20.000000000","message":"Done","commit_id":"2a54697b28b6840004dde73e1475dab9de06d5de"}],"releasenotes/notes/add_ARQ_UNBIND_FAILED_status-ea8636c64dd616eb.yaml":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"273c8345f38757379281ae316478423bfe002696","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add ``ARQ_UNBIND_FAILED`` status for Accelerator Requests (arq) unbind process. Nowdays the status is"},{"line_number":5,"context_line":"    needed to accurate record the arq status. "}],"source_content_type":"text/x-yaml","patch_set":3,"id":"10bf97b8_f9c2fc81","line":5,"range":{"start_line":5,"start_character":45,"end_line":5,"end_character":46},"updated":"2022-03-11 09:32:14.000000000","message":"nit:redundant.","commit_id":"84e030427ca94bdf708a415575ef54c515616005"}]}