)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31668},"change_message_id":"39741b488e0d0f7a1f46b80d17bf14090cf451f9","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This patch implements rollback operation task defined in ETSI NFV-SOL003"},{"line_number":10,"context_line":"v3.3.1 5.4.15."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch spports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not spported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"05a127f0_c15cf3f1","line":12,"updated":"2021-12-02 03:00:05.000000000","message":"typo: spports -\u003e supports","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f187c89af69a03a68970a93fd0f1d17d907c74d5","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This patch implements rollback operation task defined in ETSI NFV-SOL003"},{"line_number":10,"context_line":"v3.3.1 5.4.15."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch spports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not spported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"79ac6702_589221b8","line":12,"in_reply_to":"05a127f0_c15cf3f1","updated":"2021-12-02 04:30:45.000000000","message":"I will fix.","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":31668},"change_message_id":"8e93f89ef6d78b8fb465d303ffb25f5dc4aebdb3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch implements rollback operation task defined in ETSI NFV-SOL003"},{"line_number":10,"context_line":"v3.3.1 5.4.15."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch spports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not spported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3b6dd64c_4f58caf0","line":12,"in_reply_to":"79ac6702_589221b8","updated":"2021-12-02 05:00:21.000000000","message":"Done","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":31668},"change_message_id":"39741b488e0d0f7a1f46b80d17bf14090cf451f9","unresolved":true,"context_lines":[{"line_number":10,"context_line":"v3.3.1 5.4.15."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch spports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not spported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"581b4af8_d40c51c7","line":13,"updated":"2021-12-02 03:00:05.000000000","message":"typo: spported -\u003e supported","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f187c89af69a03a68970a93fd0f1d17d907c74d5","unresolved":true,"context_lines":[{"line_number":10,"context_line":"v3.3.1 5.4.15."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch spports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not spported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"aece7722_40f353e9","line":13,"in_reply_to":"581b4af8_d40c51c7","updated":"2021-12-02 04:30:45.000000000","message":"I will fix.","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":31668},"change_message_id":"8e93f89ef6d78b8fb465d303ffb25f5dc4aebdb3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"v3.3.1 5.4.15."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch spports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not spported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"58ac5b0b_ac08dc9b","line":13,"in_reply_to":"aece7722_40f353e9","updated":"2021-12-02 05:00:21.000000000","message":"Done","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d7325e5f96a392f3d18f48d047e87eb542525f14","unresolved":true,"context_lines":[{"line_number":12,"context_line":"This patch supports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not supported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Functional tests will be provided with another patch."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"7c10528d_4df4422e","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":73},"updated":"2021-12-17 06:38:01.000000000","message":"nit: Please limit the number of characters per line to 72.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure\n\u003e Subsequent lines should be wrapped at 72 characters.","commit_id":"a5405eaf30b0c3e68c15308237dffba01108e27d"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"6e9e7eb375d156eb8d3118a68eb0ec4d1840f153","unresolved":true,"context_lines":[{"line_number":12,"context_line":"This patch supports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not supported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Functional tests will be provided with another patch."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"ff8e2ba0_06f53909","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":73},"in_reply_to":"22934922_db615e05","updated":"2021-12-17 08:37:59.000000000","message":"Thank you for your comment. I fixed it.","commit_id":"a5405eaf30b0c3e68c15308237dffba01108e27d"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"e697fa8bf379927160cba565d802e190836ca613","unresolved":true,"context_lines":[{"line_number":12,"context_line":"This patch supports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not supported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Functional tests will be provided with another patch."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9dcb9bd8_a3e6650a","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":73},"in_reply_to":"7c10528d_4df4422e","updated":"2021-12-17 08:25:14.000000000","message":"I fixed it.","commit_id":"a5405eaf30b0c3e68c15308237dffba01108e27d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b20ce81d9effcf4c8a9f73b7bd157c39784a05d8","unresolved":true,"context_lines":[{"line_number":12,"context_line":"This patch supports rollback operation of existing task API, i.e"},{"line_number":13,"context_line":"instantiate. Note that rollback operation of terminate is not supported."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Rollback operation of a task API which is added in the future (ex. scale)"},{"line_number":16,"context_line":"will be implemented in the same patch of implementing the API."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Functional tests will be provided with another patch."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"22934922_db615e05","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":73},"in_reply_to":"9dcb9bd8_a3e6650a","updated":"2021-12-17 08:35:48.000000000","message":"Space character after `(ex. scale)` seems to be full-width character.\nCould you change to half-width character?","commit_id":"a5405eaf30b0c3e68c15308237dffba01108e27d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f1a85cb3d9c2f316185b6ba2fd2bae9f42827c00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"58dd195b_89eb4731","updated":"2021-11-08 23:05:28.000000000","message":"Thank you for the review.","commit_id":"168b6a600cba48cfebf77b1942e524b7f5eab50f"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7abfe096de327453de85da22a7e9f4a8191cab64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ba22c323_8167c5f1","updated":"2021-11-08 02:56:00.000000000","message":"patch set 2: add release note, fix commit message.","commit_id":"168b6a600cba48cfebf77b1942e524b7f5eab50f"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"e863a96a78e066d755488bfca51abf8a4bbfe6d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"089ac28f_18416d14","updated":"2021-11-09 23:31:20.000000000","message":"recheck","commit_id":"765582efffcc5d805546ad24e83574d4b4c2d075"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"59c032d5305795b1b89a0ece31e65e2aa4834861","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0d1fa162_22a1b64a","updated":"2021-12-02 04:25:42.000000000","message":"Thank you for the review.","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d7325e5f96a392f3d18f48d047e87eb542525f14","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"6e37dd79_09f68c3a","updated":"2021-12-17 06:38:01.000000000","message":"LGTM, but there is a small pointing out in the commit message.\nCould you confirm it?","commit_id":"a5405eaf30b0c3e68c15308237dffba01108e27d"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"e697fa8bf379927160cba565d802e190836ca613","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d5157f4e_30529e49","updated":"2021-12-17 08:25:14.000000000","message":"I fixed commit message.","commit_id":"672a30a3964979fb285ba902d5f7caa9e1be9079"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b38d8f98691640278305e1e9224764c4ea5c7629","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"f66dda92_dc9b2ea7","updated":"2021-12-17 15:37:51.000000000","message":"recheck","commit_id":"4b5204b213094c15e2fce583f9741c18ab609a42"}],"tacker/sol_refactored/conductor/conductor_v2.py":[{"author":{"_account_id":33453,"name":"Shun Higuchi","email":"higuchis@intellilink.co.jp","username":"higuchis"},"change_message_id":"942187deef7b012000885bf765da9ca8433ba1c4","unresolved":true,"context_lines":[{"line_number":216,"context_line":"            with context.session.begin(subtransactions\u003dTrue):"},{"line_number":217,"context_line":"                # it is not necessary to update inst DB because it was not"},{"line_number":218,"context_line":"                # changed when the operationState became FAILED_TEMP."},{"line_number":219,"context_line":"                # note that inst object may be changed in driver\u0027s rollback"},{"line_number":220,"context_line":"                # method temporary but must not save it."},{"line_number":221,"context_line":"                lcmocc.update(context)"},{"line_number":222,"context_line":"                # grant_req and grant are not necessary any more."},{"line_number":223,"context_line":"                grant_req.delete(context)"}],"source_content_type":"text/x-python","patch_set":2,"id":"a8763fcb_67787bb4","line":220,"range":{"start_line":219,"start_character":16,"end_line":220,"end_character":56},"updated":"2021-11-08 10:23:12.000000000","message":"Could you emphasize with \"NOTE\"?","commit_id":"168b6a600cba48cfebf77b1942e524b7f5eab50f"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f1a85cb3d9c2f316185b6ba2fd2bae9f42827c00","unresolved":true,"context_lines":[{"line_number":216,"context_line":"            with context.session.begin(subtransactions\u003dTrue):"},{"line_number":217,"context_line":"                # it is not necessary to update inst DB because it was not"},{"line_number":218,"context_line":"                # changed when the operationState became FAILED_TEMP."},{"line_number":219,"context_line":"                # note that inst object may be changed in driver\u0027s rollback"},{"line_number":220,"context_line":"                # method temporary but must not save it."},{"line_number":221,"context_line":"                lcmocc.update(context)"},{"line_number":222,"context_line":"                # grant_req and grant are not necessary any more."},{"line_number":223,"context_line":"                grant_req.delete(context)"}],"source_content_type":"text/x-python","patch_set":2,"id":"d6c7ac4d_222a22c7","line":220,"range":{"start_line":219,"start_character":16,"end_line":220,"end_character":56},"in_reply_to":"a8763fcb_67787bb4","updated":"2021-11-08 23:05:28.000000000","message":"OK, I will do it.","commit_id":"168b6a600cba48cfebf77b1942e524b7f5eab50f"},{"author":{"_account_id":33453,"name":"Shun Higuchi","email":"higuchis@intellilink.co.jp","username":"higuchis"},"change_message_id":"274ea035150f150d971955ee3768b1eb6878d6d1","unresolved":false,"context_lines":[{"line_number":216,"context_line":"            with context.session.begin(subtransactions\u003dTrue):"},{"line_number":217,"context_line":"                # it is not necessary to update inst DB because it was not"},{"line_number":218,"context_line":"                # changed when the operationState became FAILED_TEMP."},{"line_number":219,"context_line":"                # note that inst object may be changed in driver\u0027s rollback"},{"line_number":220,"context_line":"                # method temporary but must not save it."},{"line_number":221,"context_line":"                lcmocc.update(context)"},{"line_number":222,"context_line":"                # grant_req and grant are not necessary any more."},{"line_number":223,"context_line":"                grant_req.delete(context)"}],"source_content_type":"text/x-python","patch_set":2,"id":"59129d41_a7465479","line":220,"range":{"start_line":219,"start_character":16,"end_line":220,"end_character":56},"in_reply_to":"d6c7ac4d_222a22c7","updated":"2021-11-09 10:33:49.000000000","message":"Done","commit_id":"168b6a600cba48cfebf77b1942e524b7f5eab50f"},{"author":{"_account_id":31668},"change_message_id":"39741b488e0d0f7a1f46b80d17bf14090cf451f9","unresolved":true,"context_lines":[{"line_number":192,"context_line":"    def _rollback_lcm_op(self, context, lcmocc):"},{"line_number":193,"context_line":"        # just consistency check"},{"line_number":194,"context_line":"        if lcmocc.operationState !\u003d fields.LcmOperationStateType.FAILED_TEMP:"},{"line_number":195,"context_line":"            LOG.error(\"VnfLcmOpOcc unexpected operationState.\")"},{"line_number":196,"context_line":"            return"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        inst \u003d inst_utils.get_inst(context, lcmocc.vnfInstanceId)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9bd2513a_ea8d844c","line":195,"updated":"2021-12-02 03:00:05.000000000","message":"I think it is better to execute \"raise sol_ex.LcmOpOccNotFailedTemp (lcmocc_id \u003d lcmocc.id)\" processing like ”_lcm_op_occ_rollback\" of VnfLcmControllerV2.","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"59c032d5305795b1b89a0ece31e65e2aa4834861","unresolved":true,"context_lines":[{"line_number":192,"context_line":"    def _rollback_lcm_op(self, context, lcmocc):"},{"line_number":193,"context_line":"        # just consistency check"},{"line_number":194,"context_line":"        if lcmocc.operationState !\u003d fields.LcmOperationStateType.FAILED_TEMP:"},{"line_number":195,"context_line":"            LOG.error(\"VnfLcmOpOcc unexpected operationState.\")"},{"line_number":196,"context_line":"            return"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        inst \u003d inst_utils.get_inst(context, lcmocc.vnfInstanceId)"}],"source_content_type":"text/x-python","patch_set":5,"id":"d73b6947_52fa6d15","line":195,"in_reply_to":"9bd2513a_ea8d844c","updated":"2021-12-02 04:25:42.000000000","message":"It is just an assertion. It is same as L.61. It means some awfully strange thing happens. Raising exception is for the case that the error may happen.","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"},{"author":{"_account_id":31668},"change_message_id":"8e93f89ef6d78b8fb465d303ffb25f5dc4aebdb3","unresolved":false,"context_lines":[{"line_number":192,"context_line":"    def _rollback_lcm_op(self, context, lcmocc):"},{"line_number":193,"context_line":"        # just consistency check"},{"line_number":194,"context_line":"        if lcmocc.operationState !\u003d fields.LcmOperationStateType.FAILED_TEMP:"},{"line_number":195,"context_line":"            LOG.error(\"VnfLcmOpOcc unexpected operationState.\")"},{"line_number":196,"context_line":"            return"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        inst \u003d inst_utils.get_inst(context, lcmocc.vnfInstanceId)"}],"source_content_type":"text/x-python","patch_set":5,"id":"16afb8af_1eded605","line":195,"in_reply_to":"d73b6947_52fa6d15","updated":"2021-12-02 05:00:21.000000000","message":"OK. I see.","commit_id":"f0e1214f5ef68c3c07cf1fa997b3d7741df48b92"}]}