)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"f6970af430f8077631249cc43ca0d4c4b9feb81f","unresolved":true,"context_lines":[{"line_number":9,"context_line":"The following points in the vnflcm_driver reduce the readability and"},{"line_number":10,"context_line":"might be potential bugs"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"(i) There are some unsed variables, e.g., ``scale_id_list`` in ``_scale_vnf_pre``"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"(ii) ``else`` in ``_scale_vnf_pre`` looks inappropriate because all unknown request types are considered as \u0027SCALE_OUT\u0027"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ed50b51f_dd255285","line":12,"updated":"2022-02-14 06:03:40.000000000","message":"Pls keep the max chars.","commit_id":"f4f3e04e7fc71e26c6546cbd469113ed8b2f065b"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"f6970af430f8077631249cc43ca0d4c4b9feb81f","unresolved":true,"context_lines":[{"line_number":9,"context_line":"The following points in the vnflcm_driver reduce the readability and"},{"line_number":10,"context_line":"might be potential bugs"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"(i) There are some unsed variables, e.g., ``scale_id_list`` in ``_scale_vnf_pre``"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"(ii) ``else`` in ``_scale_vnf_pre`` looks inappropriate because all unknown request types are considered as \u0027SCALE_OUT\u0027"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"38020cd1_a11c9cdd","line":12,"range":{"start_line":12,"start_character":19,"end_line":12,"end_character":24},"updated":"2022-02-14 06:03:40.000000000","message":"nit: unused","commit_id":"f4f3e04e7fc71e26c6546cbd469113ed8b2f065b"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"a351d4a32d22fa96588743518e909e15d5ea0087","unresolved":true,"context_lines":[{"line_number":21,"context_line":"(ii) Wrote ``elif scale_vnf_request.type \u003d\u003d \u0027SCALE_OUT\u0027:`` explicitly"},{"line_number":22,"context_line":"(iii) Merged ``_scale_vnf`` and ``scale`` and renamed ``scale`` to"},{"line_number":23,"context_line":"``_scale_vnf`` as most LCM operation methods consist of a pair of a"},{"line_number":24,"context_line":"protected method and public method, e.g., `heal_vnf and _heal_vnf` ."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Change-Id: I49457aa35458a68f2294c229b440ba1c68cbfe91"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"eea31ab1_8386ef3c","line":24,"range":{"start_line":24,"start_character":66,"end_line":24,"end_character":67},"updated":"2022-02-22 15:47:44.000000000","message":"nit: unnecessary blank.","commit_id":"f4f3e04e7fc71e26c6546cbd469113ed8b2f065b"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"823bf2ef0dbc4cb4e179fbfb993492ded94674ab","unresolved":false,"context_lines":[{"line_number":21,"context_line":"(ii) Wrote ``elif scale_vnf_request.type \u003d\u003d \u0027SCALE_OUT\u0027:`` explicitly"},{"line_number":22,"context_line":"(iii) Merged ``_scale_vnf`` and ``scale`` and renamed ``scale`` to"},{"line_number":23,"context_line":"``_scale_vnf`` as most LCM operation methods consist of a pair of a"},{"line_number":24,"context_line":"protected method and public method, e.g., `heal_vnf and _heal_vnf` ."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Change-Id: I49457aa35458a68f2294c229b440ba1c68cbfe91"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a290f050_2fd9e7a0","line":24,"range":{"start_line":24,"start_character":66,"end_line":24,"end_character":67},"in_reply_to":"e27ed6d4_44527aad","updated":"2022-02-24 21:22:13.000000000","message":"I\u0027ve confirmed.","commit_id":"f4f3e04e7fc71e26c6546cbd469113ed8b2f065b"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"8230abc67c7e416f98c19edb5654c88511237715","unresolved":true,"context_lines":[{"line_number":21,"context_line":"(ii) Wrote ``elif scale_vnf_request.type \u003d\u003d \u0027SCALE_OUT\u0027:`` explicitly"},{"line_number":22,"context_line":"(iii) Merged ``_scale_vnf`` and ``scale`` and renamed ``scale`` to"},{"line_number":23,"context_line":"``_scale_vnf`` as most LCM operation methods consist of a pair of a"},{"line_number":24,"context_line":"protected method and public method, e.g., `heal_vnf and _heal_vnf` ."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Change-Id: I49457aa35458a68f2294c229b440ba1c68cbfe91"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e27ed6d4_44527aad","line":24,"range":{"start_line":24,"start_character":66,"end_line":24,"end_character":67},"in_reply_to":"eea31ab1_8386ef3c","updated":"2022-02-24 05:59:00.000000000","message":"Thank you for your comment.\nI\u0027ll fix it in PS3.","commit_id":"f4f3e04e7fc71e26c6546cbd469113ed8b2f065b"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"a351d4a32d22fa96588743518e909e15d5ea0087","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Refactor v1 vnflcm_driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The following points in the vnflcm_driver reduce the readability and"},{"line_number":10,"context_line":"might be potential bugs"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"(i) There are some unused variables, e.g., ``scale_id_list`` in"},{"line_number":13,"context_line":"``_scale_vnf_pre``"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7dd751dc_30561c6f","line":10,"range":{"start_line":10,"start_character":22,"end_line":10,"end_character":23},"updated":"2022-02-22 15:47:44.000000000","message":"nit: How about to add a period at the end?","commit_id":"04ec5cf1c74d8011e3294884b1aadcdd82108e38"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"8230abc67c7e416f98c19edb5654c88511237715","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Refactor v1 vnflcm_driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The following points in the vnflcm_driver reduce the readability and"},{"line_number":10,"context_line":"might be potential bugs"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"(i) There are some unused variables, e.g., ``scale_id_list`` in"},{"line_number":13,"context_line":"``_scale_vnf_pre``"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"c0a1651e_f5fb915c","line":10,"range":{"start_line":10,"start_character":22,"end_line":10,"end_character":23},"in_reply_to":"7dd751dc_30561c6f","updated":"2022-02-24 05:59:00.000000000","message":"Thank you for your comment.\nI\u0027ll fix it in PS3.","commit_id":"04ec5cf1c74d8011e3294884b1aadcdd82108e38"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"823bf2ef0dbc4cb4e179fbfb993492ded94674ab","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Refactor v1 vnflcm_driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The following points in the vnflcm_driver reduce the readability and"},{"line_number":10,"context_line":"might be potential bugs"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"(i) There are some unused variables, e.g., ``scale_id_list`` in"},{"line_number":13,"context_line":"``_scale_vnf_pre``"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"04d97870_390de520","line":10,"range":{"start_line":10,"start_character":22,"end_line":10,"end_character":23},"in_reply_to":"c0a1651e_f5fb915c","updated":"2022-02-24 21:22:13.000000000","message":"I\u0027ve confirmed.","commit_id":"04ec5cf1c74d8011e3294884b1aadcdd82108e38"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"a351d4a32d22fa96588743518e909e15d5ea0087","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4a3f0c2_8ec92726","updated":"2022-02-22 15:47:44.000000000","message":"Sorry for late review, please kindly check nit comments.\nAnd there is a merge conflict, so please solve it.","commit_id":"04ec5cf1c74d8011e3294884b1aadcdd82108e38"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"1625bb9c76ab0c5dd5c6f745b6b9c18ddf1d9d71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1f2bb0de_3a2f7765","updated":"2022-02-15 01:36:58.000000000","message":"Thank you for the review \nI fixed it in PS2.","commit_id":"04ec5cf1c74d8011e3294884b1aadcdd82108e38"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"8230abc67c7e416f98c19edb5654c88511237715","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f04cc856_59d0e07d","updated":"2022-02-24 05:59:00.000000000","message":"Rebased onto master in PS4","commit_id":"45ac7ec7b924197096e5ee3efff4b2c5139e5bfa"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"823bf2ef0dbc4cb4e179fbfb993492ded94674ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"78498db2_53dc9e29","updated":"2022-02-24 21:22:13.000000000","message":"Thanks for your rework! LGTM.","commit_id":"45ac7ec7b924197096e5ee3efff4b2c5139e5bfa"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"4556c891af02fb8fb4305d75d9fcb2d4bf2fda7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"55ec1db5_4a67baf2","updated":"2022-02-24 10:03:10.000000000","message":"recheck","commit_id":"45ac7ec7b924197096e5ee3efff4b2c5139e5bfa"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"af2f7df94f3c03051e08812ea49648444053e40e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"73c7c3e5_f6084136","updated":"2022-02-24 14:26:30.000000000","message":"recheck","commit_id":"45ac7ec7b924197096e5ee3efff4b2c5139e5bfa"}]}
