)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a4b68f0de146107594c2084211992f15b83ba975","unresolved":true,"context_lines":[{"line_number":10,"context_line":"This command can execute Change external VNF Connectivity operation."},{"line_number":11,"context_line":"This API can change VL setting."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: support-change-external-connectivity"},{"line_number":14,"context_line":"Spec: https://review.opendev.org/c/openstack/tacker-specs/+/759661"},{"line_number":15,"context_line":"Change-Id: I3a935296646361032665082a4060bc21ff51c1b1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c9060024_56c1e5b5","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":66},"updated":"2021-02-19 07:46:14.000000000","message":"Could you please correct the syntax for \"Implements\" and \"Spec\" tag.\nFor example:\nImplements: blueprint support-change-external-connectivity\nSpec: https://specs.openstack.org/openstack/tacker-specs/specs/wallaby/support-change-external-VNF-connectivity-operation.html","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"ad0380946c4b6c75042f0365550973558d94cf6e","unresolved":true,"context_lines":[{"line_number":10,"context_line":"This command can execute Change external VNF Connectivity operation."},{"line_number":11,"context_line":"This API can change VL setting."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: support-change-external-connectivity"},{"line_number":14,"context_line":"Spec: https://review.opendev.org/c/openstack/tacker-specs/+/759661"},{"line_number":15,"context_line":"Change-Id: I3a935296646361032665082a4060bc21ff51c1b1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"97b88de6_de644fd5","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":66},"in_reply_to":"c9060024_56c1e5b5","updated":"2021-02-19 15:24:10.000000000","message":"Sorry, I fixed it.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"}],"setup.cfg":[{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"ab0a640195a4c037ede9e9baaa1e8121a2e3a19a","unresolved":true,"context_lines":[{"line_number":94,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":95,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":96,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":97,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":98,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"0101d6d8_cb8684b9","line":97,"range":{"start_line":97,"start_character":5,"end_line":97,"end_character":27},"updated":"2021-02-22 16:57:08.000000000","message":"The convention style is different from existing. IMO it should be vnflcm_change_ext_conn. \nPlease share your opinion, I might be missing something.","commit_id":"da868155220564d52f2de844d3eb523c0623c538"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"5ede5bd3ca2fc6178d1fedbfd539164e69fca7b9","unresolved":true,"context_lines":[{"line_number":94,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":95,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":96,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":97,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":98,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"62966aa5_c5711d9a","line":97,"range":{"start_line":97,"start_character":5,"end_line":97,"end_character":27},"in_reply_to":"0101d6d8_cb8684b9","updated":"2021-03-04 06:24:21.000000000","message":"\u003e The convention style is different from existing. IMO it should be vnflcm_change_ext_conn. \n\u003e Please share your opinion, I might be missing something.\n\nThe underscore is converted to a space, so the \"change_ext_conn\" token is split into multiple words. In order to avoid it, we use a hyphen instead of underscore.","commit_id":"da868155220564d52f2de844d3eb523c0623c538"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"141271cce80a88f077045fec6cc1d6ca7e7c6cde","unresolved":true,"context_lines":[{"line_number":94,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":95,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":96,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":97,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":98,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"573cea24_7b381273","line":97,"range":{"start_line":97,"start_character":5,"end_line":97,"end_character":27},"in_reply_to":"62966aa5_c5711d9a","updated":"2021-03-08 03:08:02.000000000","message":"Thank you for clarification.","commit_id":"da868155220564d52f2de844d3eb523c0623c538"}],"tackerclient/osc/v1/vnflcm/vnflcm.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"e2d9a012d4ae3424a453153f73d49caa67964136","unresolved":true,"context_lines":[{"line_number":549,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":550,"context_line":"            help\u003d_(\"VNF instance ID to Change External VNF Connectivity\"))"},{"line_number":551,"context_line":"        parser.add_argument("},{"line_number":552,"context_line":"            \u0027change_ext_conn_request_file\u0027,"},{"line_number":553,"context_line":"            metavar\u003d\"\u003cparam-file\u003e\","},{"line_number":554,"context_line":"            help\u003d_(\"Specify change-ext-conn request parameters \""},{"line_number":555,"context_line":"                   \"in a json file.\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"ac7488bb_fac96dce","line":552,"range":{"start_line":552,"start_character":13,"end_line":552,"end_character":42},"updated":"2021-02-22 02:48:44.000000000","message":"Since it is obvious that it is an argument related to change_ext_conn, I think that \u0027request_file\u0027 is easier for the user to execute and is better.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"c0b189fd0e0f6b182d6d2f75ec0cf63bbe2c0efb","unresolved":false,"context_lines":[{"line_number":549,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":550,"context_line":"            help\u003d_(\"VNF instance ID to Change External VNF Connectivity\"))"},{"line_number":551,"context_line":"        parser.add_argument("},{"line_number":552,"context_line":"            \u0027change_ext_conn_request_file\u0027,"},{"line_number":553,"context_line":"            metavar\u003d\"\u003cparam-file\u003e\","},{"line_number":554,"context_line":"            help\u003d_(\"Specify change-ext-conn request parameters \""},{"line_number":555,"context_line":"                   \"in a json file.\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"751cfdff_f3611ff8","line":552,"range":{"start_line":552,"start_character":13,"end_line":552,"end_character":42},"in_reply_to":"773ed82c_c8c66f99","updated":"2021-03-05 06:21:56.000000000","message":"I\u0027ve confirmed.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"5ede5bd3ca2fc6178d1fedbfd539164e69fca7b9","unresolved":true,"context_lines":[{"line_number":549,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":550,"context_line":"            help\u003d_(\"VNF instance ID to Change External VNF Connectivity\"))"},{"line_number":551,"context_line":"        parser.add_argument("},{"line_number":552,"context_line":"            \u0027change_ext_conn_request_file\u0027,"},{"line_number":553,"context_line":"            metavar\u003d\"\u003cparam-file\u003e\","},{"line_number":554,"context_line":"            help\u003d_(\"Specify change-ext-conn request parameters \""},{"line_number":555,"context_line":"                   \"in a json file.\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"773ed82c_c8c66f99","line":552,"range":{"start_line":552,"start_character":13,"end_line":552,"end_character":42},"in_reply_to":"ac7488bb_fac96dce","updated":"2021-03-04 06:24:21.000000000","message":"\u003e Since it is obvious that it is an argument related to change_ext_conn, I think that \u0027request_file\u0027 is easier for the user to execute and is better.\n\nIt is changed to \u0027request_file\u0027.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"}],"tackerclient/v1_0/client.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"e2d9a012d4ae3424a453153f73d49caa67964136","unresolved":true,"context_lines":[{"line_number":1221,"context_line":"    def download_vnf_package(self, vnf_package):"},{"line_number":1222,"context_line":"        return self.vnf_package_client.download_vnf_package(vnf_package)"},{"line_number":1223,"context_line":""},{"line_number":1224,"context_line":"    def change_ext_conn_vnf_instance(self, vnf_id, body):"},{"line_number":1225,"context_line":"        return self.vnf_lcm_client.change_ext_conn_vnf_instance(vnf_id, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c90b9d3d_3d8b3b72","line":1224,"updated":"2021-02-22 02:48:44.000000000","message":"It was inappropriate to define VNF Package related items after line 1204, but I think it\u0027s better to move this to line 1199.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"c0b189fd0e0f6b182d6d2f75ec0cf63bbe2c0efb","unresolved":false,"context_lines":[{"line_number":1221,"context_line":"    def download_vnf_package(self, vnf_package):"},{"line_number":1222,"context_line":"        return self.vnf_package_client.download_vnf_package(vnf_package)"},{"line_number":1223,"context_line":""},{"line_number":1224,"context_line":"    def change_ext_conn_vnf_instance(self, vnf_id, body):"},{"line_number":1225,"context_line":"        return self.vnf_lcm_client.change_ext_conn_vnf_instance(vnf_id, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"c8b28b02_da2b3d46","line":1224,"in_reply_to":"438625c3_9ff5c09e","updated":"2021-03-05 06:21:56.000000000","message":"I\u0027ve confirmed.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"5ede5bd3ca2fc6178d1fedbfd539164e69fca7b9","unresolved":true,"context_lines":[{"line_number":1221,"context_line":"    def download_vnf_package(self, vnf_package):"},{"line_number":1222,"context_line":"        return self.vnf_package_client.download_vnf_package(vnf_package)"},{"line_number":1223,"context_line":""},{"line_number":1224,"context_line":"    def change_ext_conn_vnf_instance(self, vnf_id, body):"},{"line_number":1225,"context_line":"        return self.vnf_lcm_client.change_ext_conn_vnf_instance(vnf_id, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"438625c3_9ff5c09e","line":1224,"in_reply_to":"c90b9d3d_3d8b3b72","updated":"2021-03-04 06:24:21.000000000","message":"\u003e It was inappropriate to define VNF Package related items after line 1204, but I think it\u0027s better to move this to line 1199.\n\nIt is moved to line 1199.","commit_id":"b75532ee9d22543ff1fd0856910f04e3b00cb198"}]}