)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"148bee05b5dcaefebf941b83a0dd0b4d6fb00c70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4dcdae0e_f1f03822","updated":"2022-11-15 01:41:48.000000000","message":"Thank you for the review.","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"420f3b4d02db918e9871d5e22dc1bc170d4fb4db","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ee770730_949321a1","updated":"2022-11-16 03:22:18.000000000","message":"Thank you for review.","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8b40b5e8_bc9817ff","updated":"2022-12-13 01:36:43.000000000","message":"Thank you for the review.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"20e131f0_573829ac","updated":"2022-12-12 23:59:34.000000000","message":"Thank you for your great work. Please kindly check my comments.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"237de178_eaa8e541","updated":"2022-12-13 06:31:32.000000000","message":"Thank you for your re-work. LGTM!","commit_id":"2f89ccc8a52cc02f5f7494b353030a24c77ea471"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"67d9bf7e67e987b5c78bca075b15b9f1fd75874b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a7a522e4_e8707f43","updated":"2022-12-13 07:29:46.000000000","message":"Thanks for your patch, LGTM.","commit_id":"2f89ccc8a52cc02f5f7494b353030a24c77ea471"}],"tacker/sol_refactored/common/common_script_utils.py":[{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":true,"context_lines":[{"line_number":259,"context_line":""},{"line_number":260,"context_line":"    if mgd_vls:"},{"line_number":261,"context_line":"        # convert ExtVirtualLinkInfo to ExtManagedVirtualLinkData."},{"line_number":262,"context_line":"        # necessary member only."},{"line_number":263,"context_line":"        mgd_vls \u003d ["},{"line_number":264,"context_line":"            {"},{"line_number":265,"context_line":"                \u0027vnfVirtualLinkDescId\u0027: mgd_vl[\u0027vnfVirtualLinkDescId\u0027],"}],"source_content_type":"text/x-python","patch_set":6,"id":"165f85af_3e618620","line":262,"range":{"start_line":262,"start_character":20,"end_line":262,"end_character":26},"updated":"2022-12-12 23:59:34.000000000","message":"Nits, member-\u003emembers","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[{"line_number":259,"context_line":""},{"line_number":260,"context_line":"    if mgd_vls:"},{"line_number":261,"context_line":"        # convert ExtVirtualLinkInfo to ExtManagedVirtualLinkData."},{"line_number":262,"context_line":"        # necessary member only."},{"line_number":263,"context_line":"        mgd_vls \u003d ["},{"line_number":264,"context_line":"            {"},{"line_number":265,"context_line":"                \u0027vnfVirtualLinkDescId\u0027: mgd_vl[\u0027vnfVirtualLinkDescId\u0027],"}],"source_content_type":"text/x-python","patch_set":6,"id":"6bb5baeb_620bae07","line":262,"range":{"start_line":262,"start_character":20,"end_line":262,"end_character":26},"in_reply_to":"165f85af_3e618620","updated":"2022-12-13 01:36:43.000000000","message":"I will fix.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[{"line_number":259,"context_line":""},{"line_number":260,"context_line":"    if mgd_vls:"},{"line_number":261,"context_line":"        # convert ExtVirtualLinkInfo to ExtManagedVirtualLinkData."},{"line_number":262,"context_line":"        # necessary member only."},{"line_number":263,"context_line":"        mgd_vls \u003d ["},{"line_number":264,"context_line":"            {"},{"line_number":265,"context_line":"                \u0027vnfVirtualLinkDescId\u0027: mgd_vl[\u0027vnfVirtualLinkDescId\u0027],"}],"source_content_type":"text/x-python","patch_set":6,"id":"15f58067_7146de65","line":262,"range":{"start_line":262,"start_character":20,"end_line":262,"end_character":26},"in_reply_to":"6bb5baeb_620bae07","updated":"2022-12-13 06:31:32.000000000","message":"Ack","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"}],"tacker/sol_refactored/nfvo/local_nfvo.py":[{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":true,"context_lines":[{"line_number":177,"context_line":"            elif \u0027file\u0027 in sw_data:"},{"line_number":178,"context_line":"                # if artifact is specified, create glance image."},{"line_number":179,"context_line":"                # it may fail. catch exception and raise 403(not granted)"},{"line_number":180,"context_line":"                # if error occur."},{"line_number":181,"context_line":"                try:"},{"line_number":182,"context_line":"                    image \u003d self._glance_create_image(vim_info, vnfd, sw_data,"},{"line_number":183,"context_line":"                        grant_req.vnfInstanceId)"}],"source_content_type":"text/x-python","patch_set":6,"id":"f656a67e_f5c5e468","line":180,"updated":"2022-12-12 23:59:34.000000000","message":"Nits, occur-\u003eoccurs.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[{"line_number":177,"context_line":"            elif \u0027file\u0027 in sw_data:"},{"line_number":178,"context_line":"                # if artifact is specified, create glance image."},{"line_number":179,"context_line":"                # it may fail. catch exception and raise 403(not granted)"},{"line_number":180,"context_line":"                # if error occur."},{"line_number":181,"context_line":"                try:"},{"line_number":182,"context_line":"                    image \u003d self._glance_create_image(vim_info, vnfd, sw_data,"},{"line_number":183,"context_line":"                        grant_req.vnfInstanceId)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7d9e7bde_80c5ecb1","line":180,"in_reply_to":"02f56476_2780e27d","updated":"2022-12-13 06:31:32.000000000","message":"Ack","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[{"line_number":177,"context_line":"            elif \u0027file\u0027 in sw_data:"},{"line_number":178,"context_line":"                # if artifact is specified, create glance image."},{"line_number":179,"context_line":"                # it may fail. catch exception and raise 403(not granted)"},{"line_number":180,"context_line":"                # if error occur."},{"line_number":181,"context_line":"                try:"},{"line_number":182,"context_line":"                    image \u003d self._glance_create_image(vim_info, vnfd, sw_data,"},{"line_number":183,"context_line":"                        grant_req.vnfInstanceId)"}],"source_content_type":"text/x-python","patch_set":6,"id":"02f56476_2780e27d","line":180,"in_reply_to":"f656a67e_f5c5e468","updated":"2022-12-13 01:36:43.000000000","message":"I will fix.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":true,"context_lines":[{"line_number":231,"context_line":"        # handle ZoneInfo"},{"line_number":232,"context_line":"        zone_id \u003d self._handle_zoneinfo(grant_res)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"        # handle addResources."},{"line_number":235,"context_line":"        self._handle_addresources(grant_req, grant_res, zone_id)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"        # handle vimAssets"}],"source_content_type":"text/x-python","patch_set":6,"id":"d69cea7c_2352e41c","line":234,"updated":"2022-12-12 23:59:34.000000000","message":"Nits. It is better to omit dot like ZoneInfo and vimAssets.\n\"# handle addResources\"","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[{"line_number":231,"context_line":"        # handle ZoneInfo"},{"line_number":232,"context_line":"        zone_id \u003d self._handle_zoneinfo(grant_res)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"        # handle addResources."},{"line_number":235,"context_line":"        self._handle_addresources(grant_req, grant_res, zone_id)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"        # handle vimAssets"}],"source_content_type":"text/x-python","patch_set":6,"id":"0d7702ce_4693a158","line":234,"in_reply_to":"3ccef618_120a92e2","updated":"2022-12-13 06:31:32.000000000","message":"Ack","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[{"line_number":231,"context_line":"        # handle ZoneInfo"},{"line_number":232,"context_line":"        zone_id \u003d self._handle_zoneinfo(grant_res)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"        # handle addResources."},{"line_number":235,"context_line":"        self._handle_addresources(grant_req, grant_res, zone_id)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"        # handle vimAssets"}],"source_content_type":"text/x-python","patch_set":6,"id":"3ccef618_120a92e2","line":234,"in_reply_to":"d69cea7c_2352e41c","updated":"2022-12-13 01:36:43.000000000","message":"I will fix.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":true,"context_lines":[{"line_number":254,"context_line":"        # handle ZoneInfo"},{"line_number":255,"context_line":"        zone_id \u003d self._handle_zoneinfo(grant_res)"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        # handle addResources."},{"line_number":258,"context_line":"        self._handle_addresources(grant_req, grant_res, zone_id)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        # handle vimAssets"}],"source_content_type":"text/x-python","patch_set":6,"id":"61584aa2_05635adc","line":257,"updated":"2022-12-12 23:59:34.000000000","message":"ditto.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[{"line_number":254,"context_line":"        # handle ZoneInfo"},{"line_number":255,"context_line":"        zone_id \u003d self._handle_zoneinfo(grant_res)"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        # handle addResources."},{"line_number":258,"context_line":"        self._handle_addresources(grant_req, grant_res, zone_id)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        # handle vimAssets"}],"source_content_type":"text/x-python","patch_set":6,"id":"b8d458d6_baec0969","line":257,"in_reply_to":"14e1512c_732660cd","updated":"2022-12-13 06:31:32.000000000","message":"Ack","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[{"line_number":254,"context_line":"        # handle ZoneInfo"},{"line_number":255,"context_line":"        zone_id \u003d self._handle_zoneinfo(grant_res)"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        # handle addResources."},{"line_number":258,"context_line":"        self._handle_addresources(grant_req, grant_res, zone_id)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        # handle vimAssets"}],"source_content_type":"text/x-python","patch_set":6,"id":"14e1512c_732660cd","line":257,"in_reply_to":"61584aa2_05635adc","updated":"2022-12-13 01:36:43.000000000","message":"I will fix.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"}],"tacker/tests/functional/sol_v2/test_individual_vnfc_mgmt.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"363b14af75ab80dedaed81fe76fd23f24577a69b","unresolved":true,"context_lines":[{"line_number":500,"context_line":"        self.assertEqual({0}, self._get_vdu_indexes(inst_1, \u0027VDU2\u0027))"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"        # 2. Update VNF instance"},{"line_number":503,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":504,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"},{"line_number":505,"context_line":"        self.assertEqual(202, resp.status_code)"},{"line_number":506,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"e38b86a4_a02bd7fc","line":503,"updated":"2022-11-14 10:23:12.000000000","message":"Could you please add the update_req creation process to pkggen.py?\nhttps://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/sol_v2_common/samples/userdata_standard/pkggen.py\n\nAlso, the file name of the update request for basic_lcms_min is \"update_seq\", so could you change it to \"update_req\"?\nhttps://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/sol_v2_common/samples/basic_lcms_min/pkggen.py#L66","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cb0709f11a9db5ba1bbff762a07ebd39479c55f6","unresolved":false,"context_lines":[{"line_number":500,"context_line":"        self.assertEqual({0}, self._get_vdu_indexes(inst_1, \u0027VDU2\u0027))"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"        # 2. Update VNF instance"},{"line_number":503,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":504,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"},{"line_number":505,"context_line":"        self.assertEqual(202, resp.status_code)"},{"line_number":506,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"67f85e12_842ec7d9","line":503,"in_reply_to":"142fac0f_24f79587","updated":"2022-11-16 05:56:25.000000000","message":"Done","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"148bee05b5dcaefebf941b83a0dd0b4d6fb00c70","unresolved":true,"context_lines":[{"line_number":500,"context_line":"        self.assertEqual({0}, self._get_vdu_indexes(inst_1, \u0027VDU2\u0027))"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"        # 2. Update VNF instance"},{"line_number":503,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":504,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"},{"line_number":505,"context_line":"        self.assertEqual(202, resp.status_code)"},{"line_number":506,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fcd9f514_5c52e814","line":503,"in_reply_to":"e38b86a4_a02bd7fc","updated":"2022-11-15 01:41:48.000000000","message":"I will fix. The file name of the update request for \"server_notification\" is also \"update_seq\", should I fix this too?","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"16749a3e2369eb9300357c35947cbb48c0bc1637","unresolved":true,"context_lines":[{"line_number":500,"context_line":"        self.assertEqual({0}, self._get_vdu_indexes(inst_1, \u0027VDU2\u0027))"},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"        # 2. Update VNF instance"},{"line_number":503,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":504,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"},{"line_number":505,"context_line":"        self.assertEqual(202, resp.status_code)"},{"line_number":506,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"142fac0f_24f79587","line":503,"in_reply_to":"fcd9f514_5c52e814","updated":"2022-11-15 01:50:43.000000000","message":"Thank you for confirmation.\nPlease fix the \"server_notification\" in the same way.","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"363b14af75ab80dedaed81fe76fd23f24577a69b","unresolved":true,"context_lines":[{"line_number":544,"context_line":"        # check image of VDU1-1 is changed and other are not."},{"line_number":545,"context_line":"        self.assertEqual(self._get_vnfc_image(inst_2, \u0027VDU1\u0027, 0),"},{"line_number":546,"context_line":"                         self._get_vnfc_image(inst_3, \u0027VDU1\u0027, 0))"},{"line_number":547,"context_line":"        self.assertNotEqual(self._get_vnfc_image(inst_2, \u0027VDU1\u0027, 1),"},{"line_number":548,"context_line":"                            self._get_vnfc_image(inst_3, \u0027VDU1\u0027, 1))"},{"line_number":549,"context_line":"        self.assertEqual(self._get_vnfc_image(inst_2, \u0027VDU2\u0027, 0),"},{"line_number":550,"context_line":"                         self._get_vnfc_image(inst_3, \u0027VDU2\u0027, 0))"}],"source_content_type":"text/x-python","patch_set":2,"id":"1159aeb2_d91c94f9","line":547,"updated":"2022-11-14 10:23:12.000000000","message":"Is it the intended process that the name of the image is set in the metadata before heal and the id of the image is set after heal?","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"148bee05b5dcaefebf941b83a0dd0b4d6fb00c70","unresolved":true,"context_lines":[{"line_number":544,"context_line":"        # check image of VDU1-1 is changed and other are not."},{"line_number":545,"context_line":"        self.assertEqual(self._get_vnfc_image(inst_2, \u0027VDU1\u0027, 0),"},{"line_number":546,"context_line":"                         self._get_vnfc_image(inst_3, \u0027VDU1\u0027, 0))"},{"line_number":547,"context_line":"        self.assertNotEqual(self._get_vnfc_image(inst_2, \u0027VDU1\u0027, 1),"},{"line_number":548,"context_line":"                            self._get_vnfc_image(inst_3, \u0027VDU1\u0027, 1))"},{"line_number":549,"context_line":"        self.assertEqual(self._get_vnfc_image(inst_2, \u0027VDU2\u0027, 0),"},{"line_number":550,"context_line":"                         self._get_vnfc_image(inst_3, \u0027VDU2\u0027, 0))"}],"source_content_type":"text/x-python","patch_set":2,"id":"ef85a71a_a5e202a6","line":547,"in_reply_to":"1159aeb2_d91c94f9","updated":"2022-11-15 01:41:48.000000000","message":"It is intended. The image changes after heal, It confirmed that only heal vnfc is changed.","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cb0709f11a9db5ba1bbff762a07ebd39479c55f6","unresolved":false,"context_lines":[{"line_number":544,"context_line":"        # check image of VDU1-1 is changed and other are not."},{"line_number":545,"context_line":"        self.assertEqual(self._get_vnfc_image(inst_2, \u0027VDU1\u0027, 0),"},{"line_number":546,"context_line":"                         self._get_vnfc_image(inst_3, \u0027VDU1\u0027, 0))"},{"line_number":547,"context_line":"        self.assertNotEqual(self._get_vnfc_image(inst_2, \u0027VDU1\u0027, 1),"},{"line_number":548,"context_line":"                            self._get_vnfc_image(inst_3, \u0027VDU1\u0027, 1))"},{"line_number":549,"context_line":"        self.assertEqual(self._get_vnfc_image(inst_2, \u0027VDU2\u0027, 0),"},{"line_number":550,"context_line":"                         self._get_vnfc_image(inst_3, \u0027VDU2\u0027, 0))"}],"source_content_type":"text/x-python","patch_set":2,"id":"c1db182d_a9ddc4df","line":547,"in_reply_to":"ef85a71a_a5e202a6","updated":"2022-11-16 05:56:25.000000000","message":"Thank you for your reply.\nI understood that whether the image name is set or the ID is set depends on whether or not the image is included in the vnf package.","commit_id":"60d32310921793fffb4243da21db94a28795a381"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":true,"context_lines":[{"line_number":457,"context_line":""},{"line_number":458,"context_line":"        * Note:"},{"line_number":459,"context_line":"          This test focuses on checking changed image when Heal operation"},{"line_number":460,"context_line":"          run afetr the vnfdId is changed by modify operation."},{"line_number":461,"context_line":""},{"line_number":462,"context_line":"        * About LCM operations:"},{"line_number":463,"context_line":"          This test includes the following operations."}],"source_content_type":"text/x-python","patch_set":6,"id":"1cc961f0_a1f26f4b","line":460,"range":{"start_line":460,"start_character":14,"end_line":460,"end_character":19},"updated":"2022-12-12 23:59:34.000000000","message":"Typo.\nafetr -\u003e after","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[{"line_number":457,"context_line":""},{"line_number":458,"context_line":"        * Note:"},{"line_number":459,"context_line":"          This test focuses on checking changed image when Heal operation"},{"line_number":460,"context_line":"          run afetr the vnfdId is changed by modify operation."},{"line_number":461,"context_line":""},{"line_number":462,"context_line":"        * About LCM operations:"},{"line_number":463,"context_line":"          This test includes the following operations."}],"source_content_type":"text/x-python","patch_set":6,"id":"7ac9f46b_04791f34","line":460,"range":{"start_line":460,"start_character":14,"end_line":460,"end_character":19},"in_reply_to":"1cc961f0_a1f26f4b","updated":"2022-12-13 01:36:43.000000000","message":"I will fix.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[{"line_number":457,"context_line":""},{"line_number":458,"context_line":"        * Note:"},{"line_number":459,"context_line":"          This test focuses on checking changed image when Heal operation"},{"line_number":460,"context_line":"          run afetr the vnfdId is changed by modify operation."},{"line_number":461,"context_line":""},{"line_number":462,"context_line":"        * About LCM operations:"},{"line_number":463,"context_line":"          This test includes the following operations."}],"source_content_type":"text/x-python","patch_set":6,"id":"009521ae_5c5c05ab","line":460,"range":{"start_line":460,"start_character":14,"end_line":460,"end_character":19},"in_reply_to":"7ac9f46b_04791f34","updated":"2022-12-13 06:31:32.000000000","message":"Ack","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"dd416260f465224cf0b14ab9b0e47de183a6659f","unresolved":true,"context_lines":[{"line_number":558,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":559,"context_line":""},{"line_number":560,"context_line":"        # Delete VNF instance"},{"line_number":561,"context_line":"        self._delete_instance(inst_id)"}],"source_content_type":"text/x-python","patch_set":6,"id":"d4410bd5_5d945fa6","line":561,"updated":"2022-12-12 23:59:34.000000000","message":"Is it unnecessary to check resp.status_code in delete operation?","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"c8906698e0c864ba77d32dbfd4a37c403b2ebb3a","unresolved":false,"context_lines":[{"line_number":558,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":559,"context_line":""},{"line_number":560,"context_line":"        # Delete VNF instance"},{"line_number":561,"context_line":"        self._delete_instance(inst_id)"}],"source_content_type":"text/x-python","patch_set":6,"id":"d66d21a8_c92d40f1","line":561,"in_reply_to":"9d550f2e_1fc5a406","updated":"2022-12-13 06:31:32.000000000","message":"Thanks for reply. I understood.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"19204d4222862fb5914055505828a05a95bfaff6","unresolved":true,"context_lines":[{"line_number":558,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":559,"context_line":""},{"line_number":560,"context_line":"        # Delete VNF instance"},{"line_number":561,"context_line":"        self._delete_instance(inst_id)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9d550f2e_1fc5a406","line":561,"in_reply_to":"d4410bd5_5d945fa6","updated":"2022-12-13 01:36:43.000000000","message":"The resp.status_code is checked in \"_delete_instane\" method.","commit_id":"41e920515aeefa4ee3a16672bac4ba6ab98e5393"}],"tacker/tests/functional/sol_v2_common/paramgen.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"092b79998b964e0b332e91ee78bba5ac88c1d29a","unresolved":true,"context_lines":[{"line_number":576,"context_line":""},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"def update_vnf_min():"},{"line_number":579,"context_line":"    # Omit except for required attributes"},{"line_number":580,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":581,"context_line":"    #  - 1"},{"line_number":582,"context_line":"    #  - 1..N (1)"},{"line_number":583,"context_line":"    return {"},{"line_number":584,"context_line":"        \"vnfInstanceName\": \"new name\""},{"line_number":585,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"3d5fa57e_1bce2f54","line":582,"range":{"start_line":579,"start_character":0,"end_line":582,"end_character":17},"updated":"2022-11-15 08:13:03.000000000","message":"VnfInfoModificationRequest has no required attributes...\nCould you please change it to an appropriate comment?","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"420f3b4d02db918e9871d5e22dc1bc170d4fb4db","unresolved":true,"context_lines":[{"line_number":576,"context_line":""},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"def update_vnf_min():"},{"line_number":579,"context_line":"    # Omit except for required attributes"},{"line_number":580,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":581,"context_line":"    #  - 1"},{"line_number":582,"context_line":"    #  - 1..N (1)"},{"line_number":583,"context_line":"    return {"},{"line_number":584,"context_line":"        \"vnfInstanceName\": \"new name\""},{"line_number":585,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"ef4ca948_0c8edf71","line":582,"range":{"start_line":579,"start_character":0,"end_line":582,"end_character":17},"in_reply_to":"3d5fa57e_1bce2f54","updated":"2022-11-16 03:22:18.000000000","message":"As it is not a point fixed this time, How about in another FT modified timing?","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cb0709f11a9db5ba1bbff762a07ebd39479c55f6","unresolved":false,"context_lines":[{"line_number":576,"context_line":""},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"def update_vnf_min():"},{"line_number":579,"context_line":"    # Omit except for required attributes"},{"line_number":580,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":581,"context_line":"    #  - 1"},{"line_number":582,"context_line":"    #  - 1..N (1)"},{"line_number":583,"context_line":"    return {"},{"line_number":584,"context_line":"        \"vnfInstanceName\": \"new name\""},{"line_number":585,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"eea490d2_acc99e30","line":582,"range":{"start_line":579,"start_character":0,"end_line":582,"end_character":17},"in_reply_to":"ef4ca948_0c8edf71","updated":"2022-11-16 05:56:25.000000000","message":"Yes, it is okay.\nIt was a comment on the assumption that this method would be modified (commonized).\nIf this method is not corrected as answered in other comments, this correction is not necessary.","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"092b79998b964e0b332e91ee78bba5ac88c1d29a","unresolved":true,"context_lines":[{"line_number":586,"context_line":""},{"line_number":587,"context_line":""},{"line_number":588,"context_line":"def update_vnf_min_with_parameter(vnfd_id):"},{"line_number":589,"context_line":"    # Omit except for required attributes"},{"line_number":590,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":591,"context_line":"    #  - 1"},{"line_number":592,"context_line":"    #  - 1..N (1)"},{"line_number":593,"context_line":"    return {"},{"line_number":594,"context_line":"        \"vnfdId\": vnfd_id"},{"line_number":595,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"500a17de_8bc56c55","line":592,"range":{"start_line":589,"start_character":0,"end_line":592,"end_character":17},"updated":"2022-11-15 08:13:03.000000000","message":"ditto","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cb0709f11a9db5ba1bbff762a07ebd39479c55f6","unresolved":false,"context_lines":[{"line_number":586,"context_line":""},{"line_number":587,"context_line":""},{"line_number":588,"context_line":"def update_vnf_min_with_parameter(vnfd_id):"},{"line_number":589,"context_line":"    # Omit except for required attributes"},{"line_number":590,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":591,"context_line":"    #  - 1"},{"line_number":592,"context_line":"    #  - 1..N (1)"},{"line_number":593,"context_line":"    return {"},{"line_number":594,"context_line":"        \"vnfdId\": vnfd_id"},{"line_number":595,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"a75e7976_d6968802","line":592,"range":{"start_line":589,"start_character":0,"end_line":592,"end_character":17},"in_reply_to":"12a4105a_c3e61ebf","updated":"2022-11-16 05:56:25.000000000","message":"ditto","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"420f3b4d02db918e9871d5e22dc1bc170d4fb4db","unresolved":true,"context_lines":[{"line_number":586,"context_line":""},{"line_number":587,"context_line":""},{"line_number":588,"context_line":"def update_vnf_min_with_parameter(vnfd_id):"},{"line_number":589,"context_line":"    # Omit except for required attributes"},{"line_number":590,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":591,"context_line":"    #  - 1"},{"line_number":592,"context_line":"    #  - 1..N (1)"},{"line_number":593,"context_line":"    return {"},{"line_number":594,"context_line":"        \"vnfdId\": vnfd_id"},{"line_number":595,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"12a4105a_c3e61ebf","line":592,"range":{"start_line":589,"start_character":0,"end_line":592,"end_character":17},"in_reply_to":"500a17de_8bc56c55","updated":"2022-11-16 03:22:18.000000000","message":"ditto","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"092b79998b964e0b332e91ee78bba5ac88c1d29a","unresolved":true,"context_lines":[{"line_number":1105,"context_line":"    }"},{"line_number":1106,"context_line":""},{"line_number":1107,"context_line":""},{"line_number":1108,"context_line":"def sample3_update_vnf_vnfd_id(vnfd_id):"},{"line_number":1109,"context_line":"    # Omit except for required attributes"},{"line_number":1110,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":1111,"context_line":"    #  - 1"},{"line_number":1112,"context_line":"    #  - 1..N (1)"},{"line_number":1113,"context_line":"    return {"},{"line_number":1114,"context_line":"        \"vnfdId\": vnfd_id"},{"line_number":1115,"context_line":"    }"},{"line_number":1116,"context_line":""},{"line_number":1117,"context_line":""},{"line_number":1118,"context_line":"# sample4 is for change_vnfpkg test of StandardUserData"}],"source_content_type":"text/x-python","patch_set":3,"id":"26843668_443783ad","line":1115,"range":{"start_line":1108,"start_character":0,"end_line":1115,"end_character":5},"updated":"2022-11-15 08:13:03.000000000","message":"This is exactly the same as update_vnf_min_with_parameter, so I think it\u0027s better to use it in common for \"basic_lcms_min\" and \"userdata_standard\" without defining a new one.\nWhat do you think?","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"420f3b4d02db918e9871d5e22dc1bc170d4fb4db","unresolved":true,"context_lines":[{"line_number":1105,"context_line":"    }"},{"line_number":1106,"context_line":""},{"line_number":1107,"context_line":""},{"line_number":1108,"context_line":"def sample3_update_vnf_vnfd_id(vnfd_id):"},{"line_number":1109,"context_line":"    # Omit except for required attributes"},{"line_number":1110,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":1111,"context_line":"    #  - 1"},{"line_number":1112,"context_line":"    #  - 1..N (1)"},{"line_number":1113,"context_line":"    return {"},{"line_number":1114,"context_line":"        \"vnfdId\": vnfd_id"},{"line_number":1115,"context_line":"    }"},{"line_number":1116,"context_line":""},{"line_number":1117,"context_line":""},{"line_number":1118,"context_line":"# sample4 is for change_vnfpkg test of StandardUserData"}],"source_content_type":"text/x-python","patch_set":3,"id":"9cfba5a6_d1251469","line":1115,"range":{"start_line":1108,"start_character":0,"end_line":1115,"end_character":5},"in_reply_to":"26843668_443783ad","updated":"2022-11-16 03:22:18.000000000","message":"Same as \"update_vnf_min_with_parameter”, but the function name is unclear, I added it for pkggen.py of userdata_standard. I use it in \"userdata_standard FT code, how about? I will correct the comment.","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cb0709f11a9db5ba1bbff762a07ebd39479c55f6","unresolved":false,"context_lines":[{"line_number":1105,"context_line":"    }"},{"line_number":1106,"context_line":""},{"line_number":1107,"context_line":""},{"line_number":1108,"context_line":"def sample3_update_vnf_vnfd_id(vnfd_id):"},{"line_number":1109,"context_line":"    # Omit except for required attributes"},{"line_number":1110,"context_line":"    # NOTE: Only the following cardinality attributes are set."},{"line_number":1111,"context_line":"    #  - 1"},{"line_number":1112,"context_line":"    #  - 1..N (1)"},{"line_number":1113,"context_line":"    return {"},{"line_number":1114,"context_line":"        \"vnfdId\": vnfd_id"},{"line_number":1115,"context_line":"    }"},{"line_number":1116,"context_line":""},{"line_number":1117,"context_line":""},{"line_number":1118,"context_line":"# sample4 is for change_vnfpkg test of StandardUserData"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ca32911_6da984be","line":1115,"range":{"start_line":1108,"start_character":0,"end_line":1115,"end_character":5},"in_reply_to":"9cfba5a6_d1251469","updated":"2022-11-16 05:56:25.000000000","message":"Thank you for your opinion.\nIt was supposed to rename \"update_vnf_min_with_parameter\" for common use, but if readability is better as it is, no modification is required.","commit_id":"0be1ef927603b3458a5ff2fcebb2bfffef52a504"}]}