)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af5edb25ababb00f089b7f04f12ab9e2946df75c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cbc5ede8_993ec248","updated":"2022-01-06 06:49:01.000000000","message":"Hi,\n\nI\u0027ve found [1] for \u0027stable/wallaby\u0027 cannot be passed the failed test without merging this patch cherry-picked from \u0027master\u0027.\n\n[1] https://review.opendev.org/c/openstack/python-tackerclient/+/822107","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2a8d22e75b99a11a3e16619323fd41a4457da160","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"47787717_a53339a1","updated":"2022-01-06 08:46:17.000000000","message":"LGTM.","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"720cf041a2eee2a4b41c9c2e5d8c216e59ab1b7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cd6e08f8_67a2392d","updated":"2022-01-06 07:34:40.000000000","message":"LGTM.","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"00b8c9d3e709d85af1c97a4ae656b43752e099a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"165216fe_59f3c145","updated":"2022-01-06 07:59:27.000000000","message":"Thanks for your patch, there are comments about variable naming.\nPlease kindly find my comments.","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"a796936ebe7f70d111e8a75ff9a552e48dbda48f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3513356f_a7bfd036","updated":"2022-01-17 04:39:58.000000000","message":"ping","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"}],"tackerclient/tests/unit/osc/v1/test_vnflcm_op_occs.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"00b8c9d3e709d85af1c97a4ae656b43752e099a0","unresolved":true,"context_lines":[{"line_number":360,"context_line":"            self.list_vnflcm_op_occ.get_attributes(), long_listing\u003dTrue)"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"        expected_data \u003d []"},{"line_number":363,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"},{"line_number":364,"context_line":"            expected_data.append(vnflcm_op_occs_fakes.get_vnflcm_op_occ_data("},{"line_number":365,"context_line":"                vnflcm_op_occ_obj_idx, columns\u003dcolumns))"},{"line_number":366,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e4773a9b_b1867c3d","line":363,"range":{"start_line":363,"start_character":29,"end_line":363,"end_character":33},"updated":"2022-01-06 07:59:27.000000000","message":"Does \"idx\" means index?\nIf so, I think this element in `vnflcm_op_occs_obj` is not \"index\".\n\nHow about changing variable name like the followings?\n* vnflcm_op_occs_obj -\u003e vnflcm_op_occs or vnflcm_op_occs_objs\n* vnflcm_op_occ_obj_idx -\u003e vnflcm_op_occ or vnflcm_op_occ_obj\n\nPlease ignore if I\u0027m wrong.","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"2a8d22e75b99a11a3e16619323fd41a4457da160","unresolved":true,"context_lines":[{"line_number":360,"context_line":"            self.list_vnflcm_op_occ.get_attributes(), long_listing\u003dTrue)"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"        expected_data \u003d []"},{"line_number":363,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"},{"line_number":364,"context_line":"            expected_data.append(vnflcm_op_occs_fakes.get_vnflcm_op_occ_data("},{"line_number":365,"context_line":"                vnflcm_op_occ_obj_idx, columns\u003dcolumns))"},{"line_number":366,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"157537cf_eae6341d","line":363,"range":{"start_line":363,"start_character":29,"end_line":363,"end_character":33},"in_reply_to":"8e7cf82d_a240a739","updated":"2022-01-06 08:46:17.000000000","message":"Thanks for your answer. I got it.\nIt\u0027s not a critical issue, so I think it\u0027s fine as it is.","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"3dde03590d152b6ca656cb11c3fa47793febc458","unresolved":true,"context_lines":[{"line_number":360,"context_line":"            self.list_vnflcm_op_occ.get_attributes(), long_listing\u003dTrue)"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"        expected_data \u003d []"},{"line_number":363,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"},{"line_number":364,"context_line":"            expected_data.append(vnflcm_op_occs_fakes.get_vnflcm_op_occ_data("},{"line_number":365,"context_line":"                vnflcm_op_occ_obj_idx, columns\u003dcolumns))"},{"line_number":366,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"8e7cf82d_a240a739","line":363,"range":{"start_line":363,"start_character":29,"end_line":363,"end_character":33},"in_reply_to":"e4773a9b_b1867c3d","updated":"2022-01-06 08:35:31.000000000","message":"This update is just a cherry-picking, so we should not make any change without critical one basically. You can propose changing variables for master and stable　in different patches if you think it\u0027s required. Thanks.","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"00b8c9d3e709d85af1c97a4ae656b43752e099a0","unresolved":true,"context_lines":[{"line_number":388,"context_line":"            self.list_vnflcm_op_occ.get_attributes(), long_listing\u003dTrue)"},{"line_number":389,"context_line":""},{"line_number":390,"context_line":"        expected_data \u003d []"},{"line_number":391,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"},{"line_number":392,"context_line":"            expected_data.append(vnflcm_op_occs_fakes.get_vnflcm_op_occ_data("},{"line_number":393,"context_line":"                vnflcm_op_occ_obj_idx, columns\u003dcolumns))"},{"line_number":394,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"2314c7bc_70381a86","line":391,"range":{"start_line":391,"start_character":12,"end_line":391,"end_character":33},"updated":"2022-01-06 07:59:27.000000000","message":"ditto","commit_id":"900a1aa78ef37404a38fbd2f140007aeaf2446b0"}]}
