)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1a74d3b5b5b9ad3e592e886533eb1c9c74f93f61","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3fda2f31_44f5a233","line":14,"updated":"2023-04-28 01:08:45.000000000","message":"nit: repeatedly execute -\u003e retry","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"f7040093_101597f1","line":14,"in_reply_to":"322d4e44_b026fdcb","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"049c4833ef4697d2e461977c47361055855782f3","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"552ce913_6b7e5964","line":14,"in_reply_to":"3fda2f31_44f5a233","updated":"2023-05-09 04:21:56.000000000","message":"I fixed it in PS6.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"322d4e44_b026fdcb","line":14,"in_reply_to":"552ce913_6b7e5964","updated":"2023-05-09 06:45:30.000000000","message":"I don\u0027t think \"repeatably\" is necessary.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5b97ce78_1bab859c","line":14,"in_reply_to":"f7040093_101597f1","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1a74d3b5b5b9ad3e592e886533eb1c9c74f93f61","unresolved":true,"context_lines":[{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"af7cc988_8db3177b","line":15,"updated":"2023-04-28 01:08:45.000000000","message":"nit: epeatedly executes -\u003e retries","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"080cdff7_e83d65d0","line":15,"in_reply_to":"15ca0a7b_a3dd1e7b","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"c4cf9938_4f9b4a7b","line":15,"in_reply_to":"89040e4a_391e13fd","updated":"2023-05-09 06:45:30.000000000","message":"I don\u0027t think \"repeatably\" is necessary.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"049c4833ef4697d2e461977c47361055855782f3","unresolved":true,"context_lines":[{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"89040e4a_391e13fd","line":15,"in_reply_to":"af7cc988_8db3177b","updated":"2023-05-09 04:21:56.000000000","message":"I fixed it in PS6.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to repeatedly execute delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that repeatedly executes"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"15ca0a7b_a3dd1e7b","line":15,"in_reply_to":"c4cf9938_4f9b4a7b","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to retry delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that retries"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"75eefac6_81712bb0","line":16,"range":{"start_line":14,"start_character":0,"end_line":16,"end_character":35},"updated":"2023-05-10 06:50:39.000000000","message":"It seems that the target is not only delete, so how about changing it as follows?\n* Define a repeat execution method of lcm operation in BaseTackerTestV2. And remove this processing from each FT.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to retry delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that retries"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"d07bff3a_b9d33181","line":16,"range":{"start_line":14,"start_character":0,"end_line":16,"end_character":35},"in_reply_to":"306d31e3_bcd557b6","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a method to retry delete_vnf_instance in"},{"line_number":15,"context_line":"  BaseV2Test. And remove the method that retries"},{"line_number":16,"context_line":"  delete_vnf_instance from each FT."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I6230ab56dec3a5c96cd123c62168a6495d8f7092"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"306d31e3_bcd557b6","line":16,"range":{"start_line":14,"start_character":0,"end_line":16,"end_character":35},"in_reply_to":"75eefac6_81712bb0","updated":"2023-05-10 23:54:49.000000000","message":"I agree with you.\nI fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a repeat execution method of lcm operation in BaseTackerTestV2."},{"line_number":15,"context_line":"  And remove this processing from each FT."},{"line_number":16,"context_line":"* Change sleep times used in v2 API FT to the following constant."},{"line_number":17,"context_line":"   - WAIT_LCMOCC_UPDATE_TIME"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"5c20b538_4d36fe3d","line":14,"range":{"start_line":14,"start_character":38,"end_line":14,"end_character":41},"updated":"2023-06-09 05:10:13.000000000","message":"nits: lcm -\u003e LCM","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a repeat execution method of lcm operation in BaseTackerTestV2."},{"line_number":15,"context_line":"  And remove this processing from each FT."},{"line_number":16,"context_line":"* Change sleep times used in v2 API FT to the following constant."},{"line_number":17,"context_line":"   - WAIT_LCMOCC_UPDATE_TIME"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"d51bd3c1_c120d56a","line":14,"range":{"start_line":14,"start_character":38,"end_line":14,"end_character":41},"in_reply_to":"320272a3_b552652b","updated":"2023-06-14 17:26:12.000000000","message":"Done","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  to BaseV2Test."},{"line_number":12,"context_line":"* Rename duplicate method names in BaseSolV2Test and"},{"line_number":13,"context_line":"  BaseVnfLcmKubernetesV2Test."},{"line_number":14,"context_line":"* Define a repeat execution method of lcm operation in BaseTackerTestV2."},{"line_number":15,"context_line":"  And remove this processing from each FT."},{"line_number":16,"context_line":"* Change sleep times used in v2 API FT to the following constant."},{"line_number":17,"context_line":"   - WAIT_LCMOCC_UPDATE_TIME"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"320272a3_b552652b","line":14,"range":{"start_line":14,"start_character":38,"end_line":14,"end_character":41},"in_reply_to":"5c20b538_4d36fe3d","updated":"2023-06-12 05:57:13.000000000","message":"I fixed it.","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1a74d3b5b5b9ad3e592e886533eb1c9c74f93f61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"12bded24_06c406b8","updated":"2023-04-28 01:08:45.000000000","message":"The following are also V2 API FTs. Is it okay if they are not modified?\n* https://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/sol_enhanced_policy/sol/test_policy_vnflcm_apis_v2.py\n* https://opendev.org/openstack/tacker/src/branch/master/tacker/tests/functional/sol_enhanced_policy/sol_kubernetes/test_policy_cnflcm_apis_v2.py","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"6f7668db07d7c014f8767744399e866748b99992","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1a7b2940_86fe15fb","updated":"2023-05-09 04:04:19.000000000","message":"recheck - infra failure","commit_id":"60edfe0fe3f8513bd2e78a38000d3ec4506bdc85"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"62b04324_2dfc0932","updated":"2023-05-11 02:43:37.000000000","message":"LGTM","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"7ec08658e0ea822acb9cb037d47a43362249f0bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"99a2987e_36ee0174","updated":"2023-06-05 03:05:26.000000000","message":"Thank you for the review.","commit_id":"638337d83f1a8366a6faa1d2708c831885a9ea80"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"35eafab4_95bf18f3","updated":"2023-06-09 05:10:13.000000000","message":"Thank you for your patch. Please kindly check my comments.","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b724daaa94c7838fb82072b9a821427dc93456c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"50baeab3_606dcad5","updated":"2023-06-21 08:28:02.000000000","message":"Sorry for the late review, please kindly find my comments.","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"8e0b4a01_a5619b5c","updated":"2023-06-12 05:57:13.000000000","message":"Thank you for the review.","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"86c8ef72_2fc9ddb8","updated":"2023-06-14 17:26:12.000000000","message":"Thanks for your re-work!LGTM","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c20d15d9bf44135b5df988f28a3e322998de8d9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"8a74084b_8d75f201","updated":"2023-06-21 10:20:33.000000000","message":"Thank you for the review.","commit_id":"79e247778cf68c8625343f0ae9451fd3d30ec166"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"92566f12398e676eeb7ec052f45c937ec471ac2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"aab0b5bb_f20e4fab","updated":"2023-06-27 01:34:32.000000000","message":"Thanks for your rework, LGTM.","commit_id":"7eaf841525d7ead30d7883abbde6c4fb86ae2e4f"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"e062f4cff8691328d4b74339363b7fc0a8a746d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"fd245008_7199e457","updated":"2023-06-25 22:05:04.000000000","message":"recheck - TIMED_OUT","commit_id":"7eaf841525d7ead30d7883abbde6c4fb86ae2e4f"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"9d84341d038fafedfc569788ab49c1c74669a0e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"8e072f41_2dab12db","updated":"2023-06-26 23:21:01.000000000","message":"recheck - infra failure","commit_id":"7eaf841525d7ead30d7883abbde6c4fb86ae2e4f"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"52379b4f42b9e56b8ef0703f981a0d5894207ad2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"30e260a3_74e7888d","updated":"2023-06-22 23:02:10.000000000","message":"recheck infra-failure","commit_id":"7eaf841525d7ead30d7883abbde6c4fb86ae2e4f"}],"tacker/tests/functional/base_v2.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"ab81f79aa7f7b512348fd5291b1f9370c7f6bd33","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    def create_vnf_instance(self, req_body):"},{"line_number":183,"context_line":"        path \u003d \"/vnflcm/v2/vnf_instances\""},{"line_number":184,"context_line":"        return self.tacker_client.do_request("},{"line_number":185,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.0.0\")"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"    def instantiate_vnf_instance(self, inst_id, req_body):"},{"line_number":188,"context_line":"        path \u003d f\"/vnflcm/v2/vnf_instances/{inst_id}/instantiate\""}],"source_content_type":"text/x-python","patch_set":9,"id":"992769d1_601588c6","line":185,"range":{"start_line":185,"start_character":49,"end_line":185,"end_character":56},"updated":"2023-05-29 08:19:26.000000000","message":"Why don\u0027t you use a constant to avoid a silly mistake for all args because the purpose of the update is refactoring?","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"7ec08658e0ea822acb9cb037d47a43362249f0bd","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    def create_vnf_instance(self, req_body):"},{"line_number":183,"context_line":"        path \u003d \"/vnflcm/v2/vnf_instances\""},{"line_number":184,"context_line":"        return self.tacker_client.do_request("},{"line_number":185,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.0.0\")"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"    def instantiate_vnf_instance(self, inst_id, req_body):"},{"line_number":188,"context_line":"        path \u003d f\"/vnflcm/v2/vnf_instances/{inst_id}/instantiate\""}],"source_content_type":"text/x-python","patch_set":9,"id":"5cdd72ce_85400db3","line":185,"range":{"start_line":185,"start_character":49,"end_line":185,"end_character":56},"in_reply_to":"992769d1_601588c6","updated":"2023-06-05 03:05:26.000000000","message":"I fixed it.\nIt has also been added to the CommitMessage as a correction item.","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"ab81f79aa7f7b512348fd5291b1f9370c7f6bd33","unresolved":true,"context_lines":[{"line_number":331,"context_line":"            state \u003d body[\u0027operationState\u0027]"},{"line_number":332,"context_line":"            if state \u003d\u003d \u0027ROLLED_BACK\u0027:"},{"line_number":333,"context_line":"                return"},{"line_number":334,"context_line":"            if state in [\u0027ROLLING_BACK\u0027]:"},{"line_number":335,"context_line":"                continue"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"            raise Exception(f\"Operation failed. state: {state}\")"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff9dd9a1_7d86e950","line":334,"updated":"2023-05-29 08:19:26.000000000","message":"Any reason if you don\u0027t define as below?\n```\nelif state \u003d\u003d ROLLING_BACK:\n```","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"7ec08658e0ea822acb9cb037d47a43362249f0bd","unresolved":true,"context_lines":[{"line_number":331,"context_line":"            state \u003d body[\u0027operationState\u0027]"},{"line_number":332,"context_line":"            if state \u003d\u003d \u0027ROLLED_BACK\u0027:"},{"line_number":333,"context_line":"                return"},{"line_number":334,"context_line":"            if state in [\u0027ROLLING_BACK\u0027]:"},{"line_number":335,"context_line":"                continue"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"            raise Exception(f\"Operation failed. state: {state}\")"}],"source_content_type":"text/x-python","patch_set":9,"id":"109fd518_99eaf121","line":334,"in_reply_to":"ff9dd9a1_7d86e950","updated":"2023-06-05 03:05:26.000000000","message":"I fixed it.","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"}],"tacker/tests/functional/sol_enhanced_policy/sol/test_policy_vnflcm_apis_v2.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b724daaa94c7838fb82072b9a821427dc93456c6","unresolved":true,"context_lines":[{"line_number":503,"context_line":"            lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":504,"context_line":"            self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"            # wait a bit because there is a bit time lag between lcmocc DB"},{"line_number":507,"context_line":"            # update and terminate completion."},{"line_number":508,"context_line":"            time.sleep(WAIT_LCMOCC_UPDATE_TIME)"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":511,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"}],"source_content_type":"text/x-python","patch_set":15,"id":"327a2c93_8dcb6013","line":508,"range":{"start_line":506,"start_character":0,"end_line":508,"end_character":47},"updated":"2023-06-21 08:28:02.000000000","message":"According to refactoring policy of other test, how about you delete this sleep in `_step_lcm_terminate()` method and change `self.delete_vnf_instane(inst_id)` in the `_step_lcm_delete()` method to `self.exec_lcm_operation(self.delete_vnf_instance, inst_id)`?","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c20d15d9bf44135b5df988f28a3e322998de8d9d","unresolved":true,"context_lines":[{"line_number":503,"context_line":"            lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":504,"context_line":"            self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"            # wait a bit because there is a bit time lag between lcmocc DB"},{"line_number":507,"context_line":"            # update and terminate completion."},{"line_number":508,"context_line":"            time.sleep(WAIT_LCMOCC_UPDATE_TIME)"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":511,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"}],"source_content_type":"text/x-python","patch_set":15,"id":"6d5e8a97_9b938139","line":508,"range":{"start_line":506,"start_character":0,"end_line":508,"end_character":47},"in_reply_to":"327a2c93_8dcb6013","updated":"2023-06-21 10:20:33.000000000","message":"I fixed it.","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"92566f12398e676eeb7ec052f45c937ec471ac2c","unresolved":false,"context_lines":[{"line_number":503,"context_line":"            lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":504,"context_line":"            self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"            # wait a bit because there is a bit time lag between lcmocc DB"},{"line_number":507,"context_line":"            # update and terminate completion."},{"line_number":508,"context_line":"            time.sleep(WAIT_LCMOCC_UPDATE_TIME)"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":511,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"}],"source_content_type":"text/x-python","patch_set":15,"id":"80e6cc6b_ed97c8cc","line":508,"range":{"start_line":506,"start_character":0,"end_line":508,"end_character":47},"in_reply_to":"6d5e8a97_9b938139","updated":"2023-06-27 01:34:32.000000000","message":"Ack","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b724daaa94c7838fb82072b9a821427dc93456c6","unresolved":true,"context_lines":[{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":511,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"},{"line_number":512,"context_line":"        resp, body \u003d self.delete_vnf_instance(inst_id)"},{"line_number":513,"context_line":"        self.assertEqual(expected_status_code, resp.status_code)"},{"line_number":514,"context_line":""},{"line_number":515,"context_line":"    def vnflcm_apis_v2_vnf_test_before_instantiate(self):"}],"source_content_type":"text/x-python","patch_set":15,"id":"7e4b0d22_9f98d719","line":512,"range":{"start_line":512,"start_character":21,"end_line":512,"end_character":54},"updated":"2023-06-21 08:28:02.000000000","message":"ditto.","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c20d15d9bf44135b5df988f28a3e322998de8d9d","unresolved":true,"context_lines":[{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":511,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"},{"line_number":512,"context_line":"        resp, body \u003d self.delete_vnf_instance(inst_id)"},{"line_number":513,"context_line":"        self.assertEqual(expected_status_code, resp.status_code)"},{"line_number":514,"context_line":""},{"line_number":515,"context_line":"    def vnflcm_apis_v2_vnf_test_before_instantiate(self):"}],"source_content_type":"text/x-python","patch_set":15,"id":"e635e98e_2c52901b","line":512,"range":{"start_line":512,"start_character":21,"end_line":512,"end_character":54},"in_reply_to":"7e4b0d22_9f98d719","updated":"2023-06-21 10:20:33.000000000","message":"ditto","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"92566f12398e676eeb7ec052f45c937ec471ac2c","unresolved":false,"context_lines":[{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":511,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"},{"line_number":512,"context_line":"        resp, body \u003d self.delete_vnf_instance(inst_id)"},{"line_number":513,"context_line":"        self.assertEqual(expected_status_code, resp.status_code)"},{"line_number":514,"context_line":""},{"line_number":515,"context_line":"    def vnflcm_apis_v2_vnf_test_before_instantiate(self):"}],"source_content_type":"text/x-python","patch_set":15,"id":"3ac89001_62c0bd1a","line":512,"range":{"start_line":512,"start_character":21,"end_line":512,"end_character":54},"in_reply_to":"e635e98e_2c52901b","updated":"2023-06-27 01:34:32.000000000","message":"Ack","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"}],"tacker/tests/functional/sol_enhanced_policy/sol_kubernetes/test_policy_cnflcm_apis_v2.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b724daaa94c7838fb82072b9a821427dc93456c6","unresolved":true,"context_lines":[{"line_number":442,"context_line":"            lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":443,"context_line":"            self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"            # wait a bit because there is a bit time lag between lcmocc DB"},{"line_number":446,"context_line":"            # update and terminate completion."},{"line_number":447,"context_line":"            time.sleep(WAIT_LCMOCC_UPDATE_TIME)"},{"line_number":448,"context_line":""},{"line_number":449,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":450,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"}],"source_content_type":"text/x-python","patch_set":15,"id":"3d1cd9e5_8ad147b9","line":447,"range":{"start_line":445,"start_character":0,"end_line":447,"end_character":47},"updated":"2023-06-21 08:28:02.000000000","message":"According to refactoring policy of other test, how about you delete this sleep in `_step_lcm_terminate()` method and change `self.delete_vnf_instane(inst_id)` in the `_step_lcm_delete()` method to `self.exec_lcm_operation(self.delete_vnf_instance, inst_id)`?","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c20d15d9bf44135b5df988f28a3e322998de8d9d","unresolved":true,"context_lines":[{"line_number":442,"context_line":"            lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":443,"context_line":"            self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"            # wait a bit because there is a bit time lag between lcmocc DB"},{"line_number":446,"context_line":"            # update and terminate completion."},{"line_number":447,"context_line":"            time.sleep(WAIT_LCMOCC_UPDATE_TIME)"},{"line_number":448,"context_line":""},{"line_number":449,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":450,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"}],"source_content_type":"text/x-python","patch_set":15,"id":"d831da02_e1600654","line":447,"range":{"start_line":445,"start_character":0,"end_line":447,"end_character":47},"in_reply_to":"3d1cd9e5_8ad147b9","updated":"2023-06-21 10:20:33.000000000","message":"I fixed it.","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"92566f12398e676eeb7ec052f45c937ec471ac2c","unresolved":false,"context_lines":[{"line_number":442,"context_line":"            lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":443,"context_line":"            self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"            # wait a bit because there is a bit time lag between lcmocc DB"},{"line_number":446,"context_line":"            # update and terminate completion."},{"line_number":447,"context_line":"            time.sleep(WAIT_LCMOCC_UPDATE_TIME)"},{"line_number":448,"context_line":""},{"line_number":449,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":450,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"}],"source_content_type":"text/x-python","patch_set":15,"id":"c943203e_77825da1","line":447,"range":{"start_line":445,"start_character":0,"end_line":447,"end_character":47},"in_reply_to":"d831da02_e1600654","updated":"2023-06-27 01:34:32.000000000","message":"Ack","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b724daaa94c7838fb82072b9a821427dc93456c6","unresolved":true,"context_lines":[{"line_number":449,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":450,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"},{"line_number":451,"context_line":"        # Delete a VNF instance"},{"line_number":452,"context_line":"        resp, body \u003d self.delete_vnf_instance(inst_id)"},{"line_number":453,"context_line":"        self.assertEqual(expected_status_code, resp.status_code)"},{"line_number":454,"context_line":"        if expected_status_code \u003d\u003d 204:"},{"line_number":455,"context_line":"            # check deletion of VNF instance"}],"source_content_type":"text/x-python","patch_set":15,"id":"4df65c7c_4715ed0e","line":452,"range":{"start_line":452,"start_character":21,"end_line":452,"end_character":54},"updated":"2023-06-21 08:28:02.000000000","message":"ditto.","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c20d15d9bf44135b5df988f28a3e322998de8d9d","unresolved":true,"context_lines":[{"line_number":449,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":450,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"},{"line_number":451,"context_line":"        # Delete a VNF instance"},{"line_number":452,"context_line":"        resp, body \u003d self.delete_vnf_instance(inst_id)"},{"line_number":453,"context_line":"        self.assertEqual(expected_status_code, resp.status_code)"},{"line_number":454,"context_line":"        if expected_status_code \u003d\u003d 204:"},{"line_number":455,"context_line":"            # check deletion of VNF instance"}],"source_content_type":"text/x-python","patch_set":15,"id":"4eeec939_562b2e11","line":452,"range":{"start_line":452,"start_character":21,"end_line":452,"end_character":54},"in_reply_to":"4df65c7c_4715ed0e","updated":"2023-06-21 10:20:33.000000000","message":"ditto","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"92566f12398e676eeb7ec052f45c937ec471ac2c","unresolved":false,"context_lines":[{"line_number":449,"context_line":"    def _step_lcm_delete(self, username, inst_id, expected_status_code):"},{"line_number":450,"context_line":"        self.tacker_client \u003d self.get_tk_http_client_by_user(username)"},{"line_number":451,"context_line":"        # Delete a VNF instance"},{"line_number":452,"context_line":"        resp, body \u003d self.delete_vnf_instance(inst_id)"},{"line_number":453,"context_line":"        self.assertEqual(expected_status_code, resp.status_code)"},{"line_number":454,"context_line":"        if expected_status_code \u003d\u003d 204:"},{"line_number":455,"context_line":"            # check deletion of VNF instance"}],"source_content_type":"text/x-python","patch_set":15,"id":"8a7ed3e5_3aef002c","line":452,"range":{"start_line":452,"start_character":21,"end_line":452,"end_character":54},"in_reply_to":"4eeec939_562b2e11","updated":"2023-06-27 01:34:32.000000000","message":"Ack","commit_id":"88588d4379357e23866baa69496783cb6af26ed6"}],"tacker/tests/functional/sol_https_v2/test_vnffm_https_basic.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":46,"context_line":"    def setUp(self):"},{"line_number":47,"context_line":"        super(VnfFmWithHttpsRequestTest, self).setUp()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _create_fm_subscription(self, req_body):"},{"line_number":50,"context_line":"        path \u003d \"/vnffm/v1/subscriptions\""},{"line_number":51,"context_line":"        return self.tacker_client.do_request("},{"line_number":52,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"b742edb9_27bc9722","line":49,"updated":"2023-04-28 04:17:18.000000000","message":"It looks the same as below. Why are they defined separately?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_kubernetes_v2/base_v2.py#100","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    def setUp(self):"},{"line_number":47,"context_line":"        super(VnfFmWithHttpsRequestTest, self).setUp()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _create_fm_subscription(self, req_body):"},{"line_number":50,"context_line":"        path \u003d \"/vnffm/v1/subscriptions\""},{"line_number":51,"context_line":"        return self.tacker_client.do_request("},{"line_number":52,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"7918f7da_f74cbbc8","line":49,"in_reply_to":"1ce2f848_77441b8a","updated":"2023-05-09 06:45:30.000000000","message":"Thank you for your explanation.\nStrictly speaking, it is correct to use the CNF package for this test...\nCurrently, no modification is required.\nIt is better to fix it so that the CNF package will be used in another patch in the future.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":46,"context_line":"    def setUp(self):"},{"line_number":47,"context_line":"        super(VnfFmWithHttpsRequestTest, self).setUp()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def _create_fm_subscription(self, req_body):"},{"line_number":50,"context_line":"        path \u003d \"/vnffm/v1/subscriptions\""},{"line_number":51,"context_line":"        return self.tacker_client.do_request("},{"line_number":52,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ce2f848_77441b8a","line":49,"in_reply_to":"b742edb9_27bc9722","updated":"2023-05-09 04:03:53.000000000","message":"PM/FM only support CNF.\nThis test is for VNF. So this test is importing `tacker.tests.functional.sol_v2_common`. `create_fm_subscription` is undefined in `tacker.tests.functional.sol_v2_common`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        return self.tacker_client.do_request("},{"line_number":52,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def _delete_fm_subscription(self, sub_id):"},{"line_number":55,"context_line":"        path \u003d \"/vnffm/v1/subscriptions/{}\".format(sub_id)"},{"line_number":56,"context_line":"        return self.tacker_client.do_request("},{"line_number":57,"context_line":"            path, \"DELETE\", version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"2ca2b6fa_902cd736","line":54,"updated":"2023-04-28 04:17:18.000000000","message":"It looks the same as below. Why are they defined separately?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_kubernetes_v2/base_v2.py#117","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        return self.tacker_client.do_request("},{"line_number":52,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def _delete_fm_subscription(self, sub_id):"},{"line_number":55,"context_line":"        path \u003d \"/vnffm/v1/subscriptions/{}\".format(sub_id)"},{"line_number":56,"context_line":"        return self.tacker_client.do_request("},{"line_number":57,"context_line":"            path, \"DELETE\", version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"dbfba2fa_9833b454","line":54,"in_reply_to":"2ca2b6fa_902cd736","updated":"2023-05-09 04:03:53.000000000","message":"ditto.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        return self.tacker_client.do_request("},{"line_number":52,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def _delete_fm_subscription(self, sub_id):"},{"line_number":55,"context_line":"        path \u003d \"/vnffm/v1/subscriptions/{}\".format(sub_id)"},{"line_number":56,"context_line":"        return self.tacker_client.do_request("},{"line_number":57,"context_line":"            path, \"DELETE\", version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"ccc26060_59fda1bb","line":54,"in_reply_to":"dbfba2fa_9833b454","updated":"2023-05-09 06:45:30.000000000","message":"ditto","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":56,"context_line":"        return self.tacker_client.do_request("},{"line_number":57,"context_line":"            path, \"DELETE\", version\u003d\"1.3.0\")"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def _create_fm_alarm(self, req_body):"},{"line_number":60,"context_line":"        path \u003d \"/alert\""},{"line_number":61,"context_line":"        return self.tacker_client.do_request("},{"line_number":62,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"33a51ad3_efe5af20","line":59,"updated":"2023-04-28 04:17:18.000000000","message":"It looks the same as below. Why are they defined separately?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_kubernetes_v2/base_v2.py#122","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":56,"context_line":"        return self.tacker_client.do_request("},{"line_number":57,"context_line":"            path, \"DELETE\", version\u003d\"1.3.0\")"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def _create_fm_alarm(self, req_body):"},{"line_number":60,"context_line":"        path \u003d \"/alert\""},{"line_number":61,"context_line":"        return self.tacker_client.do_request("},{"line_number":62,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"b1719b95_fb2a8381","line":59,"in_reply_to":"33a51ad3_efe5af20","updated":"2023-05-09 04:03:53.000000000","message":"ditto.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        return self.tacker_client.do_request("},{"line_number":57,"context_line":"            path, \"DELETE\", version\u003d\"1.3.0\")"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def _create_fm_alarm(self, req_body):"},{"line_number":60,"context_line":"        path \u003d \"/alert\""},{"line_number":61,"context_line":"        return self.tacker_client.do_request("},{"line_number":62,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"1.3.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"d44cbcb2_590c8c40","line":59,"in_reply_to":"b1719b95_fb2a8381","updated":"2023-05-09 06:45:30.000000000","message":"ditto","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"}],"tacker/tests/functional/sol_https_v2/test_vnfpm_https_basic.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":59,"context_line":"            \u0027http://\u0027)[1].split(\u0027\"\u0027)[0]"},{"line_number":60,"context_line":"        return ip"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def _create_pm_job(self, req_body):"},{"line_number":63,"context_line":"        path \u003d \"/vnfpm/v2/pm_jobs\""},{"line_number":64,"context_line":"        return self.tacker_client.do_request("},{"line_number":65,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"27e25332_88503003","line":62,"updated":"2023-04-28 04:17:18.000000000","message":"It looks the same as below. Why are they defined separately?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_kubernetes_v2/base_v2.py#144","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":59,"context_line":"            \u0027http://\u0027)[1].split(\u0027\"\u0027)[0]"},{"line_number":60,"context_line":"        return ip"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def _create_pm_job(self, req_body):"},{"line_number":63,"context_line":"        path \u003d \"/vnfpm/v2/pm_jobs\""},{"line_number":64,"context_line":"        return self.tacker_client.do_request("},{"line_number":65,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"ff6df3f0_db4b363f","line":62,"in_reply_to":"27e25332_88503003","updated":"2023-05-09 04:03:53.000000000","message":"ditto.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":59,"context_line":"            \u0027http://\u0027)[1].split(\u0027\"\u0027)[0]"},{"line_number":60,"context_line":"        return ip"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def _create_pm_job(self, req_body):"},{"line_number":63,"context_line":"        path \u003d \"/vnfpm/v2/pm_jobs\""},{"line_number":64,"context_line":"        return self.tacker_client.do_request("},{"line_number":65,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"96d606dc_88f262d3","line":62,"in_reply_to":"ff6df3f0_db4b363f","updated":"2023-05-09 06:45:30.000000000","message":"ditto","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        return self.tacker_client.do_request("},{"line_number":65,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _create_pm_event(self, req_body):"},{"line_number":68,"context_line":"        path \u003d \"/pm_event\""},{"line_number":69,"context_line":"        return self.tacker_client.do_request("},{"line_number":70,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"623923c2_69a9d2c0","line":67,"updated":"2023-04-28 04:17:18.000000000","message":"It looks the same as below. Why are they defined separately?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_kubernetes_v2/base_v2.py#154","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        return self.tacker_client.do_request("},{"line_number":65,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _create_pm_event(self, req_body):"},{"line_number":68,"context_line":"        path \u003d \"/pm_event\""},{"line_number":69,"context_line":"        return self.tacker_client.do_request("},{"line_number":70,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"dbcf7f0f_64c20eb3","line":67,"in_reply_to":"623923c2_69a9d2c0","updated":"2023-05-09 04:03:53.000000000","message":"ditto.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        return self.tacker_client.do_request("},{"line_number":65,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def _create_pm_event(self, req_body):"},{"line_number":68,"context_line":"        path \u003d \"/pm_event\""},{"line_number":69,"context_line":"        return self.tacker_client.do_request("},{"line_number":70,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"7d604a03_3b202e31","line":67,"in_reply_to":"dbcf7f0f_64c20eb3","updated":"2023-05-09 06:45:30.000000000","message":"ditto","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        return self.tacker_client.do_request("},{"line_number":70,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _delete_pm_job(self, pm_job_id):"},{"line_number":73,"context_line":"        path \u003d f\"/vnfpm/v2/pm_jobs/{pm_job_id}\""},{"line_number":74,"context_line":"        return self.tacker_client.do_request("},{"line_number":75,"context_line":"            path, \"DELETE\", version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"efb9fa6a_f11a7a25","line":72,"updated":"2023-04-28 04:17:18.000000000","message":"It looks the same as below. Why are they defined separately?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_kubernetes_v2/base_v2.py#176","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        return self.tacker_client.do_request("},{"line_number":70,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _delete_pm_job(self, pm_job_id):"},{"line_number":73,"context_line":"        path \u003d f\"/vnfpm/v2/pm_jobs/{pm_job_id}\""},{"line_number":74,"context_line":"        return self.tacker_client.do_request("},{"line_number":75,"context_line":"            path, \"DELETE\", version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"042c1687_4bcedc10","line":72,"in_reply_to":"a0d371f6_f063c9e4","updated":"2023-05-09 06:45:30.000000000","message":"ditto","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        return self.tacker_client.do_request("},{"line_number":70,"context_line":"            path, \"POST\", body\u003dreq_body, version\u003d\"2.1.0\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _delete_pm_job(self, pm_job_id):"},{"line_number":73,"context_line":"        path \u003d f\"/vnfpm/v2/pm_jobs/{pm_job_id}\""},{"line_number":74,"context_line":"        return self.tacker_client.do_request("},{"line_number":75,"context_line":"            path, \"DELETE\", version\u003d\"2.1.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"a0d371f6_f063c9e4","line":72,"in_reply_to":"efb9fa6a_f11a7a25","updated":"2023-05-09 04:03:53.000000000","message":"ditto.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"}],"tacker/tests/functional/sol_kubernetes_oidc_auth/vnflcm_v2/test_kubernetes_oidc_auth.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":138,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":139,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"        # check usageState of VNF Package"},{"line_number":142,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":143,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"fe7b0310_15e049cc","line":141,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":138,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":139,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"        # check usageState of VNF Package"},{"line_number":142,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":143,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"a73c7ee7_9f017fc0","line":141,"in_reply_to":"e6ed2259_11b7928d","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":138,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":139,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"        # check usageState of VNF Package"},{"line_number":142,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":143,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"e6ed2259_11b7928d","line":141,"in_reply_to":"fe7b0310_15e049cc","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":97,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"2040d8fb_a482fe94","line":94,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":97,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"437d36df_da9145dd","line":94,"in_reply_to":"2040d8fb_a482fe94","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":97,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"bb4ae5c0_4a18d0cd","line":94,"in_reply_to":"2040d8fb_a482fe94","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":97,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"18ee1bf3_db485cb3","line":94,"in_reply_to":"bb4ae5c0_4a18d0cd","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":183,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # check usageState of VNF Package"},{"line_number":186,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":187,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":188,"context_line":"        k8s_vim_info \u003d copy.deepcopy(self.k8s_vim_info)"},{"line_number":189,"context_line":"        k8s_vim_info.accessInfo[\u0027client_id\u0027] \u003d \u0027badclient\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"dd960a29_dd43574d","line":186,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # check usageState of VNF Package"},{"line_number":186,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":187,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":188,"context_line":"        k8s_vim_info \u003d copy.deepcopy(self.k8s_vim_info)"},{"line_number":189,"context_line":"        k8s_vim_info.accessInfo[\u0027client_id\u0027] \u003d \u0027badclient\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"a4535177_ecc8afbf","line":186,"in_reply_to":"94727289_dfedf21f","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":183,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # check usageState of VNF Package"},{"line_number":186,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":187,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":188,"context_line":"        k8s_vim_info \u003d copy.deepcopy(self.k8s_vim_info)"},{"line_number":189,"context_line":"        k8s_vim_info.accessInfo[\u0027client_id\u0027] \u003d \u0027badclient\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"23616fca_aaa26a45","line":186,"in_reply_to":"dd960a29_dd43574d","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":183,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # check usageState of VNF Package"},{"line_number":186,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":187,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":188,"context_line":"        k8s_vim_info \u003d copy.deepcopy(self.k8s_vim_info)"},{"line_number":189,"context_line":"        k8s_vim_info.accessInfo[\u0027client_id\u0027] \u003d \u0027badclient\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"94727289_dfedf21f","line":186,"in_reply_to":"dd960a29_dd43574d","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"}],"tacker/tests/functional/sol_kubernetes_v2/base_v2.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":22,"context_line":"from tacker.tests import utils as base_utils"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":25,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class BaseVnfLcmKubernetesV2Test(base_v2.BaseTackerTestV2):"}],"source_content_type":"text/x-python","patch_set":5,"id":"680204f7_8c893c9a","line":24,"updated":"2023-04-28 04:17:18.000000000","message":"Is the definition of FAKE_SERVER_MANAGER necessary here?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":22,"context_line":"from tacker.tests import utils as base_utils"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":25,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class BaseVnfLcmKubernetesV2Test(base_v2.BaseTackerTestV2):"}],"source_content_type":"text/x-python","patch_set":5,"id":"ed76aecb_14d8049f","line":24,"in_reply_to":"680204f7_8c893c9a","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nRemoved definition of `FAKE_SERVER_MANAGER`.\nChild classes of base_v2 reference `FAKE_SERVER_MANAGER`. Therefore, defined `get_server_port()` and `set_server_callback` to `tacker/tests/functional/base_v2.py`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":22,"context_line":"from tacker.tests import utils as base_utils"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":25,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class BaseVnfLcmKubernetesV2Test(base_v2.BaseTackerTestV2):"}],"source_content_type":"text/x-python","patch_set":5,"id":"f2e0d515_6f32c980","line":24,"in_reply_to":"ed76aecb_14d8049f","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from tacker.tests import utils as base_utils"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":25,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class BaseVnfLcmKubernetesV2Test(base_v2.BaseTackerTestV2):"},{"line_number":28,"context_line":"    \"\"\"Base test case class for SOL v2 kubernetes functional tests.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"c9ceee5c_e2f06b06","line":25,"updated":"2023-04-28 04:17:18.000000000","message":"Is the definition of MOCK_NOTIFY_CALLBACK_URL necessary here?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from tacker.tests import utils as base_utils"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":25,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class BaseVnfLcmKubernetesV2Test(base_v2.BaseTackerTestV2):"},{"line_number":28,"context_line":"    \"\"\"Base test case class for SOL v2 kubernetes functional tests.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"a4be4a71_b90ceb74","line":25,"in_reply_to":"660ad08d_3d47388d","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from tacker.tests import utils as base_utils"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":25,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"class BaseVnfLcmKubernetesV2Test(base_v2.BaseTackerTestV2):"},{"line_number":28,"context_line":"    \"\"\"Base test case class for SOL v2 kubernetes functional tests.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"660ad08d_3d47388d","line":25,"in_reply_to":"c9ceee5c_e2f06b06","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nRemoved definition of `MOCK_NOTIFY_CALLBACK_URL`.\nChild classes of base_v2 reference `MOCK_NOTIFY_CALLBACK_URL`. Therefore, defined `get_notify_callback_url` to `tacker/tests/functional/base_v2.py`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"}],"tacker/tests/functional/sol_kubernetes_v2/test_change_vnfpkg.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":172,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":173,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"        # check usageState of VNF Package"},{"line_number":176,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":177,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"aa8cbafd_6b1a5c96","line":175,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":173,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"        # check usageState of VNF Package"},{"line_number":176,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":177,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"82c2ffac_53122929","line":175,"in_reply_to":"03fb8c3e_65539a59","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":172,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":173,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"        # check usageState of VNF Package"},{"line_number":176,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":177,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":178,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"03fb8c3e_65539a59","line":175,"in_reply_to":"aa8cbafd_6b1a5c96","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":297,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":298,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"        # check usageState of VNF Package"},{"line_number":301,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":302,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1bab9bbb_a322f8a9","line":300,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":297,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":298,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"        # check usageState of VNF Package"},{"line_number":301,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":302,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"}],"source_content_type":"text/x-python","patch_set":7,"id":"54d9c40c_6754ba9f","line":300,"in_reply_to":"1bab9bbb_a322f8a9","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":297,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":298,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"        # check usageState of VNF Package"},{"line_number":301,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":302,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"}],"source_content_type":"text/x-python","patch_set":7,"id":"9878bc4d_0643601e","line":300,"in_reply_to":"54d9c40c_6754ba9f","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":97,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"bf4f1621_03603b73","line":94,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":97,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"72861c67_6985607d","line":94,"in_reply_to":"bf4f1621_03603b73","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":97,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"c787786a_c01e605d","line":94,"in_reply_to":"bf4f1621_03603b73","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        # check usageState of VNF Package"},{"line_number":94,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":95,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":96,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":97,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"d76fcebf_b64a792b","line":94,"in_reply_to":"c787786a_c01e605d","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":215,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"        # check usageState of VNF Package"},{"line_number":218,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":219,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":220,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":221,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"4296aaec_6929c254","line":218,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":215,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"        # check usageState of VNF Package"},{"line_number":218,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":219,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":220,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":221,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"8b7e27cc_1ac12819","line":218,"in_reply_to":"4296aaec_6929c254","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":215,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"        # check usageState of VNF Package"},{"line_number":218,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":219,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":220,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":221,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"ed8d48da_8d7a23b5","line":218,"in_reply_to":"4296aaec_6929c254","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"        # check usageState of VNF Package"},{"line_number":218,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":219,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":220,"context_line":"        instantiate_req \u003d paramgen.change_vnfpkg_instantiate("},{"line_number":221,"context_line":"            self.auth_url, self.bearer_token)"}],"source_content_type":"text/x-python","patch_set":8,"id":"b27bfbf9_5c0a5f07","line":218,"in_reply_to":"ed8d48da_8d7a23b5","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"ab81f79aa7f7b512348fd5291b1f9370c7f6bd33","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"        test_instantiate_cnf_resources_path \u003d os.path.join("},{"line_number":32,"context_line":"            cur_dir, \"samples/test_instantiate_cnf_resources\")"},{"line_number":33,"context_line":"        cls.vnf_pkg_1, cls.vnfd_id_1 \u003d cls.create_vnf_package("},{"line_number":34,"context_line":"            test_instantiate_cnf_resources_path)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        test_change_vnf_pkg_with_deployment_path \u003d os.path.join("}],"source_content_type":"text/x-python","patch_set":9,"id":"bb4dc1eb_188cae36","line":33,"range":{"start_line":33,"start_character":12,"end_line":33,"end_character":21},"updated":"2023-05-29 08:19:26.000000000","message":"Why don\u0027t you revise such a name the meaning is not clear?","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"7ec08658e0ea822acb9cb037d47a43362249f0bd","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"        test_instantiate_cnf_resources_path \u003d os.path.join("},{"line_number":32,"context_line":"            cur_dir, \"samples/test_instantiate_cnf_resources\")"},{"line_number":33,"context_line":"        cls.vnf_pkg_1, cls.vnfd_id_1 \u003d cls.create_vnf_package("},{"line_number":34,"context_line":"            test_instantiate_cnf_resources_path)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        test_change_vnf_pkg_with_deployment_path \u003d os.path.join("}],"source_content_type":"text/x-python","patch_set":9,"id":"79f575f4_b0b0d9be","line":33,"range":{"start_line":33,"start_character":12,"end_line":33,"end_character":21},"in_reply_to":"bb4dc1eb_188cae36","updated":"2023-06-05 03:05:26.000000000","message":"I fixed it.\nIt has also been added to the CommitMessage as a correction item.","commit_id":"07c735424415bfa00851ed1c932ff4f17e59e751"}],"tacker/tests/functional/sol_kubernetes_v2/test_helm.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":272,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":273,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"    def _put_fail_file(self, operation):"},{"line_number":276,"context_line":"        with open(f\u0027/tmp/{operation}\u0027, \u0027w\u0027):"},{"line_number":277,"context_line":"            pass"},{"line_number":278,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1e173634_65fadec7","line":275,"updated":"2023-04-28 04:17:18.000000000","message":"This method is also used in some other tests.\nHow about moving to the following?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/base_v2.py","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":272,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":273,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"    def _put_fail_file(self, operation):"},{"line_number":276,"context_line":"        with open(f\u0027/tmp/{operation}\u0027, \u0027w\u0027):"},{"line_number":277,"context_line":"            pass"},{"line_number":278,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"8952de77_3c2c678e","line":275,"in_reply_to":"1e173634_65fadec7","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nDefined `put_fail_file` to `tacker/tests/functional/base_v2.py`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":272,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":273,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"    def _put_fail_file(self, operation):"},{"line_number":276,"context_line":"        with open(f\u0027/tmp/{operation}\u0027, \u0027w\u0027):"},{"line_number":277,"context_line":"            pass"},{"line_number":278,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"32ef6b62_6ecec810","line":275,"in_reply_to":"8952de77_3c2c678e","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":276,"context_line":"        with open(f\u0027/tmp/{operation}\u0027, \u0027w\u0027):"},{"line_number":277,"context_line":"            pass"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"    def _rm_fail_file(self, operation):"},{"line_number":280,"context_line":"        os.remove(f\u0027/tmp/{operation}\u0027)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"    def test_instantiate_rollback(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"b76c0546_260ec5c9","line":279,"updated":"2023-04-28 04:17:18.000000000","message":"ditto","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":276,"context_line":"        with open(f\u0027/tmp/{operation}\u0027, \u0027w\u0027):"},{"line_number":277,"context_line":"            pass"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"    def _rm_fail_file(self, operation):"},{"line_number":280,"context_line":"        os.remove(f\u0027/tmp/{operation}\u0027)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"    def test_instantiate_rollback(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"f393d451_99d1b960","line":279,"in_reply_to":"b76c0546_260ec5c9","updated":"2023-05-09 04:03:53.000000000","message":"ditto.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":276,"context_line":"        with open(f\u0027/tmp/{operation}\u0027, \u0027w\u0027):"},{"line_number":277,"context_line":"            pass"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"    def _rm_fail_file(self, operation):"},{"line_number":280,"context_line":"        os.remove(f\u0027/tmp/{operation}\u0027)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"    def test_instantiate_rollback(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"073bac65_f07d6784","line":279,"in_reply_to":"f393d451_99d1b960","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":226,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":227,"context_line":"        time.sleep(3)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        # check usageState of VNF Package"},{"line_number":230,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":231,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":232,"context_line":"        # check usageState of VNF Package"}],"source_content_type":"text/x-python","patch_set":7,"id":"6edc26ff_d9a6e84b","line":229,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":227,"context_line":"        time.sleep(3)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        # check usageState of VNF Package"},{"line_number":230,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":231,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":232,"context_line":"        # check usageState of VNF Package"}],"source_content_type":"text/x-python","patch_set":7,"id":"2329ac40_38843e15","line":229,"in_reply_to":"18d31f6a_c0d27d64","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":226,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":227,"context_line":"        time.sleep(3)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"        # check usageState of VNF Package"},{"line_number":230,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":231,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":232,"context_line":"        # check usageState of VNF Package"}],"source_content_type":"text/x-python","patch_set":7,"id":"18d31f6a_c0d27d64","line":229,"in_reply_to":"6edc26ff_d9a6e84b","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":229,"context_line":"        # check usageState of VNF Package"},{"line_number":230,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":231,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":232,"context_line":"        # check usageState of VNF Package"},{"line_number":233,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":234,"context_line":"        self.assertEqual(\u0027IN_USE\u0027, usage_state)"},{"line_number":235,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"f7df7a11_d46cde62","line":232,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        # check usageState of VNF Package"},{"line_number":230,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":231,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":232,"context_line":"        # check usageState of VNF Package"},{"line_number":233,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":234,"context_line":"        self.assertEqual(\u0027IN_USE\u0027, usage_state)"},{"line_number":235,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"3c89061f_68d0ebd6","line":232,"in_reply_to":"732696f7_c6d64745","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":229,"context_line":"        # check usageState of VNF Package"},{"line_number":230,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":231,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":232,"context_line":"        # check usageState of VNF Package"},{"line_number":233,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_2).get(\u0027usageState\u0027)"},{"line_number":234,"context_line":"        self.assertEqual(\u0027IN_USE\u0027, usage_state)"},{"line_number":235,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"732696f7_c6d64745","line":232,"in_reply_to":"f7df7a11_d46cde62","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":268,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":269,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        # check usageState of VNF Package"},{"line_number":272,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":273,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"ebc84a6b_b6a21aad","line":271,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use base_v2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":269,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        # check usageState of VNF Package"},{"line_number":272,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":273,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"9c412492_77092c17","line":271,"in_reply_to":"53687cf5_53e86e53","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":268,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":269,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        # check usageState of VNF Package"},{"line_number":272,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":273,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"53687cf5_53e86e53","line":271,"in_reply_to":"ebc84a6b_b6a21aad","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":104,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        # check usageState of VNF Package"},{"line_number":107,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":108,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":109,"context_line":"        if not use_register_vim:"},{"line_number":110,"context_line":"            instantiate_req \u003d paramgen.helm_instantiate("}],"source_content_type":"text/x-python","patch_set":8,"id":"276366ad_fe159a3a","line":107,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":104,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        # check usageState of VNF Package"},{"line_number":107,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":108,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":109,"context_line":"        if not use_register_vim:"},{"line_number":110,"context_line":"            instantiate_req \u003d paramgen.helm_instantiate("}],"source_content_type":"text/x-python","patch_set":8,"id":"f1969b2e_846cc112","line":107,"in_reply_to":"276366ad_fe159a3a","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":104,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        # check usageState of VNF Package"},{"line_number":107,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":108,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":109,"context_line":"        if not use_register_vim:"},{"line_number":110,"context_line":"            instantiate_req \u003d paramgen.helm_instantiate("}],"source_content_type":"text/x-python","patch_set":8,"id":"cbfc8b09_5498d5b3","line":107,"in_reply_to":"f1969b2e_846cc112","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":449,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        # check usageState of VNF Package"},{"line_number":452,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":453,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":454,"context_line":"        instantiate_req \u003d paramgen.helm_instantiate("},{"line_number":455,"context_line":"            self.auth_url, self.bearer_token, self.ssl_ca_cert)"}],"source_content_type":"text/x-python","patch_set":8,"id":"11f867f6_3eb674c0","line":452,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":449,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        # check usageState of VNF Package"},{"line_number":452,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":453,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":454,"context_line":"        instantiate_req \u003d paramgen.helm_instantiate("},{"line_number":455,"context_line":"            self.auth_url, self.bearer_token, self.ssl_ca_cert)"}],"source_content_type":"text/x-python","patch_set":8,"id":"a55d3914_2000ed89","line":452,"in_reply_to":"11f867f6_3eb674c0","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":449,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":450,"context_line":""},{"line_number":451,"context_line":"        # check usageState of VNF Package"},{"line_number":452,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":453,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":454,"context_line":"        instantiate_req \u003d paramgen.helm_instantiate("},{"line_number":455,"context_line":"            self.auth_url, self.bearer_token, self.ssl_ca_cert)"}],"source_content_type":"text/x-python","patch_set":8,"id":"83af6dab_09adb745","line":452,"in_reply_to":"a55d3914_2000ed89","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"}],"tacker/tests/functional/sol_kubernetes_v2/test_threshold_prometheus_plugin.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from tacker.tests.functional.sol_kubernetes_v2 import base_v2"},{"line_number":22,"context_line":"from tacker.tests.functional.sol_kubernetes_v2 import paramgen"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d FakeServerManager()"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":5,"id":"66e61383_555ca929","line":24,"updated":"2023-04-28 04:17:18.000000000","message":"Is the definition of FAKE_SERVER_MANAGER necessary here?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from tacker.tests.functional.sol_kubernetes_v2 import base_v2"},{"line_number":22,"context_line":"from tacker.tests.functional.sol_kubernetes_v2 import paramgen"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d FakeServerManager()"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":5,"id":"6a1f6ccb_b1ae33b2","line":24,"in_reply_to":"1557d2c8_1e453d6b","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from tacker.tests.functional.sol_kubernetes_v2 import base_v2"},{"line_number":22,"context_line":"from tacker.tests.functional.sol_kubernetes_v2 import paramgen"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"FAKE_SERVER_MANAGER \u003d FakeServerManager()"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":5,"id":"1557d2c8_1e453d6b","line":24,"in_reply_to":"66e61383_555ca929","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nRemoved definition of `FAKE_SERVER_MANAGER`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":true,"context_lines":[{"line_number":571,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":572,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"    def test_pm_threshold_with_all_attibutes(self):"},{"line_number":575,"context_line":"        \"\"\"Test PM Threshold operations with all attributes set"},{"line_number":576,"context_line":""},{"line_number":577,"context_line":"        * About attributes:"}],"source_content_type":"text/x-python","patch_set":13,"id":"1bf5d0b1_38cceb7e","line":574,"range":{"start_line":574,"start_character":35,"end_line":574,"end_character":44},"updated":"2023-06-09 05:10:13.000000000","message":"I found other typo,, so could you fix it in this patch?\ntypo: attibutes-\u003eattributes","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":true,"context_lines":[{"line_number":571,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":572,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"    def test_pm_threshold_with_all_attibutes(self):"},{"line_number":575,"context_line":"        \"\"\"Test PM Threshold operations with all attributes set"},{"line_number":576,"context_line":""},{"line_number":577,"context_line":"        * About attributes:"}],"source_content_type":"text/x-python","patch_set":13,"id":"446e875f_7a5eaff2","line":574,"range":{"start_line":574,"start_character":35,"end_line":574,"end_character":44},"in_reply_to":"1bf5d0b1_38cceb7e","updated":"2023-06-12 05:57:13.000000000","message":"I fixed it.","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[{"line_number":571,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":572,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"    def test_pm_threshold_with_all_attibutes(self):"},{"line_number":575,"context_line":"        \"\"\"Test PM Threshold operations with all attributes set"},{"line_number":576,"context_line":""},{"line_number":577,"context_line":"        * About attributes:"}],"source_content_type":"text/x-python","patch_set":13,"id":"4d227472_58ace06c","line":574,"range":{"start_line":574,"start_character":35,"end_line":574,"end_character":44},"in_reply_to":"446e875f_7a5eaff2","updated":"2023-06-14 17:26:12.000000000","message":"Thanks!\nDone","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"}],"tacker/tests/functional/sol_kubernetes_v2/test_vnflcm_basic.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":233,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":234,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"        # check usageState of VNF Package"},{"line_number":237,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":238,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":239,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"52cf121c_3fcaad4a","line":236,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":233,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":234,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"        # check usageState of VNF Package"},{"line_number":237,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":238,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":239,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"acb2dcb1_7b7c7847","line":236,"in_reply_to":"52cf121c_3fcaad4a","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":233,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":234,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"        # check usageState of VNF Package"},{"line_number":237,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":238,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":239,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"a180f970_bc697a9d","line":236,"in_reply_to":"acb2dcb1_7b7c7847","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":331,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":332,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        # check usageState of VNF Package"},{"line_number":335,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":336,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":337,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"12700a71_ea4be142","line":334,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":331,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":332,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        # check usageState of VNF Package"},{"line_number":335,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":336,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":337,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"44a6e32a_8479ca94","line":334,"in_reply_to":"12700a71_ea4be142","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":331,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":332,"context_line":"        self.assertEqual(404, resp.status_code)"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        # check usageState of VNF Package"},{"line_number":335,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_1).get(\u0027usageState\u0027)"},{"line_number":336,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":337,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"771d1d99_a063280a","line":334,"in_reply_to":"44a6e32a_8479ca94","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":98,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        # check usageState of VNF Package"},{"line_number":101,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":102,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":103,"context_line":"        instantiate_req \u003d paramgen.max_sample_instantiate("},{"line_number":104,"context_line":"            self.auth_url, self.bearer_token, ssl_ca_cert\u003dself.ssl_ca_cert)"}],"source_content_type":"text/x-python","patch_set":8,"id":"625186ee_a7476b10","line":101,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":98,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        # check usageState of VNF Package"},{"line_number":101,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":102,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":103,"context_line":"        instantiate_req \u003d paramgen.max_sample_instantiate("},{"line_number":104,"context_line":"            self.auth_url, self.bearer_token, ssl_ca_cert\u003dself.ssl_ca_cert)"}],"source_content_type":"text/x-python","patch_set":8,"id":"c87cb4ce_67721927","line":101,"in_reply_to":"4f2dee36_f8ac6acf","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":98,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        # check usageState of VNF Package"},{"line_number":101,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":102,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":103,"context_line":"        instantiate_req \u003d paramgen.max_sample_instantiate("},{"line_number":104,"context_line":"            self.auth_url, self.bearer_token, ssl_ca_cert\u003dself.ssl_ca_cert)"}],"source_content_type":"text/x-python","patch_set":8,"id":"4f2dee36_f8ac6acf","line":101,"in_reply_to":"625186ee_a7476b10","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":281,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        # check usageState of VNF Package"},{"line_number":284,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":285,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":286,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":287,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"cbaa0c67_c6f95f5f","line":284,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":281,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        # check usageState of VNF Package"},{"line_number":284,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":285,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":286,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":287,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"e1407c8a_8efa9b1b","line":284,"in_reply_to":"6caaca73_32f17d7d","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":281,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        # check usageState of VNF Package"},{"line_number":284,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":285,"context_line":"        # 2. Instantiate a VNF instance"},{"line_number":286,"context_line":"        vim_id \u003d self.get_k8s_vim_id()"},{"line_number":287,"context_line":"        instantiate_req \u003d paramgen.min_sample_instantiate(vim_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"6caaca73_32f17d7d","line":284,"in_reply_to":"cbaa0c67_c6f95f5f","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"}],"tacker/tests/functional/sol_v2/test_individual_vnfc_mgmt.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"4459602396dd6dd4bb6310982530e9d9305acd74","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from tacker.tests.functional.sol_v2_common import test_vnflcm_basic_common"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def create_coordinate_response(req_header, req_body):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    resp_body \u003d {"},{"line_number":26,"context_line":"        \u0027id\u0027: \u0027aeca5328-085c-4cd6-a6f4-c010e9082528\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"6af9a6c4_8732bd62","line":23,"updated":"2023-04-28 06:26:35.000000000","message":"Is this used outside of IndividualVnfcMgmtTest?\nIf it is not used, I feel that it should be defined as an internal method of IndividualVnfcMgmtTest.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from tacker.tests.functional.sol_v2_common import test_vnflcm_basic_common"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def create_coordinate_response(req_header, req_body):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    resp_body \u003d {"},{"line_number":26,"context_line":"        \u0027id\u0027: \u0027aeca5328-085c-4cd6-a6f4-c010e9082528\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9037f95e_e773df51","line":23,"in_reply_to":"3cb8d43f_acf0d18d","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from tacker.tests.functional.sol_v2_common import test_vnflcm_basic_common"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def create_coordinate_response(req_header, req_body):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    resp_body \u003d {"},{"line_number":26,"context_line":"        \u0027id\u0027: \u0027aeca5328-085c-4cd6-a6f4-c010e9082528\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3cb8d43f_acf0d18d","line":23,"in_reply_to":"6af9a6c4_8732bd62","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":369,"context_line":""},{"line_number":370,"context_line":"        # Delete VNF instance"},{"line_number":371,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":372,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    def test_rollback_operations(self):"},{"line_number":375,"context_line":"        \"\"\"Test rollback operations using StandardUserData"}],"source_content_type":"text/x-python","patch_set":7,"id":"2a3897ba_afcac0e0","line":372,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":369,"context_line":""},{"line_number":370,"context_line":"        # Delete VNF instance"},{"line_number":371,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":372,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    def test_rollback_operations(self):"},{"line_number":375,"context_line":"        \"\"\"Test rollback operations using StandardUserData"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a4be799_179506f7","line":372,"in_reply_to":"2a3897ba_afcac0e0","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":369,"context_line":""},{"line_number":370,"context_line":"        # Delete VNF instance"},{"line_number":371,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":372,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    def test_rollback_operations(self):"},{"line_number":375,"context_line":"        \"\"\"Test rollback operations using StandardUserData"}],"source_content_type":"text/x-python","patch_set":7,"id":"e79809da_02b765fe","line":372,"in_reply_to":"5a4be799_179506f7","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":533,"context_line":""},{"line_number":534,"context_line":"        # Delete VNF instance"},{"line_number":535,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":536,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":537,"context_line":""},{"line_number":538,"context_line":"    def test_heal_vnfc_after_image_change(self):"},{"line_number":539,"context_line":"        \"\"\"Test heal operation after image change using StandardUserData"}],"source_content_type":"text/x-python","patch_set":7,"id":"f2cc043b_a0ce789a","line":536,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":533,"context_line":""},{"line_number":534,"context_line":"        # Delete VNF instance"},{"line_number":535,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":536,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":537,"context_line":""},{"line_number":538,"context_line":"    def test_heal_vnfc_after_image_change(self):"},{"line_number":539,"context_line":"        \"\"\"Test heal operation after image change using StandardUserData"}],"source_content_type":"text/x-python","patch_set":7,"id":"186a1fff_e47361bb","line":536,"in_reply_to":"b06c176a_70a72626","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":533,"context_line":""},{"line_number":534,"context_line":"        # Delete VNF instance"},{"line_number":535,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":536,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":537,"context_line":""},{"line_number":538,"context_line":"    def test_heal_vnfc_after_image_change(self):"},{"line_number":539,"context_line":"        \"\"\"Test heal operation after image change using StandardUserData"}],"source_content_type":"text/x-python","patch_set":7,"id":"b06c176a_70a72626","line":536,"in_reply_to":"f2cc043b_a0ce789a","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":642,"context_line":""},{"line_number":643,"context_line":"        # Delete VNF instance"},{"line_number":644,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":645,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def test_change_vnfpkg_nw(self):"},{"line_number":648,"context_line":"        \"\"\"Test change_vnfpkg with additional functions"}],"source_content_type":"text/x-python","patch_set":7,"id":"1b36db72_97834eb5","line":645,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":642,"context_line":""},{"line_number":643,"context_line":"        # Delete VNF instance"},{"line_number":644,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":645,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def test_change_vnfpkg_nw(self):"},{"line_number":648,"context_line":"        \"\"\"Test change_vnfpkg with additional functions"}],"source_content_type":"text/x-python","patch_set":7,"id":"f60241f3_e61a02f4","line":645,"in_reply_to":"0f212a56_e55e8b9b","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":642,"context_line":""},{"line_number":643,"context_line":"        # Delete VNF instance"},{"line_number":644,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":645,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def test_change_vnfpkg_nw(self):"},{"line_number":648,"context_line":"        \"\"\"Test change_vnfpkg with additional functions"}],"source_content_type":"text/x-python","patch_set":7,"id":"0f212a56_e55e8b9b","line":645,"in_reply_to":"1b36db72_97834eb5","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":755,"context_line":""},{"line_number":756,"context_line":"        # Delete VNF instance"},{"line_number":757,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":758,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":759,"context_line":""},{"line_number":760,"context_line":"    def test_change_vnfpkg_nw_rollback(self):"},{"line_number":761,"context_line":"        \"\"\"Test rollback of change_vnfpkg with additional functions"}],"source_content_type":"text/x-python","patch_set":7,"id":"bdc03b42_55111d6a","line":758,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":755,"context_line":""},{"line_number":756,"context_line":"        # Delete VNF instance"},{"line_number":757,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":758,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":759,"context_line":""},{"line_number":760,"context_line":"    def test_change_vnfpkg_nw_rollback(self):"},{"line_number":761,"context_line":"        \"\"\"Test rollback of change_vnfpkg with additional functions"}],"source_content_type":"text/x-python","patch_set":7,"id":"6ea5177f_1f5d384f","line":758,"in_reply_to":"1ac4c391_7b159cf0","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":755,"context_line":""},{"line_number":756,"context_line":"        # Delete VNF instance"},{"line_number":757,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":758,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":759,"context_line":""},{"line_number":760,"context_line":"    def test_change_vnfpkg_nw_rollback(self):"},{"line_number":761,"context_line":"        \"\"\"Test rollback of change_vnfpkg with additional functions"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ac4c391_7b159cf0","line":758,"in_reply_to":"bdc03b42_55111d6a","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        # Delete VNF instance"},{"line_number":867,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":868,"context_line":"        self.assertEqual(204, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":7,"id":"6263426e_da973468","line":868,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        # Delete VNF instance"},{"line_number":867,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":868,"context_line":"        self.assertEqual(204, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":7,"id":"a15c631a_6eb5de7e","line":868,"in_reply_to":"6263426e_da973468","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        # Delete VNF instance"},{"line_number":867,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":868,"context_line":"        self.assertEqual(204, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":7,"id":"ea5878ed_aaa803ae","line":868,"in_reply_to":"a15c631a_6eb5de7e","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"}],"tacker/tests/functional/sol_v2/test_server_notification.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"4459602396dd6dd4bb6310982530e9d9305acd74","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from tacker.tests.functional.sol_v2_common import paramgen"},{"line_number":23,"context_line":"from tacker.tests.functional.sol_v2_common import test_vnflcm_basic_common"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"test_count \u003d 0"},{"line_number":26,"context_line":"RETRY_LIMIT \u003d 10"},{"line_number":27,"context_line":"RETRY_TIMEOUT \u003d 3"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"88a8c750_e007d1c6","line":25,"updated":"2023-04-28 06:26:35.000000000","message":"It is better to change it to upper case.\ntest_count -\u003e TEST_COUNT","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from tacker.tests.functional.sol_v2_common import paramgen"},{"line_number":23,"context_line":"from tacker.tests.functional.sol_v2_common import test_vnflcm_basic_common"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"test_count \u003d 0"},{"line_number":26,"context_line":"RETRY_LIMIT \u003d 10"},{"line_number":27,"context_line":"RETRY_TIMEOUT \u003d 3"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"de292af8_9d6f9a28","line":25,"in_reply_to":"0399ff0b_470cd542","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from tacker.tests.functional.sol_v2_common import paramgen"},{"line_number":23,"context_line":"from tacker.tests.functional.sol_v2_common import test_vnflcm_basic_common"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"test_count \u003d 0"},{"line_number":26,"context_line":"RETRY_LIMIT \u003d 10"},{"line_number":27,"context_line":"RETRY_TIMEOUT \u003d 3"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"0399ff0b_470cd542","line":25,"in_reply_to":"88a8c750_e007d1c6","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"4459602396dd6dd4bb6310982530e9d9305acd74","unresolved":true,"context_lines":[{"line_number":27,"context_line":"RETRY_TIMEOUT \u003d 3"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"def make_alarm_id(header, body):"},{"line_number":31,"context_line":"    from tacker.tests.functional.sol_v2 import test_server_notification"},{"line_number":32,"context_line":"    id \u003d f\"alarm_id_{test_server_notification.test_count}\""},{"line_number":33,"context_line":"    return {\u0027alarm_id\u0027: id}"}],"source_content_type":"text/x-python","patch_set":5,"id":"221da1c7_bdf18f16","line":30,"updated":"2023-04-28 06:26:35.000000000","message":"Is this used outside of ServerNotificationTest?\nIf it is not used, I feel that it should be defined as an internal method of ServerNotificationTest.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":27,"context_line":"RETRY_TIMEOUT \u003d 3"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"def make_alarm_id(header, body):"},{"line_number":31,"context_line":"    from tacker.tests.functional.sol_v2 import test_server_notification"},{"line_number":32,"context_line":"    id \u003d f\"alarm_id_{test_server_notification.test_count}\""},{"line_number":33,"context_line":"    return {\u0027alarm_id\u0027: id}"}],"source_content_type":"text/x-python","patch_set":5,"id":"2dce35b1_1dd8c06d","line":30,"in_reply_to":"221da1c7_bdf18f16","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":27,"context_line":"RETRY_TIMEOUT \u003d 3"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"def make_alarm_id(header, body):"},{"line_number":31,"context_line":"    from tacker.tests.functional.sol_v2 import test_server_notification"},{"line_number":32,"context_line":"    id \u003d f\"alarm_id_{test_server_notification.test_count}\""},{"line_number":33,"context_line":"    return {\u0027alarm_id\u0027: id}"}],"source_content_type":"text/x-python","patch_set":5,"id":"82f934bf_68967b17","line":30,"in_reply_to":"2dce35b1_1dd8c06d","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        update_vnf_path \u003d os.path.join(cur_dir, \"../sol_v2_common/\""},{"line_number":65,"context_line":"                                                \"samples/update_vnf\")"},{"line_number":66,"context_line":"        # no image contained"},{"line_number":67,"context_line":"        cls.up_pkg, cls.up_vnfd_id \u003d cls.create_vnf_package("},{"line_number":68,"context_line":"            update_vnf_path)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # for server_notification test"}],"source_content_type":"text/x-python","patch_set":13,"id":"b689bcbb_ad60102b","line":67,"updated":"2023-06-09 05:10:13.000000000","message":"IMO, it is better to use `upd_` instead of `up_` as a shorthand for `update`.\nWhat do you think?","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        update_vnf_path \u003d os.path.join(cur_dir, \"../sol_v2_common/\""},{"line_number":65,"context_line":"                                                \"samples/update_vnf\")"},{"line_number":66,"context_line":"        # no image contained"},{"line_number":67,"context_line":"        cls.up_pkg, cls.up_vnfd_id \u003d cls.create_vnf_package("},{"line_number":68,"context_line":"            update_vnf_path)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # for server_notification test"}],"source_content_type":"text/x-python","patch_set":13,"id":"400390b9_17b74ccb","line":67,"in_reply_to":"68861cac_6217af76","updated":"2023-06-14 17:26:12.000000000","message":"Done","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        update_vnf_path \u003d os.path.join(cur_dir, \"../sol_v2_common/\""},{"line_number":65,"context_line":"                                                \"samples/update_vnf\")"},{"line_number":66,"context_line":"        # no image contained"},{"line_number":67,"context_line":"        cls.up_pkg, cls.up_vnfd_id \u003d cls.create_vnf_package("},{"line_number":68,"context_line":"            update_vnf_path)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # for server_notification test"}],"source_content_type":"text/x-python","patch_set":13,"id":"68861cac_6217af76","line":67,"in_reply_to":"b689bcbb_ad60102b","updated":"2023-06-12 05:57:13.000000000","message":"I agree with you. I fixed it.","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"}],"tacker/tests/functional/sol_v2/test_vnflcm_basic.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":312,"context_line":"        # 5. Update VNF"},{"line_number":313,"context_line":"        # check attribute value before update VNF"},{"line_number":314,"context_line":"        # check usageState of VNF Package 1"},{"line_number":315,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":316,"context_line":"        # check usageState of VNF Package 3"},{"line_number":317,"context_line":"        self.check_package_usage(self.vnf_pkg_3, \u0027NOT_IN_USE\u0027)"},{"line_number":318,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"3e5cc22e_4f1f4fed","line":315,"updated":"2023-05-10 06:50:39.000000000","message":"As with L.357, I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":312,"context_line":"        # 5. Update VNF"},{"line_number":313,"context_line":"        # check attribute value before update VNF"},{"line_number":314,"context_line":"        # check usageState of VNF Package 1"},{"line_number":315,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":316,"context_line":"        # check usageState of VNF Package 3"},{"line_number":317,"context_line":"        self.check_package_usage(self.vnf_pkg_3, \u0027NOT_IN_USE\u0027)"},{"line_number":318,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"e21550f8_1848d035","line":315,"in_reply_to":"3e5cc22e_4f1f4fed","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":312,"context_line":"        # 5. Update VNF"},{"line_number":313,"context_line":"        # check attribute value before update VNF"},{"line_number":314,"context_line":"        # check usageState of VNF Package 1"},{"line_number":315,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":316,"context_line":"        # check usageState of VNF Package 3"},{"line_number":317,"context_line":"        self.check_package_usage(self.vnf_pkg_3, \u0027NOT_IN_USE\u0027)"},{"line_number":318,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"bf9e7d2d_a9a07a91","line":315,"in_reply_to":"e21550f8_1848d035","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":532,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":533,"context_line":""},{"line_number":534,"context_line":"        # check usageState of VNF Package"},{"line_number":535,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":536,"context_line":"        # check instantiationState of VNF"},{"line_number":537,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":538,"context_line":"        self.assertEqual(200, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":8,"id":"8341a80b_732759ed","line":535,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":532,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":533,"context_line":""},{"line_number":534,"context_line":"        # check usageState of VNF Package"},{"line_number":535,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":536,"context_line":"        # check instantiationState of VNF"},{"line_number":537,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":538,"context_line":"        self.assertEqual(200, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":8,"id":"d47384f4_a4e0b426","line":535,"in_reply_to":"8341a80b_732759ed","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":532,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":533,"context_line":""},{"line_number":534,"context_line":"        # check usageState of VNF Package"},{"line_number":535,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":536,"context_line":"        # check instantiationState of VNF"},{"line_number":537,"context_line":"        resp, body \u003d self.show_vnf_instance(inst_id)"},{"line_number":538,"context_line":"        self.assertEqual(200, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":8,"id":"b1bedf9f_5b2db7b9","line":535,"in_reply_to":"d47384f4_a4e0b426","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":576,"context_line":"        # 5. Update VNF"},{"line_number":577,"context_line":"        # check attribute value before update VNF"},{"line_number":578,"context_line":"        # check usageState of VNF Package 1"},{"line_number":579,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":580,"context_line":"        # check usageState of VNF Package 3"},{"line_number":581,"context_line":"        self.check_package_usage(self.vnf_pkg_3, \u0027NOT_IN_USE\u0027)"},{"line_number":582,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"52408103_2fc84afb","line":579,"updated":"2023-05-10 06:50:39.000000000","message":"As with L.623, I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":576,"context_line":"        # 5. Update VNF"},{"line_number":577,"context_line":"        # check attribute value before update VNF"},{"line_number":578,"context_line":"        # check usageState of VNF Package 1"},{"line_number":579,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":580,"context_line":"        # check usageState of VNF Package 3"},{"line_number":581,"context_line":"        self.check_package_usage(self.vnf_pkg_3, \u0027NOT_IN_USE\u0027)"},{"line_number":582,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"623ece14_25f5ee2c","line":579,"in_reply_to":"52408103_2fc84afb","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":576,"context_line":"        # 5. Update VNF"},{"line_number":577,"context_line":"        # check attribute value before update VNF"},{"line_number":578,"context_line":"        # check usageState of VNF Package 1"},{"line_number":579,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":580,"context_line":"        # check usageState of VNF Package 3"},{"line_number":581,"context_line":"        self.check_package_usage(self.vnf_pkg_3, \u0027NOT_IN_USE\u0027)"},{"line_number":582,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"fddfe60b_30684f7c","line":579,"in_reply_to":"623ece14_25f5ee2c","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":924,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":925,"context_line":""},{"line_number":926,"context_line":"        # check usageState of VNF Package"},{"line_number":927,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":928,"context_line":"        # check instantiationState of VNF"},{"line_number":929,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":930,"context_line":"                         body[\u0027instantiationState\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"3a9fa071_3ff91d24","line":927,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":924,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":925,"context_line":""},{"line_number":926,"context_line":"        # check usageState of VNF Package"},{"line_number":927,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":928,"context_line":"        # check instantiationState of VNF"},{"line_number":929,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":930,"context_line":"                         body[\u0027instantiationState\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"b07d4676_57573f52","line":927,"in_reply_to":"3a9fa071_3ff91d24","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":924,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":925,"context_line":""},{"line_number":926,"context_line":"        # check usageState of VNF Package"},{"line_number":927,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":928,"context_line":"        # check instantiationState of VNF"},{"line_number":929,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":930,"context_line":"                         body[\u0027instantiationState\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"86a58ae5_b29dce6e","line":927,"in_reply_to":"b07d4676_57573f52","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":true,"context_lines":[{"line_number":1225,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":1226,"context_line":""},{"line_number":1227,"context_line":"        # check usageState of VNF Package"},{"line_number":1228,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":1229,"context_line":"        # check instantiationState of VNF"},{"line_number":1230,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":1231,"context_line":"                         body[\u0027instantiationState\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"5c81f7ab_c4c5b01c","line":1228,"updated":"2023-05-10 06:50:39.000000000","message":"I think it\u0027s better not to remove blank line.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"aca4508c331eae412cefefbbc67d969ae28009c0","unresolved":true,"context_lines":[{"line_number":1225,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":1226,"context_line":""},{"line_number":1227,"context_line":"        # check usageState of VNF Package"},{"line_number":1228,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":1229,"context_line":"        # check instantiationState of VNF"},{"line_number":1230,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":1231,"context_line":"                         body[\u0027instantiationState\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"b6fa4b49_36b7b58c","line":1228,"in_reply_to":"5c81f7ab_c4c5b01c","updated":"2023-05-10 23:54:49.000000000","message":"I fixed it in PS9.","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a66163b9ea72e7a4efb22d00752fe94df93aa670","unresolved":false,"context_lines":[{"line_number":1225,"context_line":"        inst_id \u003d body[\u0027id\u0027]"},{"line_number":1226,"context_line":""},{"line_number":1227,"context_line":"        # check usageState of VNF Package"},{"line_number":1228,"context_line":"        self.check_package_usage(self.vnf_pkg_1, \u0027IN_USE\u0027)"},{"line_number":1229,"context_line":"        # check instantiationState of VNF"},{"line_number":1230,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":1231,"context_line":"                         body[\u0027instantiationState\u0027])"}],"source_content_type":"text/x-python","patch_set":8,"id":"2405f201_f3a79c25","line":1228,"in_reply_to":"b6fa4b49_36b7b58c","updated":"2023-05-11 02:43:37.000000000","message":"Done","commit_id":"8ae6283ea459983949a158132e49d404cae37a7d"}],"tacker/tests/functional/sol_v2/test_vnflcm_error_handling.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":438,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":439,"context_line":"        self.check_resp_headers_in_delete(resp)"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"        # check usageState of VNF Package"},{"line_number":442,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_3)[\u0027usageState\u0027]"},{"line_number":443,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":444,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"5e077c91_28ccc9a5","line":441,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":438,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":439,"context_line":"        self.check_resp_headers_in_delete(resp)"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"        # check usageState of VNF Package"},{"line_number":442,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_3)[\u0027usageState\u0027]"},{"line_number":443,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":444,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"ff4a59f3_fe8a15ac","line":441,"in_reply_to":"4c4f4c81_2c06ecc1","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":438,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":439,"context_line":"        self.check_resp_headers_in_delete(resp)"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"        # check usageState of VNF Package"},{"line_number":442,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_3)[\u0027usageState\u0027]"},{"line_number":443,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":444,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"4c4f4c81_2c06ecc1","line":441,"in_reply_to":"5e077c91_28ccc9a5","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":661,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":662,"context_line":"        self.check_resp_headers_in_delete(resp)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"        # check usageState of VNF Package"},{"line_number":665,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_3)[\u0027usageState\u0027]"},{"line_number":666,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":667,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"38b9c840_9919793c","line":664,"updated":"2023-05-09 06:45:30.000000000","message":"Why not use basev2.check_package_usage?","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":661,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":662,"context_line":"        self.check_resp_headers_in_delete(resp)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"        # check usageState of VNF Package"},{"line_number":665,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_3)[\u0027usageState\u0027]"},{"line_number":666,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":667,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"efc7d7e1_aee6b41e","line":664,"in_reply_to":"38b9c840_9919793c","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":661,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":662,"context_line":"        self.check_resp_headers_in_delete(resp)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"        # check usageState of VNF Package"},{"line_number":665,"context_line":"        usage_state \u003d self.get_vnf_package(self.vnf_pkg_3)[\u0027usageState\u0027]"},{"line_number":666,"context_line":"        self.assertEqual(\u0027NOT_IN_USE\u0027, usage_state)"},{"line_number":667,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"252858d0_5fc194b2","line":664,"in_reply_to":"efc7d7e1_aee6b41e","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"}],"tacker/tests/functional/sol_v2_common/base_v2.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from tacker.sol_refactored import objects"},{"line_number":23,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"d3886fce_1e69e6e2","line":25,"updated":"2023-04-28 04:17:18.000000000","message":"Is the definition of FAKE_SERVER_MANAGER necessary here?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from tacker.sol_refactored import objects"},{"line_number":23,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fb0109f4_b33b0c03","line":25,"in_reply_to":"a638daa2_4c7b918b","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":22,"context_line":"from tacker.sol_refactored import objects"},{"line_number":23,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"a638daa2_4c7b918b","line":25,"in_reply_to":"d3886fce_1e69e6e2","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nRemoved definition of `FAKE_SERVER_MANAGER`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":23,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"df616a1f_a4969461","line":26,"updated":"2023-04-28 04:17:18.000000000","message":"Is the definition of MOCK_NOTIFY_CALLBACK_URL necessary here?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":23,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fd92c14b_318fe395","line":26,"in_reply_to":"df616a1f_a4969461","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nRemoved definition of `MOCK_NOTIFY_CALLBACK_URL`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from tacker.tests.functional import base_v2"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bb0bcbdc_bd77bfa2","line":26,"in_reply_to":"fd92c14b_318fe395","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"6fde08b6d62b40d11523458e4e4b6b95b6a106e2","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"class BaseSolV2Test(base_v2.BaseTackerTestV2):"}],"source_content_type":"text/x-python","patch_set":5,"id":"1d953734_661f9a1c","line":27,"updated":"2023-04-28 04:17:18.000000000","message":"Is the definition of LOG necessary here?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"class BaseSolV2Test(base_v2.BaseTackerTestV2):"}],"source_content_type":"text/x-python","patch_set":5,"id":"27ca4ddc_a13cd883","line":27,"in_reply_to":"1d953734_661f9a1c","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nRemoved definition of `LOG`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"FAKE_SERVER_MANAGER \u003d base_v2.FAKE_SERVER_MANAGER"},{"line_number":26,"context_line":"MOCK_NOTIFY_CALLBACK_URL \u003d base_v2.MOCK_NOTIFY_CALLBACK_URL"},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"class BaseSolV2Test(base_v2.BaseTackerTestV2):"}],"source_content_type":"text/x-python","patch_set":5,"id":"8a2e177c_e09801fc","line":27,"in_reply_to":"27ca4ddc_a13cd883","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"}],"tacker/tests/functional/sol_v2_common/test_vnflcm_basic_common.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"dfa98e827c57ff393b4e3618b4524c7436233d72","unresolved":true,"context_lines":[{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"        lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":1611,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":1612,"context_line":""},{"line_number":1613,"context_line":"        # check deletion of Heat-stack"},{"line_number":1614,"context_line":"        stack_status, _ \u003d self.heat_client.get_status(stack_name)"},{"line_number":1615,"context_line":"        self.assertIsNone(stack_status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"8573df0e_c5e143e0","line":1612,"updated":"2023-04-28 05:40:24.000000000","message":"Is sleep processing unnecessary for the min pattern?\nIn the max pattern, sleep processing is performed.\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/sol_v2_common/test_vnflcm_basic_common.py#1150","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"        lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":1611,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":1612,"context_line":""},{"line_number":1613,"context_line":"        # check deletion of Heat-stack"},{"line_number":1614,"context_line":"        stack_status, _ \u003d self.heat_client.get_status(stack_name)"},{"line_number":1615,"context_line":"        self.assertIsNone(stack_status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"c6661506_e3d4a2e2","line":1612,"in_reply_to":"78845bfe_e414c13c","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"049c4833ef4697d2e461977c47361055855782f3","unresolved":true,"context_lines":[{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"        lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":1611,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":1612,"context_line":""},{"line_number":1613,"context_line":"        # check deletion of Heat-stack"},{"line_number":1614,"context_line":"        stack_status, _ \u003d self.heat_client.get_status(stack_name)"},{"line_number":1615,"context_line":"        self.assertIsNone(stack_status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"c069439b_5a8c7c78","line":1612,"in_reply_to":"8573df0e_c5e143e0","updated":"2023-05-09 04:21:56.000000000","message":"I fixed it in PS6.\n\nChanged `update_vnf_instance` to `exec_lcmocc`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"        lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":1611,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":1612,"context_line":""},{"line_number":1613,"context_line":"        # check deletion of Heat-stack"},{"line_number":1614,"context_line":"        stack_status, _ \u003d self.heat_client.get_status(stack_name)"},{"line_number":1615,"context_line":"        self.assertIsNone(stack_status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"78845bfe_e414c13c","line":1612,"in_reply_to":"b01f164b_3b579ecd","updated":"2023-05-10 02:53:11.000000000","message":"I agree with you.\nI fixed it in PS8.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"        lcmocc_id \u003d os.path.basename(resp.headers[\u0027Location\u0027])"},{"line_number":1611,"context_line":"        self.wait_lcmocc_complete(lcmocc_id)"},{"line_number":1612,"context_line":""},{"line_number":1613,"context_line":"        # check deletion of Heat-stack"},{"line_number":1614,"context_line":"        stack_status, _ \u003d self.heat_client.get_status(stack_name)"},{"line_number":1615,"context_line":"        self.assertIsNone(stack_status)"}],"source_content_type":"text/x-python","patch_set":5,"id":"b01f164b_3b579ecd","line":1612,"in_reply_to":"c069439b_5a8c7c78","updated":"2023-05-09 06:45:30.000000000","message":"`lcmocc` seems to suggest VnfLcmOpOcc, so how about changing it to `exec_lcm_operation`?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":true,"context_lines":[{"line_number":217,"context_line":"            vnfd_path \u003d \"contents/Definitions/v2_sample1_df_simple.yaml\""},{"line_number":218,"context_line":"            max_zip_path, max_vnfd_id \u003d self.create_vnf_package("},{"line_number":219,"context_line":"                basic_lcms_max_path, image_path\u003dimage_path, nfvo\u003dTrue)"},{"line_number":220,"context_line":"            up_zip_path, up_vnfd_id \u003d self.create_vnf_package("},{"line_number":221,"context_line":"                update_vnf_path, nfvo\u003dTrue)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"            self._register_vnf_package_mock_response(max_vnfd_id,"}],"source_content_type":"text/x-python","patch_set":13,"id":"bfe44072_c7f85344","line":220,"updated":"2023-06-09 05:10:13.000000000","message":"IMO, it is better to use `upd_` instead of `up_` as a shorthand for `update`.\nWhat do you think?","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[{"line_number":217,"context_line":"            vnfd_path \u003d \"contents/Definitions/v2_sample1_df_simple.yaml\""},{"line_number":218,"context_line":"            max_zip_path, max_vnfd_id \u003d self.create_vnf_package("},{"line_number":219,"context_line":"                basic_lcms_max_path, image_path\u003dimage_path, nfvo\u003dTrue)"},{"line_number":220,"context_line":"            up_zip_path, up_vnfd_id \u003d self.create_vnf_package("},{"line_number":221,"context_line":"                update_vnf_path, nfvo\u003dTrue)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"            self._register_vnf_package_mock_response(max_vnfd_id,"}],"source_content_type":"text/x-python","patch_set":13,"id":"e0271344_45d944e8","line":220,"in_reply_to":"4cff9c84_8f32bdd6","updated":"2023-06-14 17:26:12.000000000","message":"Done","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":true,"context_lines":[{"line_number":217,"context_line":"            vnfd_path \u003d \"contents/Definitions/v2_sample1_df_simple.yaml\""},{"line_number":218,"context_line":"            max_zip_path, max_vnfd_id \u003d self.create_vnf_package("},{"line_number":219,"context_line":"                basic_lcms_max_path, image_path\u003dimage_path, nfvo\u003dTrue)"},{"line_number":220,"context_line":"            up_zip_path, up_vnfd_id \u003d self.create_vnf_package("},{"line_number":221,"context_line":"                update_vnf_path, nfvo\u003dTrue)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"            self._register_vnf_package_mock_response(max_vnfd_id,"}],"source_content_type":"text/x-python","patch_set":13,"id":"4cff9c84_8f32bdd6","line":220,"in_reply_to":"bfe44072_c7f85344","updated":"2023-06-12 05:57:13.000000000","message":"I agree with you. I fixed it.","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"}],"tacker/tests/functional/sol_v2_individual_vnfc_mgmt/test_individual_vnfc_mgmt_basic.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"dfa98e827c57ff393b4e3618b4524c7436233d72","unresolved":true,"context_lines":[{"line_number":293,"context_line":""},{"line_number":294,"context_line":"        return body"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"    def _check_package_usage(self, package_id, state\u003d\u0027NOT_IN_USE\u0027):"},{"line_number":297,"context_line":"        usage_state \u003d self.get_vnf_package(package_id)[\u0027usageState\u0027]"},{"line_number":298,"context_line":"        self.assertEqual(state, usage_state)"},{"line_number":299,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"698d190a_afea506c","line":296,"updated":"2023-04-28 05:40:24.000000000","message":"This method is also used in some other tests.\nHow about moving to the following?\n* https://review.opendev.org/c/openstack/tacker/+/881181/5/tacker/tests/functional/base_v2.py","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":293,"context_line":""},{"line_number":294,"context_line":"        return body"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"    def _check_package_usage(self, package_id, state\u003d\u0027NOT_IN_USE\u0027):"},{"line_number":297,"context_line":"        usage_state \u003d self.get_vnf_package(package_id)[\u0027usageState\u0027]"},{"line_number":298,"context_line":"        self.assertEqual(state, usage_state)"},{"line_number":299,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"085fe863_935ebc25","line":296,"in_reply_to":"1e70eb7b_d98edcea","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":293,"context_line":""},{"line_number":294,"context_line":"        return body"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"    def _check_package_usage(self, package_id, state\u003d\u0027NOT_IN_USE\u0027):"},{"line_number":297,"context_line":"        usage_state \u003d self.get_vnf_package(package_id)[\u0027usageState\u0027]"},{"line_number":298,"context_line":"        self.assertEqual(state, usage_state)"},{"line_number":299,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1e70eb7b_d98edcea","line":296,"in_reply_to":"698d190a_afea506c","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nDefined `check_package_usage` to `tacker/tests/functional/base_v2.py`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"dfa98e827c57ff393b4e3618b4524c7436233d72","unresolved":true,"context_lines":[{"line_number":460,"context_line":"            self.assertNotIn(\u0027vimConnectionInfo\u0027, inst)"},{"line_number":461,"context_line":"            self.assertNotIn(\u0027instantiatedVnfInfo\u0027, inst)"},{"line_number":462,"context_line":"            self.assertNotIn(\u0027metadata\u0027, inst)"},{"line_number":463,"context_line":"        self.exec_delete_vnf_instance(tmp_body[\u0027id\u0027])"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"        # 4. Show VNF LCM operation occurrence"},{"line_number":466,"context_line":"        expected_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":5,"id":"66a1d4f9_65d4906e","line":463,"updated":"2023-04-28 05:40:24.000000000","message":"This is the VNF instance before `instantiate`, is a delete retry required?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":460,"context_line":"            self.assertNotIn(\u0027vimConnectionInfo\u0027, inst)"},{"line_number":461,"context_line":"            self.assertNotIn(\u0027instantiatedVnfInfo\u0027, inst)"},{"line_number":462,"context_line":"            self.assertNotIn(\u0027metadata\u0027, inst)"},{"line_number":463,"context_line":"        self.exec_delete_vnf_instance(tmp_body[\u0027id\u0027])"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"        # 4. Show VNF LCM operation occurrence"},{"line_number":466,"context_line":"        expected_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":5,"id":"62076dd0_546eef55","line":463,"in_reply_to":"2bd92f3a_5dfe06a0","updated":"2023-05-10 02:53:11.000000000","message":"I agree with you.\nI fixed it in PS8.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":460,"context_line":"            self.assertNotIn(\u0027vimConnectionInfo\u0027, inst)"},{"line_number":461,"context_line":"            self.assertNotIn(\u0027instantiatedVnfInfo\u0027, inst)"},{"line_number":462,"context_line":"            self.assertNotIn(\u0027metadata\u0027, inst)"},{"line_number":463,"context_line":"        self.exec_delete_vnf_instance(tmp_body[\u0027id\u0027])"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"        # 4. Show VNF LCM operation occurrence"},{"line_number":466,"context_line":"        expected_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":5,"id":"a0bf8165_f60a009d","line":463,"in_reply_to":"62076dd0_546eef55","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":460,"context_line":"            self.assertNotIn(\u0027vimConnectionInfo\u0027, inst)"},{"line_number":461,"context_line":"            self.assertNotIn(\u0027instantiatedVnfInfo\u0027, inst)"},{"line_number":462,"context_line":"            self.assertNotIn(\u0027metadata\u0027, inst)"},{"line_number":463,"context_line":"        self.exec_delete_vnf_instance(tmp_body[\u0027id\u0027])"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"        # 4. Show VNF LCM operation occurrence"},{"line_number":466,"context_line":"        expected_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":5,"id":"d0fc6864_30e26725","line":463,"in_reply_to":"66a1d4f9_65d4906e","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":460,"context_line":"            self.assertNotIn(\u0027vimConnectionInfo\u0027, inst)"},{"line_number":461,"context_line":"            self.assertNotIn(\u0027instantiatedVnfInfo\u0027, inst)"},{"line_number":462,"context_line":"            self.assertNotIn(\u0027metadata\u0027, inst)"},{"line_number":463,"context_line":"        self.exec_delete_vnf_instance(tmp_body[\u0027id\u0027])"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"        # 4. Show VNF LCM operation occurrence"},{"line_number":466,"context_line":"        expected_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":5,"id":"2bd92f3a_5dfe06a0","line":463,"in_reply_to":"d0fc6864_30e26725","updated":"2023-05-09 06:45:30.000000000","message":"Shouldn\u0027t the following be used?\n* self.delete_vnf_instance(tmp_body[\u0027id\u0027])\nhttps://review.opendev.org/c/openstack/tacker/+/881181/5..6/tacker/tests/functional/base_v2.py#202","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"dfa98e827c57ff393b4e3618b4524c7436233d72","unresolved":true,"context_lines":[{"line_number":1076,"context_line":"        self.assertEqual(200, resp.status_code)"},{"line_number":1077,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":1078,"context_line":"                         body[\u0027instantiationState\u0027])"},{"line_number":1079,"context_line":""},{"line_number":1080,"context_line":"        # 14. Update VNF again"},{"line_number":1081,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":1082,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a98d0bd_53e29746","line":1079,"updated":"2023-04-28 05:40:24.000000000","message":"Is sleep processing that waits for the completion of `terminate` unnecessary?","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"207cda2b3cbf3ec9212f22616789eb617c64f273","unresolved":true,"context_lines":[{"line_number":1076,"context_line":"        self.assertEqual(200, resp.status_code)"},{"line_number":1077,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":1078,"context_line":"                         body[\u0027instantiationState\u0027])"},{"line_number":1079,"context_line":""},{"line_number":1080,"context_line":"        # 14. Update VNF again"},{"line_number":1081,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":1082,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"}],"source_content_type":"text/x-python","patch_set":5,"id":"97ff313c_f1cfe42d","line":1079,"in_reply_to":"5a98d0bd_53e29746","updated":"2023-05-09 04:03:53.000000000","message":"I fixed it in PS6.\n\nChanged `update_vnf_instance` to `exec_lcmocc`.","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":false,"context_lines":[{"line_number":1076,"context_line":"        self.assertEqual(200, resp.status_code)"},{"line_number":1077,"context_line":"        self.assertEqual(fields.VnfInstanceState.NOT_INSTANTIATED,"},{"line_number":1078,"context_line":"                         body[\u0027instantiationState\u0027])"},{"line_number":1079,"context_line":""},{"line_number":1080,"context_line":"        # 14. Update VNF again"},{"line_number":1081,"context_line":"        update_req \u003d paramgen.update_vnf_min_with_parameter(self.vnfd_id_2)"},{"line_number":1082,"context_line":"        resp, body \u003d self.update_vnf_instance(inst_id, update_req)"}],"source_content_type":"text/x-python","patch_set":5,"id":"448f0f00_4f1d8338","line":1079,"in_reply_to":"97ff313c_f1cfe42d","updated":"2023-05-09 06:45:30.000000000","message":"Done","commit_id":"7bbb3d253b1d0b5ee17f62dc1ba791cb57eff314"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        # 30. Delete VNF"},{"line_number":867,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":868,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":869,"context_line":""},{"line_number":870,"context_line":"    def test_basic_lcms_min(self):"},{"line_number":871,"context_line":"        \"\"\"Test LCM operations with omitting except for required attributes"}],"source_content_type":"text/x-python","patch_set":7,"id":"a4104eb3_485c930d","line":868,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        # 30. Delete VNF"},{"line_number":867,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":868,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":869,"context_line":""},{"line_number":870,"context_line":"    def test_basic_lcms_min(self):"},{"line_number":871,"context_line":"        \"\"\"Test LCM operations with omitting except for required attributes"}],"source_content_type":"text/x-python","patch_set":7,"id":"b52969ad_d83f3cd1","line":868,"in_reply_to":"a4104eb3_485c930d","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        # 30. Delete VNF"},{"line_number":867,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":868,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":869,"context_line":""},{"line_number":870,"context_line":"    def test_basic_lcms_min(self):"},{"line_number":871,"context_line":"        \"\"\"Test LCM operations with omitting except for required attributes"}],"source_content_type":"text/x-python","patch_set":7,"id":"976688af_ef2f8bda","line":868,"in_reply_to":"b52969ad_d83f3cd1","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":1141,"context_line":""},{"line_number":1142,"context_line":"        # 19. Delete VNF"},{"line_number":1143,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1144,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1145,"context_line":""},{"line_number":1146,"context_line":"    def test_retry_rollback_scale_out(self):"},{"line_number":1147,"context_line":"        \"\"\"Test retry and rollback scale out operations"}],"source_content_type":"text/x-python","patch_set":7,"id":"1edd98ab_52f6dec8","line":1144,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":1141,"context_line":""},{"line_number":1142,"context_line":"        # 19. Delete VNF"},{"line_number":1143,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1144,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1145,"context_line":""},{"line_number":1146,"context_line":"    def test_retry_rollback_scale_out(self):"},{"line_number":1147,"context_line":"        \"\"\"Test retry and rollback scale out operations"}],"source_content_type":"text/x-python","patch_set":7,"id":"fd471f1e_045fe94b","line":1144,"in_reply_to":"1edd98ab_52f6dec8","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":1141,"context_line":""},{"line_number":1142,"context_line":"        # 19. Delete VNF"},{"line_number":1143,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1144,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1145,"context_line":""},{"line_number":1146,"context_line":"    def test_retry_rollback_scale_out(self):"},{"line_number":1147,"context_line":"        \"\"\"Test retry and rollback scale out operations"}],"source_content_type":"text/x-python","patch_set":7,"id":"ffd1111c_26945344","line":1144,"in_reply_to":"fd471f1e_045fe94b","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":1287,"context_line":""},{"line_number":1288,"context_line":"        # 10. Delete VNF"},{"line_number":1289,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1290,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1291,"context_line":""},{"line_number":1292,"context_line":"    def test_rollback_instantiate(self):"},{"line_number":1293,"context_line":"        \"\"\"Test rollback instantiate operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"c17ba893_e0f578eb","line":1290,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":1287,"context_line":""},{"line_number":1288,"context_line":"        # 10. Delete VNF"},{"line_number":1289,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1290,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1291,"context_line":""},{"line_number":1292,"context_line":"    def test_rollback_instantiate(self):"},{"line_number":1293,"context_line":"        \"\"\"Test rollback instantiate operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"23ce518e_ea53e66c","line":1290,"in_reply_to":"49bb7bb4_5675624e","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":1287,"context_line":""},{"line_number":1288,"context_line":"        # 10. Delete VNF"},{"line_number":1289,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1290,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1291,"context_line":""},{"line_number":1292,"context_line":"    def test_rollback_instantiate(self):"},{"line_number":1293,"context_line":"        \"\"\"Test rollback instantiate operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"49bb7bb4_5675624e","line":1290,"in_reply_to":"c17ba893_e0f578eb","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":1394,"context_line":""},{"line_number":1395,"context_line":"        # 6. Delete VNF"},{"line_number":1396,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1397,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"    def test_rollback_chgextconn_and_update(self):"},{"line_number":1400,"context_line":"        \"\"\"Test rollback change_ext_conn and update operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"621167bd_925bd439","line":1397,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":1394,"context_line":""},{"line_number":1395,"context_line":"        # 6. Delete VNF"},{"line_number":1396,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1397,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"    def test_rollback_chgextconn_and_update(self):"},{"line_number":1400,"context_line":"        \"\"\"Test rollback change_ext_conn and update operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"92e0357d_97619f4f","line":1397,"in_reply_to":"621167bd_925bd439","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":1394,"context_line":""},{"line_number":1395,"context_line":"        # 6. Delete VNF"},{"line_number":1396,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1397,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1398,"context_line":""},{"line_number":1399,"context_line":"    def test_rollback_chgextconn_and_update(self):"},{"line_number":1400,"context_line":"        \"\"\"Test rollback change_ext_conn and update operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"b4831e42_348c8672","line":1397,"in_reply_to":"92e0357d_97619f4f","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":1585,"context_line":""},{"line_number":1586,"context_line":"        # 12. Delete VNF"},{"line_number":1587,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1588,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1589,"context_line":""},{"line_number":1590,"context_line":"    def test_rollback_change_vnfpkg(self):"},{"line_number":1591,"context_line":"        \"\"\"Test rollback change_vnfpkg operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"c00cd0d4_633de9ea","line":1588,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":1585,"context_line":""},{"line_number":1586,"context_line":"        # 12. Delete VNF"},{"line_number":1587,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1588,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1589,"context_line":""},{"line_number":1590,"context_line":"    def test_rollback_change_vnfpkg(self):"},{"line_number":1591,"context_line":"        \"\"\"Test rollback change_vnfpkg operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"e6d5f7c3_11bceafb","line":1588,"in_reply_to":"c00cd0d4_633de9ea","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":1585,"context_line":""},{"line_number":1586,"context_line":"        # 12. Delete VNF"},{"line_number":1587,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1588,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1589,"context_line":""},{"line_number":1590,"context_line":"    def test_rollback_change_vnfpkg(self):"},{"line_number":1591,"context_line":"        \"\"\"Test rollback change_vnfpkg operation"}],"source_content_type":"text/x-python","patch_set":7,"id":"fc145973_f53895cc","line":1588,"in_reply_to":"e6d5f7c3_11bceafb","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":1811,"context_line":""},{"line_number":1812,"context_line":"        # 13. Delete VNF"},{"line_number":1813,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1814,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1815,"context_line":""},{"line_number":1816,"context_line":"    def test_various_lcm_operations_before_and_after(self):"},{"line_number":1817,"context_line":"        \"\"\"Test various vnflcm operations before and after"}],"source_content_type":"text/x-python","patch_set":7,"id":"3cd33dce_e4b5014b","line":1814,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":1811,"context_line":""},{"line_number":1812,"context_line":"        # 13. Delete VNF"},{"line_number":1813,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1814,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1815,"context_line":""},{"line_number":1816,"context_line":"    def test_various_lcm_operations_before_and_after(self):"},{"line_number":1817,"context_line":"        \"\"\"Test various vnflcm operations before and after"}],"source_content_type":"text/x-python","patch_set":7,"id":"46b8abfc_0dea7bc9","line":1814,"in_reply_to":"3cd33dce_e4b5014b","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":1811,"context_line":""},{"line_number":1812,"context_line":"        # 13. Delete VNF"},{"line_number":1813,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":1814,"context_line":"        self.assertEqual(204, resp.status_code)"},{"line_number":1815,"context_line":""},{"line_number":1816,"context_line":"    def test_various_lcm_operations_before_and_after(self):"},{"line_number":1817,"context_line":"        \"\"\"Test various vnflcm operations before and after"}],"source_content_type":"text/x-python","patch_set":7,"id":"c26765f7_5356cb46","line":1814,"in_reply_to":"46b8abfc_0dea7bc9","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"cd0c67b2b2ac696ea673f3c3eaa9588c439b5866","unresolved":true,"context_lines":[{"line_number":2398,"context_line":""},{"line_number":2399,"context_line":"        # 32. Delete VNF"},{"line_number":2400,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":2401,"context_line":"        self.assertEqual(204, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1fe46557_66cda937","line":2401,"updated":"2023-05-09 06:45:30.000000000","message":"Since `create` and `delete` are not specific to StandardUserData and are common processes, confirmation in this test is unnecessary.\nNo need to add this line.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"fcc067c18d212b2ed666d0445aa2ef55ad050987","unresolved":true,"context_lines":[{"line_number":2398,"context_line":""},{"line_number":2399,"context_line":"        # 32. Delete VNF"},{"line_number":2400,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":2401,"context_line":"        self.assertEqual(204, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":7,"id":"60d2ba8c_0bcd90ce","line":2401,"in_reply_to":"1fe46557_66cda937","updated":"2023-05-10 02:53:11.000000000","message":"I fixed it in PS8.","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"da37fff1595999b6bb830aaaaf42572ab8567d51","unresolved":false,"context_lines":[{"line_number":2398,"context_line":""},{"line_number":2399,"context_line":"        # 32. Delete VNF"},{"line_number":2400,"context_line":"        resp, body \u003d self.exec_lcmocc(self.delete_vnf_instance, inst_id)"},{"line_number":2401,"context_line":"        self.assertEqual(204, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":7,"id":"271bc6a2_963b82f6","line":2401,"in_reply_to":"60d2ba8c_0bcd90ce","updated":"2023-05-10 06:50:39.000000000","message":"Done","commit_id":"5b980752f5689a5b0660cec9ff0676d97d9412ae"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":true,"context_lines":[{"line_number":57,"context_line":"        # vnf package for update vnf max pattern"},{"line_number":58,"context_line":"        pkg_path_3 \u003d os.path.join(cur_dir,"},{"line_number":59,"context_line":"            \"../sol_v2_common/samples/update_vnf_max_individual_vnfc\")"},{"line_number":60,"context_line":"        cls.up_max_pkg, cls.up_max_vnfd_id \u003d cls.create_vnf_package("},{"line_number":61,"context_line":"            pkg_path_3, image_path\u003dimage_path,"},{"line_number":62,"context_line":"            userdata_path\u003duserdata_path)"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"9bcfbcf5_113498ee","line":60,"updated":"2023-06-09 05:10:13.000000000","message":"IMO, it is better to use `upd_` instead of `up_` as a shorthand for `update`.\nWhat do you think?","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":true,"context_lines":[{"line_number":57,"context_line":"        # vnf package for update vnf max pattern"},{"line_number":58,"context_line":"        pkg_path_3 \u003d os.path.join(cur_dir,"},{"line_number":59,"context_line":"            \"../sol_v2_common/samples/update_vnf_max_individual_vnfc\")"},{"line_number":60,"context_line":"        cls.up_max_pkg, cls.up_max_vnfd_id \u003d cls.create_vnf_package("},{"line_number":61,"context_line":"            pkg_path_3, image_path\u003dimage_path,"},{"line_number":62,"context_line":"            userdata_path\u003duserdata_path)"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"fa56763b_04e579c5","line":60,"in_reply_to":"9bcfbcf5_113498ee","updated":"2023-06-12 05:57:13.000000000","message":"I agree with you. I fixed it.","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        # vnf package for update vnf max pattern"},{"line_number":58,"context_line":"        pkg_path_3 \u003d os.path.join(cur_dir,"},{"line_number":59,"context_line":"            \"../sol_v2_common/samples/update_vnf_max_individual_vnfc\")"},{"line_number":60,"context_line":"        cls.up_max_pkg, cls.up_max_vnfd_id \u003d cls.create_vnf_package("},{"line_number":61,"context_line":"            pkg_path_3, image_path\u003dimage_path,"},{"line_number":62,"context_line":"            userdata_path\u003duserdata_path)"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"9fff7eb5_75a0c35d","line":60,"in_reply_to":"fa56763b_04e579c5","updated":"2023-06-14 17:26:12.000000000","message":"Done","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d71c15b378c03d5f1e34aa449b923bd3a53baca7","unresolved":true,"context_lines":[{"line_number":72,"context_line":"            cur_dir,"},{"line_number":73,"context_line":"            \"../sol_v2_common/samples/\""},{"line_number":74,"context_line":"            \"change_vnfpkg_or_update_min_individual_vnfc\")"},{"line_number":75,"context_line":"        cls.up_new_min_pkg, cls.up_new_min_vnfd_id \u003d cls.create_vnf_package("},{"line_number":76,"context_line":"            pkg_path_5, image_path\u003dimage_path, userdata_path\u003duserdata_path)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        cls.expected_list_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":13,"id":"b2dde67c_c82f5807","line":75,"updated":"2023-06-09 05:10:13.000000000","message":"ditto (up_new_min_pkg)","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"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":"d54f026aa66fcb1f598bd2b9c305bfdda38b9c98","unresolved":false,"context_lines":[{"line_number":72,"context_line":"            cur_dir,"},{"line_number":73,"context_line":"            \"../sol_v2_common/samples/\""},{"line_number":74,"context_line":"            \"change_vnfpkg_or_update_min_individual_vnfc\")"},{"line_number":75,"context_line":"        cls.up_new_min_pkg, cls.up_new_min_vnfd_id \u003d cls.create_vnf_package("},{"line_number":76,"context_line":"            pkg_path_5, image_path\u003dimage_path, userdata_path\u003duserdata_path)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        cls.expected_list_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":13,"id":"0fb9b8a0_88992071","line":75,"in_reply_to":"6c826d60_9d0dcb74","updated":"2023-06-14 17:26:12.000000000","message":"Done","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"a082aa06da0c49c73b1f1fa3ca376152f375a02d","unresolved":true,"context_lines":[{"line_number":72,"context_line":"            cur_dir,"},{"line_number":73,"context_line":"            \"../sol_v2_common/samples/\""},{"line_number":74,"context_line":"            \"change_vnfpkg_or_update_min_individual_vnfc\")"},{"line_number":75,"context_line":"        cls.up_new_min_pkg, cls.up_new_min_vnfd_id \u003d cls.create_vnf_package("},{"line_number":76,"context_line":"            pkg_path_5, image_path\u003dimage_path, userdata_path\u003duserdata_path)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        cls.expected_list_attrs \u003d ["}],"source_content_type":"text/x-python","patch_set":13,"id":"6c826d60_9d0dcb74","line":75,"in_reply_to":"b2dde67c_c82f5807","updated":"2023-06-12 05:57:13.000000000","message":"ditto","commit_id":"0ec3ddbdf9ae896bcb2967490f2a3430bf30e799"}]}
