)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1e3253e5929f7dcfda0db6e6854ef0c964113734","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2b15959d_82ed7b81","updated":"2022-02-09 03:05:11.000000000","message":"Thank you for your patch. Would you please check my comments.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"cbea5bd34dd33e9e0cb41114ca1ae6611e99014b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0dfd2dcf_383b3d8c","updated":"2022-01-26 00:44:31.000000000","message":"Thanks for your patch, please kindly find my comments.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"2c19a02d0063f62cd1c6bc25622426e647bb2cb1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"029c6c6f_756f61a7","updated":"2022-02-09 02:58:53.000000000","message":"Thanks for your work. I have some comments, so could you check that?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1da0944e68194d1c703865fe6c5968e0ba71c32d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7ff3d689_fd1f5006","updated":"2022-02-18 03:06:43.000000000","message":"Thank you rework.","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"10538691_3ea2e4fc","updated":"2022-02-18 01:48:31.000000000","message":"To Ayumu Ueha, Hirro Kitamura, Hirofumi Noguchi:\nThank you for your comments. I fixed them in PS 2.\nPlease kindly check them again.","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"eb7b9f4d9fe87b15b39c1535ca1cdbf6d5202835","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fcfe6db4_dc4a57ea","updated":"2022-02-18 05:18:13.000000000","message":"To Hirro Kitamura:\nThank you for your comments. I fixed them in PS3.\nPlease kindly check them again.","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"c9569606c02822e3b3773f24153b068d8d551205","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fd741bf7_aa3c0db7","updated":"2022-02-21 08:48:07.000000000","message":"I\u0027m sorry but I\u0027d like to correct my comment, could you confirm it?","commit_id":"51289fe20ee977d70544d8e37751c19e62dc7be8"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"722a4959410d38e159d4089d38933887b1c9714c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3e1ad5fc_dc8852f9","updated":"2022-02-21 04:27:40.000000000","message":"LGTM","commit_id":"51289fe20ee977d70544d8e37751c19e62dc7be8"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"38b0e76d03e0a5c8100262a0fce9090034820271","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"359a955d_74733184","updated":"2022-02-21 03:15:40.000000000","message":"To Hirro Kitamura:\nThank you for your rechecked, I fixed your comments in PS4.\nPlease kindly check them again.","commit_id":"51289fe20ee977d70544d8e37751c19e62dc7be8"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"34098dac85008aac04e6ade2a616b05f626dd4b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"899626bd_db9036a8","updated":"2022-02-24 00:18:14.000000000","message":"Sorry for the late confirmation, I have a additional comment. Please kindly check it.","commit_id":"1fe064c5d61e61a29a165c37aec5661500e6f0f8"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"3c10eada682048ca9d2b43aa481c01ea84ac8570","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"022842fe_90e9f769","updated":"2022-02-22 02:22:47.000000000","message":"Thank you!","commit_id":"1fe064c5d61e61a29a165c37aec5661500e6f0f8"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"372e20b19edfad87599dbe75f533dc49ce13d12e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3d3c26cd_277d435f","updated":"2022-02-22 01:30:55.000000000","message":"To Hiroo Kitamura:\nThank you for your comments. I reverted to \u0027vnflcm_change-vnfpkg\u0027 in PS5.\nPlease kindly check them again.","commit_id":"1fe064c5d61e61a29a165c37aec5661500e6f0f8"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"4e63e7df35ce00abc3e0a86781722685f644d2dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"41aaf63b_b2e9ecc6","updated":"2022-02-25 01:33:57.000000000","message":"Thanks for your rework, LGTM.","commit_id":"ab901b646626e5d77858bd2e768f3ca7b1edde0b"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"8899dc9923936f1bee391723e295822c569c165c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"df02ef07_fa11d890","updated":"2022-02-25 00:52:53.000000000","message":"To Ayumu Ueha:\nThank you for your comment. I fixed it in PS6.\nPlease kindly check it again.","commit_id":"ab901b646626e5d77858bd2e768f3ca7b1edde0b"}],"setup.cfg":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"2c19a02d0063f62cd1c6bc25622426e647bb2cb1","unresolved":true,"context_lines":[{"line_number":110,"context_line":"     vnflcm_list \u003d tackerclient.osc.v1.vnflcm.vnflcm:ListVnfLcm"},{"line_number":111,"context_line":"     vnflcm_instantiate \u003d tackerclient.osc.v1.vnflcm.vnflcm:InstantiateVnfLcm"},{"line_number":112,"context_line":"     vnflcm_terminate \u003d tackerclient.osc.v1.vnflcm.vnflcm:TerminateVnfLcm"},{"line_number":113,"context_line":"     vnflcm_change-vnfpkg \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeVnfPkgVnfLcm"},{"line_number":114,"context_line":"     vnflcm_delete \u003d tackerclient.osc.v1.vnflcm.vnflcm:DeleteVnfLcm"},{"line_number":115,"context_line":"     vnflcm_op_list \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ListVnfLcmOp"},{"line_number":116,"context_line":"     vnflcm_op_show \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ShowVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"1762b704_9f56b725","line":113,"range":{"start_line":113,"start_character":5,"end_line":113,"end_character":25},"updated":"2022-02-09 02:58:53.000000000","message":"It\u0027s better to use \u0027_\u0027 instead of \u0027-\u0027.\n(i.e.) vnflcm_change_vnfpkg","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":110,"context_line":"     vnflcm_list \u003d tackerclient.osc.v1.vnflcm.vnflcm:ListVnfLcm"},{"line_number":111,"context_line":"     vnflcm_instantiate \u003d tackerclient.osc.v1.vnflcm.vnflcm:InstantiateVnfLcm"},{"line_number":112,"context_line":"     vnflcm_terminate \u003d tackerclient.osc.v1.vnflcm.vnflcm:TerminateVnfLcm"},{"line_number":113,"context_line":"     vnflcm_change-vnfpkg \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeVnfPkgVnfLcm"},{"line_number":114,"context_line":"     vnflcm_delete \u003d tackerclient.osc.v1.vnflcm.vnflcm:DeleteVnfLcm"},{"line_number":115,"context_line":"     vnflcm_op_list \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ListVnfLcmOp"},{"line_number":116,"context_line":"     vnflcm_op_show \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ShowVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"c786132f_a914130a","line":113,"range":{"start_line":113,"start_character":5,"end_line":113,"end_character":25},"in_reply_to":"1762b704_9f56b725","updated":"2022-02-18 01:48:31.000000000","message":"I fixed it in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"3c10eada682048ca9d2b43aa481c01ea84ac8570","unresolved":false,"context_lines":[{"line_number":110,"context_line":"     vnflcm_list \u003d tackerclient.osc.v1.vnflcm.vnflcm:ListVnfLcm"},{"line_number":111,"context_line":"     vnflcm_instantiate \u003d tackerclient.osc.v1.vnflcm.vnflcm:InstantiateVnfLcm"},{"line_number":112,"context_line":"     vnflcm_terminate \u003d tackerclient.osc.v1.vnflcm.vnflcm:TerminateVnfLcm"},{"line_number":113,"context_line":"     vnflcm_change-vnfpkg \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeVnfPkgVnfLcm"},{"line_number":114,"context_line":"     vnflcm_delete \u003d tackerclient.osc.v1.vnflcm.vnflcm:DeleteVnfLcm"},{"line_number":115,"context_line":"     vnflcm_op_list \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ListVnfLcmOp"},{"line_number":116,"context_line":"     vnflcm_op_show \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ShowVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"d619f532_e7969dbb","line":113,"range":{"start_line":113,"start_character":5,"end_line":113,"end_character":25},"in_reply_to":"32437e4e_21e46b73","updated":"2022-02-22 02:22:47.000000000","message":"Done","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"372e20b19edfad87599dbe75f533dc49ce13d12e","unresolved":true,"context_lines":[{"line_number":110,"context_line":"     vnflcm_list \u003d tackerclient.osc.v1.vnflcm.vnflcm:ListVnfLcm"},{"line_number":111,"context_line":"     vnflcm_instantiate \u003d tackerclient.osc.v1.vnflcm.vnflcm:InstantiateVnfLcm"},{"line_number":112,"context_line":"     vnflcm_terminate \u003d tackerclient.osc.v1.vnflcm.vnflcm:TerminateVnfLcm"},{"line_number":113,"context_line":"     vnflcm_change-vnfpkg \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeVnfPkgVnfLcm"},{"line_number":114,"context_line":"     vnflcm_delete \u003d tackerclient.osc.v1.vnflcm.vnflcm:DeleteVnfLcm"},{"line_number":115,"context_line":"     vnflcm_op_list \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ListVnfLcmOp"},{"line_number":116,"context_line":"     vnflcm_op_show \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ShowVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"32437e4e_21e46b73","line":113,"range":{"start_line":113,"start_character":5,"end_line":113,"end_character":25},"in_reply_to":"4f3ce9f0_f9e3f957","updated":"2022-02-22 01:30:55.000000000","message":"I reverted to \"vnflcm_change-vnfpkg\" in PS5.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"543c2927e1b73d60e74145cc096059f238cbb41d","unresolved":false,"context_lines":[{"line_number":110,"context_line":"     vnflcm_list \u003d tackerclient.osc.v1.vnflcm.vnflcm:ListVnfLcm"},{"line_number":111,"context_line":"     vnflcm_instantiate \u003d tackerclient.osc.v1.vnflcm.vnflcm:InstantiateVnfLcm"},{"line_number":112,"context_line":"     vnflcm_terminate \u003d tackerclient.osc.v1.vnflcm.vnflcm:TerminateVnfLcm"},{"line_number":113,"context_line":"     vnflcm_change-vnfpkg \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeVnfPkgVnfLcm"},{"line_number":114,"context_line":"     vnflcm_delete \u003d tackerclient.osc.v1.vnflcm.vnflcm:DeleteVnfLcm"},{"line_number":115,"context_line":"     vnflcm_op_list \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ListVnfLcmOp"},{"line_number":116,"context_line":"     vnflcm_op_show \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ShowVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"eb515636_eed96847","line":113,"range":{"start_line":113,"start_character":5,"end_line":113,"end_character":25},"in_reply_to":"c786132f_a914130a","updated":"2022-02-18 02:28:12.000000000","message":"Done","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"c9569606c02822e3b3773f24153b068d8d551205","unresolved":true,"context_lines":[{"line_number":110,"context_line":"     vnflcm_list \u003d tackerclient.osc.v1.vnflcm.vnflcm:ListVnfLcm"},{"line_number":111,"context_line":"     vnflcm_instantiate \u003d tackerclient.osc.v1.vnflcm.vnflcm:InstantiateVnfLcm"},{"line_number":112,"context_line":"     vnflcm_terminate \u003d tackerclient.osc.v1.vnflcm.vnflcm:TerminateVnfLcm"},{"line_number":113,"context_line":"     vnflcm_change-vnfpkg \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeVnfPkgVnfLcm"},{"line_number":114,"context_line":"     vnflcm_delete \u003d tackerclient.osc.v1.vnflcm.vnflcm:DeleteVnfLcm"},{"line_number":115,"context_line":"     vnflcm_op_list \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ListVnfLcmOp"},{"line_number":116,"context_line":"     vnflcm_op_show \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:ShowVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"4f3ce9f0_f9e3f957","line":113,"range":{"start_line":113,"start_character":5,"end_line":113,"end_character":25},"in_reply_to":"eb515636_eed96847","updated":"2022-02-21 08:48:07.000000000","message":"I\u0027m so sorry to have you fix it as I pointed out.\nWith this fix, when executing the CLI command, we had to specify \"vnflcm change ext conn\", which was an unnatural command when compared to other commands(e.g. \"vnflcm op rollback\" and \"vnflcm op retry\").\nSorry to bother you, but could you revert to \"vnflcm_change-vnfpkg\"?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"}],"tackerclient/common/exceptions.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"cbea5bd34dd33e9e0cb41114ca1ae6611e99014b","unresolved":true,"context_lines":[{"line_number":207,"context_line":"    message \u003d _(\"Invalid input: %(reason)s\")"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedVersionForChangeVnfpkg(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f738bd5f_6ef6100b","line":210,"range":{"start_line":210,"start_character":6,"end_line":210,"end_character":39},"updated":"2022-01-26 00:44:31.000000000","message":"If other commands might require similar exceptions, why don\u0027t you change to a common definition?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"722a4959410d38e159d4089d38933887b1c9714c","unresolved":true,"context_lines":[{"line_number":207,"context_line":"    message \u003d _(\"Invalid input: %(reason)s\")"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedVersionForChangeVnfpkg(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7ec4b8e5_ba7ddf87","line":210,"range":{"start_line":210,"start_character":6,"end_line":210,"end_character":39},"in_reply_to":"4af2f1e1_dcac0842","updated":"2022-02-21 04:27:40.000000000","message":"LGTM.\nUeha san, if you are OK too, could you change to resolved?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"34098dac85008aac04e6ade2a616b05f626dd4b1","unresolved":false,"context_lines":[{"line_number":207,"context_line":"    message \u003d _(\"Invalid input: %(reason)s\")"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedVersionForChangeVnfpkg(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"2f689cf9_0b827386","line":210,"range":{"start_line":210,"start_character":6,"end_line":210,"end_character":39},"in_reply_to":"7ec4b8e5_ba7ddf87","updated":"2022-02-24 00:18:14.000000000","message":"LGTM too.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":207,"context_line":"    message \u003d _(\"Invalid input: %(reason)s\")"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedVersionForChangeVnfpkg(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"4af2f1e1_dcac0842","line":210,"range":{"start_line":210,"start_character":6,"end_line":210,"end_character":39},"in_reply_to":"ce820ea2_60f67a9c","updated":"2022-02-18 01:48:31.000000000","message":"I have changed it to UnsupportedCommandVersion in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"2c19a02d0063f62cd1c6bc25622426e647bb2cb1","unresolved":true,"context_lines":[{"line_number":207,"context_line":"    message \u003d _(\"Invalid input: %(reason)s\")"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedVersionForChangeVnfpkg(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ce820ea2_60f67a9c","line":210,"range":{"start_line":210,"start_character":6,"end_line":210,"end_character":39},"in_reply_to":"f738bd5f_6ef6100b","updated":"2022-02-09 02:58:53.000000000","message":"I agree with Ueha san.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"543c2927e1b73d60e74145cc096059f238cbb41d","unresolved":true,"context_lines":[{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedCommandVersion(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"# Command line exceptions"}],"source_content_type":"text/x-python","patch_set":2,"id":"ed73dc18_2f7c8dab","line":211,"range":{"start_line":211,"start_character":44,"end_line":211,"end_character":46},"updated":"2022-02-18 02:28:12.000000000","message":"I think it would be better if this could be made more common by making it a variable as shown below.\n---\n\"This command is not supported in version %(version)s\"","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"3695be29ea46854b3f25261832b1d4cc2841cb38","unresolved":false,"context_lines":[{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedCommandVersion(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"# Command line exceptions"}],"source_content_type":"text/x-python","patch_set":2,"id":"74298f18_bcf8ef32","line":211,"range":{"start_line":211,"start_character":44,"end_line":211,"end_character":46},"in_reply_to":"6554cde4_6d3a80dc","updated":"2022-02-18 09:17:03.000000000","message":"Done","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"eb7b9f4d9fe87b15b39c1535ca1cdbf6d5202835","unresolved":true,"context_lines":[{"line_number":208,"context_line":""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"class UnsupportedCommandVersion(TackerClientException):"},{"line_number":211,"context_line":"    message \u003d _(\"This command only supports V2 version.\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"# Command line exceptions"}],"source_content_type":"text/x-python","patch_set":2,"id":"6554cde4_6d3a80dc","line":211,"range":{"start_line":211,"start_character":44,"end_line":211,"end_character":46},"in_reply_to":"ed73dc18_2f7c8dab","updated":"2022-02-18 05:18:13.000000000","message":"I fixed it in PS 3.","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"}],"tackerclient/osc/v1/vnflcm/samples/change_vnfpkg_vnf_instance_param_sample.json":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"2c19a02d0063f62cd1c6bc25622426e647bb2cb1","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"\t\"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"\t\"additionalParams\": {"},{"line_number":4,"context_line":"\t\t\"upgrade_type\": \"RollingUpdate\","}],"source_content_type":"application/json","patch_set":1,"id":"894a29e3_683ee56d","line":1,"updated":"2022-02-09 02:58:53.000000000","message":"How about creating and placing the following new directories to clarify that they are dedicated to the V2-API?\n```\ntackerclient/osc/v2/vnflcm/samples/change_vnfpkg_vnf_instance_param_sample.json\n```","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"\t\"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"\t\"additionalParams\": {"},{"line_number":4,"context_line":"\t\t\"upgrade_type\": \"RollingUpdate\","}],"source_content_type":"application/json","patch_set":1,"id":"9a0fac10_d6946856","line":1,"in_reply_to":"894a29e3_683ee56d","updated":"2022-02-18 01:48:31.000000000","message":"I added this directory in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"543c2927e1b73d60e74145cc096059f238cbb41d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"\t\"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"\t\"additionalParams\": {"},{"line_number":4,"context_line":"\t\t\"upgrade_type\": \"RollingUpdate\","}],"source_content_type":"application/json","patch_set":1,"id":"e073ac15_e4978ce4","line":1,"in_reply_to":"9a0fac10_d6946856","updated":"2022-02-18 02:28:12.000000000","message":"Done","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"cbea5bd34dd33e9e0cb41114ca1ae6611e99014b","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"\t\"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"\t\"additionalParams\": {"},{"line_number":4,"context_line":"\t\t\"upgrade_type\": \"RollingUpdate\","},{"line_number":5,"context_line":"\t\t\"lcm-operation-coordinate-old-vnf\": \"./Scripts/coordinate_old_vnf.py\","}],"source_content_type":"application/json","patch_set":1,"id":"fb8fa4ab_315a043d","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":1},"updated":"2022-01-26 00:44:31.000000000","message":"Could you use spaces instead of tabs?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"34098dac85008aac04e6ade2a616b05f626dd4b1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"\t\"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"\t\"additionalParams\": {"},{"line_number":4,"context_line":"\t\t\"upgrade_type\": \"RollingUpdate\","},{"line_number":5,"context_line":"\t\t\"lcm-operation-coordinate-old-vnf\": \"./Scripts/coordinate_old_vnf.py\","}],"source_content_type":"application/json","patch_set":1,"id":"b6dbbbb5_4824cda8","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":1},"in_reply_to":"a5f5583d_c0c44492","updated":"2022-02-24 00:18:14.000000000","message":"Ack","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"\t\"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"\t\"additionalParams\": {"},{"line_number":4,"context_line":"\t\t\"upgrade_type\": \"RollingUpdate\","},{"line_number":5,"context_line":"\t\t\"lcm-operation-coordinate-old-vnf\": \"./Scripts/coordinate_old_vnf.py\","}],"source_content_type":"application/json","patch_set":1,"id":"a5f5583d_c0c44492","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":1},"in_reply_to":"fb8fa4ab_315a043d","updated":"2022-02-18 01:48:31.000000000","message":"I have fixed it in PS 2. And changed its newline character.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"}],"tackerclient/osc/v1/vnflcm/vnflcm.py":[{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1e3253e5929f7dcfda0db6e6854ef0c964113734","unresolved":true,"context_lines":[{"line_number":569,"context_line":""},{"line_number":570,"context_line":""},{"line_number":571,"context_line":"class ChangeVnfPkgVnfLcm(command.Command):"},{"line_number":572,"context_line":"    _description \u003d _(\"Change Current VNFPackage\")"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"    def get_parser(self, prog_name):"},{"line_number":575,"context_line":"        parser \u003d super(ChangeVnfPkgVnfLcm, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"786cfc8b_0e56574b","line":572,"updated":"2022-02-09 03:05:11.000000000","message":"It is better that the operation name is same as one in SOL.\nSOL003 v3.3.1 defines \"Change current VNF package\".\nSo, there should be a space between \"VNF\"  and \"Package\".","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1da0944e68194d1c703865fe6c5968e0ba71c32d","unresolved":false,"context_lines":[{"line_number":569,"context_line":""},{"line_number":570,"context_line":""},{"line_number":571,"context_line":"class ChangeVnfPkgVnfLcm(command.Command):"},{"line_number":572,"context_line":"    _description \u003d _(\"Change Current VNFPackage\")"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"    def get_parser(self, prog_name):"},{"line_number":575,"context_line":"        parser \u003d super(ChangeVnfPkgVnfLcm, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf173315_50af8776","line":572,"in_reply_to":"5981ab4a_863e81bb","updated":"2022-02-18 03:06:43.000000000","message":"Done.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":569,"context_line":""},{"line_number":570,"context_line":""},{"line_number":571,"context_line":"class ChangeVnfPkgVnfLcm(command.Command):"},{"line_number":572,"context_line":"    _description \u003d _(\"Change Current VNFPackage\")"},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"    def get_parser(self, prog_name):"},{"line_number":575,"context_line":"        parser \u003d super(ChangeVnfPkgVnfLcm, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5981ab4a_863e81bb","line":572,"in_reply_to":"786cfc8b_0e56574b","updated":"2022-02-18 01:48:31.000000000","message":"I fixed it in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1e3253e5929f7dcfda0db6e6854ef0c964113734","unresolved":true,"context_lines":[{"line_number":576,"context_line":"        parser.add_argument("},{"line_number":577,"context_line":"            _VNF_INSTANCE,"},{"line_number":578,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":579,"context_line":"            help\u003d_(\"VNF instance ID to Change Current VNFPackage\"))"},{"line_number":580,"context_line":"        parser.add_argument("},{"line_number":581,"context_line":"            \u0027request_file\u0027,"},{"line_number":582,"context_line":"            metavar\u003d\"\u003cparam-file\u003e\","}],"source_content_type":"text/x-python","patch_set":1,"id":"34d28ce9_befeac09","line":579,"updated":"2022-02-09 03:05:11.000000000","message":"ditto.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":576,"context_line":"        parser.add_argument("},{"line_number":577,"context_line":"            _VNF_INSTANCE,"},{"line_number":578,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":579,"context_line":"            help\u003d_(\"VNF instance ID to Change Current VNFPackage\"))"},{"line_number":580,"context_line":"        parser.add_argument("},{"line_number":581,"context_line":"            \u0027request_file\u0027,"},{"line_number":582,"context_line":"            metavar\u003d\"\u003cparam-file\u003e\","}],"source_content_type":"text/x-python","patch_set":1,"id":"e7433880_9a5d0306","line":579,"in_reply_to":"34d28ce9_befeac09","updated":"2022-02-18 01:48:31.000000000","message":"ditto","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1da0944e68194d1c703865fe6c5968e0ba71c32d","unresolved":false,"context_lines":[{"line_number":576,"context_line":"        parser.add_argument("},{"line_number":577,"context_line":"            _VNF_INSTANCE,"},{"line_number":578,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":579,"context_line":"            help\u003d_(\"VNF instance ID to Change Current VNFPackage\"))"},{"line_number":580,"context_line":"        parser.add_argument("},{"line_number":581,"context_line":"            \u0027request_file\u0027,"},{"line_number":582,"context_line":"            metavar\u003d\"\u003cparam-file\u003e\","}],"source_content_type":"text/x-python","patch_set":1,"id":"474cec35_a0d54b88","line":579,"in_reply_to":"e7433880_9a5d0306","updated":"2022-02-18 03:06:43.000000000","message":"Done.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1e3253e5929f7dcfda0db6e6854ef0c964113734","unresolved":true,"context_lines":[{"line_number":591,"context_line":"            parsed_args.vnf_instance, jsonfile2body("},{"line_number":592,"context_line":"                parsed_args.request_file))"},{"line_number":593,"context_line":"        if not result:"},{"line_number":594,"context_line":"            print((_(\u0027Change Current VNFPackage for VNF Instance %s \u0027"},{"line_number":595,"context_line":"                     \u0027has been accepted.\u0027) % parsed_args.vnf_instance))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b6ab03ab_8aa873fa","line":594,"updated":"2022-02-09 03:05:11.000000000","message":"ditto.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":591,"context_line":"            parsed_args.vnf_instance, jsonfile2body("},{"line_number":592,"context_line":"                parsed_args.request_file))"},{"line_number":593,"context_line":"        if not result:"},{"line_number":594,"context_line":"            print((_(\u0027Change Current VNFPackage for VNF Instance %s \u0027"},{"line_number":595,"context_line":"                     \u0027has been accepted.\u0027) % parsed_args.vnf_instance))"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffa70f6e_87acac44","line":594,"in_reply_to":"b6ab03ab_8aa873fa","updated":"2022-02-18 01:48:31.000000000","message":"ditto","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"1da0944e68194d1c703865fe6c5968e0ba71c32d","unresolved":false,"context_lines":[{"line_number":591,"context_line":"            parsed_args.vnf_instance, jsonfile2body("},{"line_number":592,"context_line":"                parsed_args.request_file))"},{"line_number":593,"context_line":"        if not result:"},{"line_number":594,"context_line":"            print((_(\u0027Change Current VNFPackage for VNF Instance %s \u0027"},{"line_number":595,"context_line":"                     \u0027has been accepted.\u0027) % parsed_args.vnf_instance))"}],"source_content_type":"text/x-python","patch_set":1,"id":"794453de_d2b6438d","line":594,"in_reply_to":"ffa70f6e_87acac44","updated":"2022-02-18 03:06:43.000000000","message":"Done.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"}],"tackerclient/osc/v2/vnflcm/samples/change_vnfpkg_vnf_instance_param_sample.json":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"543c2927e1b73d60e74145cc096059f238cbb41d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"          \"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"          \"additionalParams\": {"},{"line_number":4,"context_line":"                     \"upgrade_type\": \"RollingUpdate\","},{"line_number":5,"context_line":"                     \"lcm-operation-coordinate-old-vnf\": \"./Scripts/coordinate_old_vnf.py\","}],"source_content_type":"application/json","patch_set":2,"id":"f7bdae22_6d11a0c7","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":10},"updated":"2022-02-18 02:28:12.000000000","message":"As with the other samples, it\u0027s better to have four spaces \"    \".","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"3695be29ea46854b3f25261832b1d4cc2841cb38","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"          \"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"          \"additionalParams\": {"},{"line_number":4,"context_line":"                     \"upgrade_type\": \"RollingUpdate\","},{"line_number":5,"context_line":"                     \"lcm-operation-coordinate-old-vnf\": \"./Scripts/coordinate_old_vnf.py\","}],"source_content_type":"application/json","patch_set":2,"id":"19ec360c_121ca178","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":10},"in_reply_to":"5ff2d7f3_16fc264c","updated":"2022-02-18 09:17:03.000000000","message":"Done","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"eb7b9f4d9fe87b15b39c1535ca1cdbf6d5202835","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"          \"vnfdId\": \"c6595341-a5bb-8246-53c4-7aeb843d60c5\","},{"line_number":3,"context_line":"          \"additionalParams\": {"},{"line_number":4,"context_line":"                     \"upgrade_type\": \"RollingUpdate\","},{"line_number":5,"context_line":"                     \"lcm-operation-coordinate-old-vnf\": \"./Scripts/coordinate_old_vnf.py\","}],"source_content_type":"application/json","patch_set":2,"id":"5ff2d7f3_16fc264c","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":10},"in_reply_to":"f7bdae22_6d11a0c7","updated":"2022-02-18 05:18:13.000000000","message":"I fixed them in PS 3.","commit_id":"6f42843e162714a6e858e4b8014742f414148c2a"}],"tackerclient/tests/unit/osc/v1/test_vnflcm.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"3695be29ea46854b3f25261832b1d4cc2841cb38","unresolved":true,"context_lines":[{"line_number":880,"context_line":"                               self.change_vnfpkg_vnf_lcm.take_action,"},{"line_number":881,"context_line":"                               parsed_args)"},{"line_number":882,"context_line":""},{"line_number":883,"context_line":"        expected_msg \u003d \"This command is not supported in version 2.0.0\""},{"line_number":884,"context_line":"        self.assertEqual(expected_msg, str(ex))"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"0a90d249_f3253db9","line":883,"range":{"start_line":883,"start_character":65,"end_line":883,"end_character":70},"updated":"2022-02-18 09:17:03.000000000","message":"I think \"1\" is correct.\nAlso, I don\u0027t think you have checked the minor version, so it is better to set it to \"1\".","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"38b0e76d03e0a5c8100262a0fce9090034820271","unresolved":true,"context_lines":[{"line_number":880,"context_line":"                               self.change_vnfpkg_vnf_lcm.take_action,"},{"line_number":881,"context_line":"                               parsed_args)"},{"line_number":882,"context_line":""},{"line_number":883,"context_line":"        expected_msg \u003d \"This command is not supported in version 2.0.0\""},{"line_number":884,"context_line":"        self.assertEqual(expected_msg, str(ex))"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"15934e24_2a1a4c1b","line":883,"range":{"start_line":883,"start_character":65,"end_line":883,"end_character":70},"in_reply_to":"0a90d249_f3253db9","updated":"2022-02-21 03:15:40.000000000","message":"I fixed it in PS4.","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"722a4959410d38e159d4089d38933887b1c9714c","unresolved":false,"context_lines":[{"line_number":880,"context_line":"                               self.change_vnfpkg_vnf_lcm.take_action,"},{"line_number":881,"context_line":"                               parsed_args)"},{"line_number":882,"context_line":""},{"line_number":883,"context_line":"        expected_msg \u003d \"This command is not supported in version 2.0.0\""},{"line_number":884,"context_line":"        self.assertEqual(expected_msg, str(ex))"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"2f44e4c8_4417aa83","line":883,"range":{"start_line":883,"start_character":65,"end_line":883,"end_character":70},"in_reply_to":"15934e24_2a1a4c1b","updated":"2022-02-21 04:27:40.000000000","message":"Done","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"}],"tackerclient/tests/unit/osc/v2/test_vnflcm.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"cbea5bd34dd33e9e0cb41114ca1ae6611e99014b","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"b1a834dc_a43ce9a0","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"updated":"2022-01-26 00:44:31.000000000","message":"Isn\u0027t the order of definition of mock reversed?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f6cd52f6038121c63eed7687526b9bbc590c9b1d","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"72c31e8d_15b27c12","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"2cc49304_6fa3c157","updated":"2022-02-24 08:52:17.000000000","message":"I think the following.\n\nThe mock for `os.open` is `mock_access` and the mock for `os.access` is `mock_open`.\n\n`mock_access` returns the expected return value of `os.open`, and the other one too.\nSo there seems to be no problem with the test.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"1f3d721299975bce8daf4d2f49cb70bdab5d28f2","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"2cc49304_6fa3c157","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"6d8191dc_67c3e0a5","updated":"2022-02-24 08:31:55.000000000","message":"Do you know why the test has been passed even though the mistake???","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"8899dc9923936f1bee391723e295822c569c165c","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"94f05daf_4c7ffef5","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"72c31e8d_15b27c12","updated":"2022-02-25 00:52:53.000000000","message":"I fixed it in PS6.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"fed71929fc82b2775048c8c2753c0c0165962215","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"87a8e89a_63a905c2","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"72c31e8d_15b27c12","updated":"2022-02-24 13:52:43.000000000","message":"Umm... anyway, we should fix the patch before the deadline 25th. Thanks.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"4e63e7df35ce00abc3e0a86781722685f644d2dc","unresolved":false,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"96f8bbba_ff855fde","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"94f05daf_4c7ffef5","updated":"2022-02-25 01:33:57.000000000","message":"Thanks for updating.\n\nI think there is no problem because of the following:\n`os.access` returns True or False, that indicates whether the file is accessible or not.\n`os.open` returns \"file object\".\nhttps://docs.python.org/3/library/functions.html#open","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfd821f4_6a9cdb47","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"b1a834dc_a43ce9a0","updated":"2022-02-18 01:48:31.000000000","message":"I added it\u0027s `return_value` in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"34098dac85008aac04e6ade2a616b05f626dd4b1","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch(\"os.open\")"},{"line_number":129,"context_line":"    @mock.patch(\"os.access\")"},{"line_number":130,"context_line":"    def test_take_action_invalid_format_param_file(self, mock_open,"},{"line_number":131,"context_line":"                                                   mock_access):"},{"line_number":132,"context_line":"        vnf_instance \u003d vnflcm_fakes.vnf_instance_response()"},{"line_number":133,"context_line":"        sample_param_file \u003d \"./invalid_param_file.json\""},{"line_number":134,"context_line":"        arglist \u003d [vnf_instance[\u0027id\u0027], sample_param_file]"}],"source_content_type":"text/x-python","patch_set":1,"id":"6d8191dc_67c3e0a5","line":131,"range":{"start_line":130,"start_character":57,"end_line":131,"end_character":62},"in_reply_to":"bfd821f4_6a9cdb47","updated":"2022-02-24 00:18:14.000000000","message":"I think the order of the mock arguments should be below:\n\n```\n    @mock.patch(\"os.open\")\n    @mock.patch(\"os.access\")\n    def test_take_action_invalid_format_param_file(self, mock_access,\n                                                   mock_open):\n```","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"}],"tackerclient/v1_0/client.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"2c19a02d0063f62cd1c6bc25622426e647bb2cb1","unresolved":true,"context_lines":[{"line_number":967,"context_line":"                         vnf_id, body\u003dbody, headers\u003dself.headers)"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"    @APIParamsCall"},{"line_number":970,"context_line":"    def change_vnfpkg_vnf_instance(self, vnf_id, body):"},{"line_number":971,"context_line":"        if self.vnf_instance_path.split(\u0027/\u0027)[2] \u003d\u003d \u0027v2\u0027:"},{"line_number":972,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":973,"context_line":"                             vnf_id, body\u003dbody)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2ba6fdb7_8b51dec7","line":970,"updated":"2022-02-09 02:58:53.000000000","message":"It\u0027s better to describe in NOTE that it is only supported by V2-API.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":967,"context_line":"                         vnf_id, body\u003dbody, headers\u003dself.headers)"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"    @APIParamsCall"},{"line_number":970,"context_line":"    def change_vnfpkg_vnf_instance(self, vnf_id, body):"},{"line_number":971,"context_line":"        if self.vnf_instance_path.split(\u0027/\u0027)[2] \u003d\u003d \u0027v2\u0027:"},{"line_number":972,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":973,"context_line":"                             vnf_id, body\u003dbody)"}],"source_content_type":"text/x-python","patch_set":1,"id":"686c7a35_1a50f797","line":970,"in_reply_to":"2ba6fdb7_8b51dec7","updated":"2022-02-18 01:48:31.000000000","message":"I added this note in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"543c2927e1b73d60e74145cc096059f238cbb41d","unresolved":false,"context_lines":[{"line_number":967,"context_line":"                         vnf_id, body\u003dbody, headers\u003dself.headers)"},{"line_number":968,"context_line":""},{"line_number":969,"context_line":"    @APIParamsCall"},{"line_number":970,"context_line":"    def change_vnfpkg_vnf_instance(self, vnf_id, body):"},{"line_number":971,"context_line":"        if self.vnf_instance_path.split(\u0027/\u0027)[2] \u003d\u003d \u0027v2\u0027:"},{"line_number":972,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":973,"context_line":"                             vnf_id, body\u003dbody)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a14b0441_0a4794b6","line":970,"in_reply_to":"686c7a35_1a50f797","updated":"2022-02-18 02:28:12.000000000","message":"Done","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"2c19a02d0063f62cd1c6bc25622426e647bb2cb1","unresolved":true,"context_lines":[{"line_number":970,"context_line":"    def change_vnfpkg_vnf_instance(self, vnf_id, body):"},{"line_number":971,"context_line":"        if self.vnf_instance_path.split(\u0027/\u0027)[2] \u003d\u003d \u0027v2\u0027:"},{"line_number":972,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":973,"context_line":"                             vnf_id, body\u003dbody)"},{"line_number":974,"context_line":"        else:"},{"line_number":975,"context_line":"            raise exceptions.UnsupportedVersionForChangeVnfpkg()"},{"line_number":976,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"698781c8_7cba108f","line":973,"range":{"start_line":973,"start_character":46,"end_line":973,"end_character":47},"updated":"2022-02-09 02:58:53.000000000","message":"Don\u0027t you need \"headers \u003d self.headers\"?","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"c0045df133187168844e06f93849e764fe18ec63","unresolved":true,"context_lines":[{"line_number":970,"context_line":"    def change_vnfpkg_vnf_instance(self, vnf_id, body):"},{"line_number":971,"context_line":"        if self.vnf_instance_path.split(\u0027/\u0027)[2] \u003d\u003d \u0027v2\u0027:"},{"line_number":972,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":973,"context_line":"                             vnf_id, body\u003dbody)"},{"line_number":974,"context_line":"        else:"},{"line_number":975,"context_line":"            raise exceptions.UnsupportedVersionForChangeVnfpkg()"},{"line_number":976,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9c3bc5e9_5ab02104","line":973,"range":{"start_line":973,"start_character":46,"end_line":973,"end_character":47},"in_reply_to":"698781c8_7cba108f","updated":"2022-02-18 01:48:31.000000000","message":"I added it in PS 2.","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"543c2927e1b73d60e74145cc096059f238cbb41d","unresolved":false,"context_lines":[{"line_number":970,"context_line":"    def change_vnfpkg_vnf_instance(self, vnf_id, body):"},{"line_number":971,"context_line":"        if self.vnf_instance_path.split(\u0027/\u0027)[2] \u003d\u003d \u0027v2\u0027:"},{"line_number":972,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":973,"context_line":"                             vnf_id, body\u003dbody)"},{"line_number":974,"context_line":"        else:"},{"line_number":975,"context_line":"            raise exceptions.UnsupportedVersionForChangeVnfpkg()"},{"line_number":976,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"c33e2f69_df4f6515","line":973,"range":{"start_line":973,"start_character":46,"end_line":973,"end_character":47},"in_reply_to":"9c3bc5e9_5ab02104","updated":"2022-02-18 02:28:12.000000000","message":"Done","commit_id":"b4e6bb986e1f49dc3c4ea7e435c301e8e7ff3616"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"3695be29ea46854b3f25261832b1d4cc2841cb38","unresolved":true,"context_lines":[{"line_number":973,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":974,"context_line":"                             vnf_id, body\u003dbody, headers\u003dself.headers)"},{"line_number":975,"context_line":"        else:"},{"line_number":976,"context_line":"            raise exceptions.UnsupportedCommandVersion(version\u003d\u00272.0.0\u0027)"},{"line_number":977,"context_line":""},{"line_number":978,"context_line":"    @APIParamsCall"},{"line_number":979,"context_line":"    def retry_vnf_instance(self, occ_id):"}],"source_content_type":"text/x-python","patch_set":3,"id":"4293e0aa_cbd61135","line":976,"range":{"start_line":976,"start_character":64,"end_line":976,"end_character":69},"updated":"2022-02-18 09:17:03.000000000","message":"I think \"1\" is correct.\nAlso, I don\u0027t think you have checked the minor version, so it is better to set it to \"1\".","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"},{"author":{"_account_id":33492,"name":"Yi Feng","email":"fengyi@fujitsu.com","username":"YiFeng"},"change_message_id":"38b0e76d03e0a5c8100262a0fce9090034820271","unresolved":true,"context_lines":[{"line_number":973,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":974,"context_line":"                             vnf_id, body\u003dbody, headers\u003dself.headers)"},{"line_number":975,"context_line":"        else:"},{"line_number":976,"context_line":"            raise exceptions.UnsupportedCommandVersion(version\u003d\u00272.0.0\u0027)"},{"line_number":977,"context_line":""},{"line_number":978,"context_line":"    @APIParamsCall"},{"line_number":979,"context_line":"    def retry_vnf_instance(self, occ_id):"}],"source_content_type":"text/x-python","patch_set":3,"id":"f1c6b010_a209835d","line":976,"range":{"start_line":976,"start_character":64,"end_line":976,"end_character":69},"in_reply_to":"4293e0aa_cbd61135","updated":"2022-02-21 03:15:40.000000000","message":"I fixed it in PS4.","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"722a4959410d38e159d4089d38933887b1c9714c","unresolved":false,"context_lines":[{"line_number":973,"context_line":"            return self.post((self.vnf_instance_path + \"/change_vnfpkg\") %"},{"line_number":974,"context_line":"                             vnf_id, body\u003dbody, headers\u003dself.headers)"},{"line_number":975,"context_line":"        else:"},{"line_number":976,"context_line":"            raise exceptions.UnsupportedCommandVersion(version\u003d\u00272.0.0\u0027)"},{"line_number":977,"context_line":""},{"line_number":978,"context_line":"    @APIParamsCall"},{"line_number":979,"context_line":"    def retry_vnf_instance(self, occ_id):"}],"source_content_type":"text/x-python","patch_set":3,"id":"10347b21_2ed253d4","line":976,"range":{"start_line":976,"start_character":64,"end_line":976,"end_character":69},"in_reply_to":"f1c6b010_a209835d","updated":"2022-02-21 04:27:40.000000000","message":"Done","commit_id":"2875c0f293fd573bbe9467eb194dd3d4837825e7"}]}