)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6436c94449415a243dfc97cfa3b2d01c7a3fd633","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Fix Terraform infra-driver instantiate rollback"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes instantiate rollback operation fails for"},{"line_number":10,"context_line":"Terraform infra-driver. This patch fixes instantiate rollback"},{"line_number":11,"context_line":"operation fails for Terraform infra-driver. The main fixes are"},{"line_number":12,"context_line":"as follows:"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- Change it to use \"InstantiateVnfRequest\" instead of"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1db6573a_7e2770d0","line":11,"range":{"start_line":10,"start_character":24,"end_line":11,"end_character":43},"updated":"2023-11-16 06:19:52.000000000","message":"The same sentence is repeated. Could you delete it?","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"20bd8581816daf4ec2b6c31fe660109d22e9d359","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Fix Terraform infra-driver instantiate rollback"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes instantiate rollback operation fails for"},{"line_number":10,"context_line":"Terraform infra-driver. This patch fixes instantiate rollback"},{"line_number":11,"context_line":"operation fails for Terraform infra-driver. The main fixes are"},{"line_number":12,"context_line":"as follows:"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- Change it to use \"InstantiateVnfRequest\" instead of"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"e526b679_979f3a73","line":11,"range":{"start_line":10,"start_character":24,"end_line":11,"end_character":43},"in_reply_to":"1db6573a_7e2770d0","updated":"2023-11-16 10:35:50.000000000","message":"Correct in Patchset 3.","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"897cb8f910fe81c634d9fbf339ea7104337acc5f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix Terraform infra-driver instantiate rollback"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes instantiate rollback operation fails for"},{"line_number":10,"context_line":"Terraform infra-driver. This patch fixes instantiate rollback"},{"line_number":11,"context_line":"operation fails for Terraform infra-driver. The main fixes are"},{"line_number":12,"context_line":"as follows:"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- Change it to use \"InstantiateVnfRequest\" instead of"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"30be422f_af753c15","line":11,"range":{"start_line":10,"start_character":24,"end_line":11,"end_character":43},"in_reply_to":"e526b679_979f3a73","updated":"2023-12-26 09:59:42.000000000","message":"Acknowledged","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6436c94449415a243dfc97cfa3b2d01c7a3fd633","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"90b36e45_71a60a53","updated":"2023-11-16 06:19:52.000000000","message":"Thanks for your patch, please kindly find my comments","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"b6f35158403a2254c695f74eb94eb6c1c0d1ed2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7a77c8ed_b1ff87d7","updated":"2023-11-15 23:29:48.000000000","message":"recheck","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"cc337e319147975a1d1eed4b29a2996ba442eb0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"73aee876_5bc80f11","updated":"2023-12-27 00:10:22.000000000","message":"LGTM","commit_id":"f40a63e49cd762c8774492b0139f173dbb94bf0f"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"20bd8581816daf4ec2b6c31fe660109d22e9d359","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c94421c6_e4626300","updated":"2023-11-16 10:35:50.000000000","message":"Thank you for your review. I reflect your comment, kindly take a confirm.","commit_id":"f40a63e49cd762c8774492b0139f173dbb94bf0f"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"427c8540d81095cb53bb7abbfb8785889faa7fac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c6bd3f0c_76d264f3","updated":"2023-11-17 00:10:34.000000000","message":"recheck","commit_id":"f40a63e49cd762c8774492b0139f173dbb94bf0f"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"897cb8f910fe81c634d9fbf339ea7104337acc5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f33d52c2_724e189c","updated":"2023-12-26 09:59:42.000000000","message":"recheck - infra failure\n\nSorry for late reply, thanks for your rework, LGTM.\nI was going to comment after Zuul+1, but I will recheck it.","commit_id":"f40a63e49cd762c8774492b0139f173dbb94bf0f"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e9cd5a5776c8bc2699771acc427d6f138fc1bec3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5a80e2c2_dfe7e071","updated":"2023-12-26 14:45:48.000000000","message":"recheck - infra failure in sol-kubernetes job","commit_id":"f40a63e49cd762c8774492b0139f173dbb94bf0f"}],"tacker/tests/functional/sol_terraform_v2/test_terraform.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6436c94449415a243dfc97cfa3b2d01c7a3fd633","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":194,"context_line":"        self.check_package_usage(self.basic_pkg, state\u003d\u0027NOT_IN_USE\u0027)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        # TODO(yasufum) consider to add a test for instantiate_rollback here."},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    def test_instantiate_rollback(self):"},{"line_number":199,"context_line":"        \"\"\"Test basic LCM operations"}],"source_content_type":"text/x-python","patch_set":1,"id":"0e0f71cf_7f742675","line":196,"range":{"start_line":196,"start_character":8,"end_line":196,"end_character":77},"updated":"2023-11-16 06:19:52.000000000","message":"Could you delete this comment? Or are there any more tests to add?","commit_id":"5e9e46d4f13e2add875785e4d620833b281302c6"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"20bd8581816daf4ec2b6c31fe660109d22e9d359","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":194,"context_line":"        self.check_package_usage(self.basic_pkg, state\u003d\u0027NOT_IN_USE\u0027)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        # TODO(yasufum) consider to add a test for instantiate_rollback here."},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    def test_instantiate_rollback(self):"},{"line_number":199,"context_line":"        \"\"\"Test basic LCM operations"}],"source_content_type":"text/x-python","patch_set":1,"id":"f6a7d223_bf56eed9","line":196,"range":{"start_line":196,"start_character":8,"end_line":196,"end_character":77},"in_reply_to":"0e0f71cf_7f742675","updated":"2023-11-16 10:35:50.000000000","message":"Delete this comment.","commit_id":"5e9e46d4f13e2add875785e4d620833b281302c6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"897cb8f910fe81c634d9fbf339ea7104337acc5f","unresolved":false,"context_lines":[{"line_number":193,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":194,"context_line":"        self.check_package_usage(self.basic_pkg, state\u003d\u0027NOT_IN_USE\u0027)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        # TODO(yasufum) consider to add a test for instantiate_rollback here."},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    def test_instantiate_rollback(self):"},{"line_number":199,"context_line":"        \"\"\"Test basic LCM operations"}],"source_content_type":"text/x-python","patch_set":1,"id":"5ae55d03_c2737b2e","line":196,"range":{"start_line":196,"start_character":8,"end_line":196,"end_character":77},"in_reply_to":"f6a7d223_bf56eed9","updated":"2023-12-26 09:59:42.000000000","message":"Acknowledged","commit_id":"5e9e46d4f13e2add875785e4d620833b281302c6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6436c94449415a243dfc97cfa3b2d01c7a3fd633","unresolved":true,"context_lines":[{"line_number":336,"context_line":"        # TODO(yasufum) consider to add a test for instantiate_rollback here."},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"    def test_instantiate_rollback(self):"},{"line_number":339,"context_line":"        \"\"\"Test basic LCM operations"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        * About LCM operations:"},{"line_number":342,"context_line":"          This test includes the following operations."}],"source_content_type":"text/x-python","patch_set":2,"id":"59a857d4_5a8f1224","line":339,"range":{"start_line":339,"start_character":16,"end_line":339,"end_character":36},"updated":"2023-11-16 06:19:52.000000000","message":"I think this test is not \"basic LCM operations\".\nCould you change to the appropriate comment?","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"897cb8f910fe81c634d9fbf339ea7104337acc5f","unresolved":false,"context_lines":[{"line_number":336,"context_line":"        # TODO(yasufum) consider to add a test for instantiate_rollback here."},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"    def test_instantiate_rollback(self):"},{"line_number":339,"context_line":"        \"\"\"Test basic LCM operations"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        * About LCM operations:"},{"line_number":342,"context_line":"          This test includes the following operations."}],"source_content_type":"text/x-python","patch_set":2,"id":"631f023d_ffefcc3c","line":339,"range":{"start_line":339,"start_character":16,"end_line":339,"end_character":36},"in_reply_to":"53f9061d_c8fa0263","updated":"2023-12-26 09:59:42.000000000","message":"Acknowledged","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"20bd8581816daf4ec2b6c31fe660109d22e9d359","unresolved":true,"context_lines":[{"line_number":336,"context_line":"        # TODO(yasufum) consider to add a test for instantiate_rollback here."},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"    def test_instantiate_rollback(self):"},{"line_number":339,"context_line":"        \"\"\"Test basic LCM operations"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        * About LCM operations:"},{"line_number":342,"context_line":"          This test includes the following operations."}],"source_content_type":"text/x-python","patch_set":2,"id":"53f9061d_c8fa0263","line":339,"range":{"start_line":339,"start_character":16,"end_line":339,"end_character":36},"in_reply_to":"59a857d4_5a8f1224","updated":"2023-11-16 10:35:50.000000000","message":"Correct in Patchset 3.","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6436c94449415a243dfc97cfa3b2d01c7a3fd633","unresolved":true,"context_lines":[{"line_number":348,"context_line":"          - 6. Delete a VNF instance"},{"line_number":349,"context_line":"        \"\"\""},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        # 1. Create a new VNF instance resource"},{"line_number":352,"context_line":"        create_req \u003d tf_paramgen.create_req_by_vnfd_id(self.basic_vnfd_id)"},{"line_number":353,"context_line":"        resp, body \u003d self.create_vnf_instance(create_req)"},{"line_number":354,"context_line":"        self.assertEqual(201, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa79d65a_14a68749","line":351,"range":{"start_line":351,"start_character":13,"end_line":351,"end_character":47},"updated":"2023-11-16 06:19:52.000000000","message":"nit: How about you modify it to match the comments in the docstring above?\n```\n1. Create VNF instance\n```","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"897cb8f910fe81c634d9fbf339ea7104337acc5f","unresolved":false,"context_lines":[{"line_number":348,"context_line":"          - 6. Delete a VNF instance"},{"line_number":349,"context_line":"        \"\"\""},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        # 1. Create a new VNF instance resource"},{"line_number":352,"context_line":"        create_req \u003d tf_paramgen.create_req_by_vnfd_id(self.basic_vnfd_id)"},{"line_number":353,"context_line":"        resp, body \u003d self.create_vnf_instance(create_req)"},{"line_number":354,"context_line":"        self.assertEqual(201, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":2,"id":"192ca522_b9312fc9","line":351,"range":{"start_line":351,"start_character":13,"end_line":351,"end_character":47},"in_reply_to":"67ecccbe_750a3425","updated":"2023-12-26 09:59:42.000000000","message":"Acknowledged","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"20bd8581816daf4ec2b6c31fe660109d22e9d359","unresolved":true,"context_lines":[{"line_number":348,"context_line":"          - 6. Delete a VNF instance"},{"line_number":349,"context_line":"        \"\"\""},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        # 1. Create a new VNF instance resource"},{"line_number":352,"context_line":"        create_req \u003d tf_paramgen.create_req_by_vnfd_id(self.basic_vnfd_id)"},{"line_number":353,"context_line":"        resp, body \u003d self.create_vnf_instance(create_req)"},{"line_number":354,"context_line":"        self.assertEqual(201, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":2,"id":"67ecccbe_750a3425","line":351,"range":{"start_line":351,"start_character":13,"end_line":351,"end_character":47},"in_reply_to":"fa79d65a_14a68749","updated":"2023-11-16 10:35:50.000000000","message":"Correct in Patchset 3.","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6436c94449415a243dfc97cfa3b2d01c7a3fd633","unresolved":true,"context_lines":[{"line_number":355,"context_line":"        self.check_resp_headers_in_create(resp)"},{"line_number":356,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"        # 2. Instantiate a VNF instance \u003d\u003e FAILED_TEMP"},{"line_number":359,"context_line":"        self.put_fail_file(\u0027instantiate_end\u0027)"},{"line_number":360,"context_line":"        instantiate_req \u003d tf_paramgen.instantiate_req()"},{"line_number":361,"context_line":"        resp, body \u003d self.instantiate_vnf_instance(inst_id, instantiate_req)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c240ceaa_429bdcb2","line":358,"range":{"start_line":358,"start_character":13,"end_line":358,"end_character":54},"updated":"2023-11-16 06:19:52.000000000","message":"nit: How about you modify it to match the comments in the docstring above?\n```\n2. Instantiate VNF \u003d\u003e FAILED_TEMP\n```","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"897cb8f910fe81c634d9fbf339ea7104337acc5f","unresolved":false,"context_lines":[{"line_number":355,"context_line":"        self.check_resp_headers_in_create(resp)"},{"line_number":356,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"        # 2. Instantiate a VNF instance \u003d\u003e FAILED_TEMP"},{"line_number":359,"context_line":"        self.put_fail_file(\u0027instantiate_end\u0027)"},{"line_number":360,"context_line":"        instantiate_req \u003d tf_paramgen.instantiate_req()"},{"line_number":361,"context_line":"        resp, body \u003d self.instantiate_vnf_instance(inst_id, instantiate_req)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c7a75b1f_1e999ec4","line":358,"range":{"start_line":358,"start_character":13,"end_line":358,"end_character":54},"in_reply_to":"5f218874_edeae500","updated":"2023-12-26 09:59:42.000000000","message":"Acknowledged","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"},{"author":{"_account_id":31730,"name":"Naoaki Horie","email":"naoaki.h.opst@gmail.com","username":"naoaki-horie"},"change_message_id":"20bd8581816daf4ec2b6c31fe660109d22e9d359","unresolved":true,"context_lines":[{"line_number":355,"context_line":"        self.check_resp_headers_in_create(resp)"},{"line_number":356,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"        # 2. Instantiate a VNF instance \u003d\u003e FAILED_TEMP"},{"line_number":359,"context_line":"        self.put_fail_file(\u0027instantiate_end\u0027)"},{"line_number":360,"context_line":"        instantiate_req \u003d tf_paramgen.instantiate_req()"},{"line_number":361,"context_line":"        resp, body \u003d self.instantiate_vnf_instance(inst_id, instantiate_req)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f218874_edeae500","line":358,"range":{"start_line":358,"start_character":13,"end_line":358,"end_character":54},"in_reply_to":"c240ceaa_429bdcb2","updated":"2023-11-16 10:35:50.000000000","message":"Correct in Patchset 3.","commit_id":"24f06060e6f25a54b168943c991d9ba39d021639"}]}