)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"630f058314112c7fd74c26105c02bff32db7e5ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0f73cc39_89dd740e","updated":"2022-04-11 23:59:27.000000000","message":"recheck","commit_id":"d89c00640011bcc38b47406fcb29494c421ebe81"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"45a78fb8_3509fded","updated":"2022-05-16 06:49:07.000000000","message":"Thanks for your patch, please kindly find my comments.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"}],"tacker/tests/compliance/base.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":333,"context_line":"        cls.http_client.do_request(VNFINS_INST_PATH % vnfid,"},{"line_number":334,"context_line":"            \u0027POST\u0027, body\u003djsonutils.dumps(body))"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"        # cls._wait_vnf_status(vnfid, \u0027instantiationState\u0027, \u0027INSTANTIATED\u0027)"},{"line_number":337,"context_line":"        lcmid \u003d cls._get_lcm_op_occs_id(vnfid, lcm\u003d\u0027INSTANTIATE\u0027)"},{"line_number":338,"context_line":"        cls._wait_lcm_status(lcmid, value\u003d\u0027FAILED_TEMP\u0027)"},{"line_number":339,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"67c1d9b3_35a9b7aa","line":336,"range":{"start_line":336,"start_character":8,"end_line":336,"end_character":75},"updated":"2022-05-16 06:49:07.000000000","message":"Could you delete this line if it is not necessary comment.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":358,"context_line":"            VNFINS_GET_LCM_OP_OCCS_PATH, \u0027GET\u0027)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"        lcmid \u003d None"},{"line_number":361,"context_line":"        for entry in resbody:"},{"line_number":362,"context_line":"            lcm_dict \u003d entry"},{"line_number":363,"context_line":"            if ((lcm_dict[\u0027vnfInstanceId\u0027] \u003d\u003d vnfid) and"},{"line_number":364,"context_line":"                    (lcm_dict[\u0027operation\u0027] \u003d\u003d lcm)):"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a203bda_4422038f","line":361,"updated":"2022-05-16 06:49:07.000000000","message":"Don\u0027t you need to sort the new order of `start_time` or `state_entered_time`?\nFor example, do it get the same lcmid when test run scale out twice?\n(I think there is no need to consider if the resbody is in the order of newest.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"}],"tacker/tests/compliance/sol002/test_vnflcm.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        # Post-Conditions:"},{"line_number":143,"context_line":"        lcmid \u003d None"},{"line_number":144,"context_line":"        while(lcmid is None):"},{"line_number":145,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027], lcm\u003d\u0027MODIFY_INFO\u0027)"},{"line_number":146,"context_line":"            time.sleep(5)"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"        res \u003d self._wait_lcm_status(lcmid)"},{"line_number":149,"context_line":"        self._terminate_vnf_instance(vnf[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"21523a64_d3b43288","line":146,"range":{"start_line":144,"start_character":0,"end_line":146,"end_character":25},"updated":"2022-05-16 06:49:07.000000000","message":"How about you set a limit on the number of retries? This avoids the possibility of an infinite loop.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"dfef7ba213f655d2cc2029d141201d3ce6fd6be2","unresolved":true,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        variables \u003d [\u0027vnfInstanceId:\u0027 + vnf[\u0027id\u0027]]"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        body \u003d copy.deepcopy(rootbase.INSTANTIATION_BODY)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"        body[\u0027extVirtualLinks\u0027][0][\u0027resourceId\u0027] \u003d self.net0_id"},{"line_number":171,"context_line":"        body[\u0027vimConnectionInfo\u0027][0][\u0027id\u0027] \u003d uuidutils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":3,"id":"a6cc55ad_490ddce9","line":168,"updated":"2022-06-06 18:45:24.000000000","message":"It might the same comment for some previous update, please change the const to a function if it\u0027s required to use `deepcopy()` so many times. It\u0027s to avoid unexpected behavior if we make a mistake to use the const without `deepcopy()`.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":237,"context_line":""},{"line_number":238,"context_line":"        # Post-Conditions:"},{"line_number":239,"context_line":"        lcmid \u003d None"},{"line_number":240,"context_line":"        while(lcmid is None):"},{"line_number":241,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id_for_scale(vnf[\u0027id\u0027], scale_type,"},{"line_number":242,"context_line":"                    lcm\u003d\u0027SCALE\u0027)"},{"line_number":243,"context_line":"            time.sleep(5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ea44a045_da7d613b","line":240,"range":{"start_line":240,"start_character":8,"end_line":240,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"dfef7ba213f655d2cc2029d141201d3ce6fd6be2","unresolved":true,"context_lines":[{"line_number":240,"context_line":"        while(lcmid is None):"},{"line_number":241,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id_for_scale(vnf[\u0027id\u0027], scale_type,"},{"line_number":242,"context_line":"                    lcm\u003d\u0027SCALE\u0027)"},{"line_number":243,"context_line":"            time.sleep(5)"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"        res \u003d self._wait_lcm_status(lcmid)"},{"line_number":246,"context_line":"        self.assertEqual(0, res)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ef1a260_cb9d5491","line":243,"range":{"start_line":243,"start_character":23,"end_line":243,"end_character":24},"updated":"2022-06-06 18:45:24.000000000","message":"Why don\u0027t you define this value as a variable?","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        # Post-Conditions:"},{"line_number":262,"context_line":"        lcmid \u003d None"},{"line_number":263,"context_line":"        while(lcmid is None):"},{"line_number":264,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id_for_scale(vnf_id, scale_type,"},{"line_number":265,"context_line":"                lcm\u003d\u0027SCALE\u0027)"},{"line_number":266,"context_line":"            time.sleep(5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"14cb315d_a46216e1","line":263,"range":{"start_line":263,"start_character":8,"end_line":263,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":358,"context_line":""},{"line_number":359,"context_line":"        # Post-Conditions:"},{"line_number":360,"context_line":"        lcmid \u003d None"},{"line_number":361,"context_line":"        while(lcmid is None):"},{"line_number":362,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027], lcm\u003d\u0027HEAL\u0027)"},{"line_number":363,"context_line":"            time.sleep(5)"},{"line_number":364,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"c248f1b0_8297956a","line":361,"range":{"start_line":361,"start_character":8,"end_line":361,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":382,"context_line":"        self._instantiate_error_vnf_instance(vnf[\u0027id\u0027])"},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"        lcmid \u003d None"},{"line_number":385,"context_line":"        while(lcmid is None):"},{"line_number":386,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027])"},{"line_number":387,"context_line":"            time.sleep(5)"},{"line_number":388,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"46c45389_97e62b37","line":385,"range":{"start_line":385,"start_character":8,"end_line":385,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":429,"context_line":""},{"line_number":430,"context_line":"        # Post-Conditions:"},{"line_number":431,"context_line":"        lcmid \u003d None"},{"line_number":432,"context_line":"        while(lcmid is None):"},{"line_number":433,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027], lcm\u003d\u0027CHANGE_EXT_CONN\u0027)"},{"line_number":434,"context_line":"            time.sleep(5)"},{"line_number":435,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"edece8f4_3161ffb2","line":432,"range":{"start_line":432,"start_character":8,"end_line":432,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"}],"tacker/tests/compliance/sol003/test_vnflcm.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        # Post-Conditions:"},{"line_number":143,"context_line":"        lcmid \u003d None"},{"line_number":144,"context_line":"        while(lcmid is None):"},{"line_number":145,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027], lcm\u003d\u0027MODIFY_INFO\u0027)"},{"line_number":146,"context_line":"            time.sleep(5)"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"efb188af_44ea3d0c","line":144,"range":{"start_line":144,"start_character":8,"end_line":144,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"How about you set a limit on the number of retries? This avoids the possibility of an infinite loop.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":338,"context_line":""},{"line_number":339,"context_line":"        # Post-Conditions:"},{"line_number":340,"context_line":"        lcmid \u003d None"},{"line_number":341,"context_line":"        while(lcmid is None):"},{"line_number":342,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id_for_scale(vnf[\u0027id\u0027], scale_type,"},{"line_number":343,"context_line":"                lcm\u003d\u0027SCALE\u0027)"},{"line_number":344,"context_line":"            time.sleep(5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"2a1fb87a_bcece75d","line":341,"range":{"start_line":341,"start_character":8,"end_line":341,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":361,"context_line":""},{"line_number":362,"context_line":"        # Post-Conditions:"},{"line_number":363,"context_line":"        lcmid \u003d None"},{"line_number":364,"context_line":"        while(lcmid is None):"},{"line_number":365,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id_for_scale(vnf_id, scale_type,"},{"line_number":366,"context_line":"                lcm\u003d\u0027SCALE\u0027)"},{"line_number":367,"context_line":"            time.sleep(5)"}],"source_content_type":"text/x-python","patch_set":3,"id":"b0ada868_92ebe4e2","line":364,"range":{"start_line":364,"start_character":8,"end_line":364,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":398,"context_line":""},{"line_number":399,"context_line":"        # Post-Conditions:"},{"line_number":400,"context_line":"        lcmid \u003d None"},{"line_number":401,"context_line":"        while(lcmid is None):"},{"line_number":402,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027], lcm\u003d\u0027HEAL\u0027)"},{"line_number":403,"context_line":"            time.sleep(5)"},{"line_number":404,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f2240a04_557b29c6","line":401,"range":{"start_line":401,"start_character":8,"end_line":401,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":439,"context_line":""},{"line_number":440,"context_line":"        # Post-Conditions:"},{"line_number":441,"context_line":"        lcmid \u003d None"},{"line_number":442,"context_line":"        while(lcmid is None):"},{"line_number":443,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027], lcm\u003d\u0027CHANGE_EXT_CONN\u0027)"},{"line_number":444,"context_line":"            time.sleep(5)"},{"line_number":445,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"a44aa9b1_7dfa6e68","line":442,"range":{"start_line":442,"start_character":8,"end_line":442,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7dfba634d4ddbaade791aeec539c4b6f2eb1ce50","unresolved":true,"context_lines":[{"line_number":463,"context_line":"        self._instantiate_error_vnf_instance(vnf[\u0027id\u0027])"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"        lcmid \u003d None"},{"line_number":466,"context_line":"        while(lcmid is None):"},{"line_number":467,"context_line":"            lcmid \u003d self._get_lcm_op_occs_id(vnf[\u0027id\u0027])"},{"line_number":468,"context_line":"            time.sleep(5)"},{"line_number":469,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"b6a0f86a_81e6477f","line":466,"range":{"start_line":466,"start_character":8,"end_line":466,"end_character":29},"updated":"2022-05-16 06:49:07.000000000","message":"ditto.","commit_id":"c869af7df788f0895be841860684c8416f31a88a"}]}