)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"b5d1e411464a6f490578968dad43c734bdee7ca8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"80d94916_e9bc868c","updated":"2022-02-16 22:57:45.000000000","message":"Thank you for the review.","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"6e36c2eabcd27fc8f046d06fd58478e504916a0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"91a28c7e_5e266ba0","updated":"2022-02-18 02:49:28.000000000","message":"Thank you for the review.","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"dd7129cd0ba6c56dece8052daef647144b42a7d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"226dd67e_aee442c2","updated":"2022-02-21 08:42:35.000000000","message":"I\u0027m sorry but I\u0027d like to correct my comment, could you confirm it?","commit_id":"811ebb364787cd417385b986b2b94bfa502b29c8"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"449a99888cb858cc57d44dd859860c9549a75b6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"67b9a787_c46e12c3","updated":"2022-02-18 05:10:19.000000000","message":"Thanks for your rework. LGTM","commit_id":"811ebb364787cd417385b986b2b94bfa502b29c8"},{"author":{"_account_id":32581,"name":"Hirofumi Noguchi","email":"hirofumi.noguchi.rs@hco.ntt.co.jp","username":"hirofumi-noguchi"},"change_message_id":"b538d596b5a74c8bafdd51fd0196ab59b33d1be4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b0eb70bf_9317b56d","updated":"2022-02-22 00:14:27.000000000","message":"LGTM","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"0381c07beeb93cae7bd37eab161ce1ab4c4004ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7dd05288_6552e9f5","updated":"2022-02-22 03:03:14.000000000","message":"Thank you for the review.","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"43dbba4ccb4c0caf92f607b33d583472fc731d6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b1d9b31b_bd19a392","updated":"2022-02-21 23:56:22.000000000","message":"Thank you!","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"08f055a169dc9993b61574391fe89293b7d5ac13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f1bc887a_e7a65d50","updated":"2022-02-22 00:53:50.000000000","message":"Thanks for your patch, please kindly find my comment.","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e3aee9bba01b119ba5401f8f3eea374d8eb8ce5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"21be3576_3bf8dedc","updated":"2022-02-23 23:59:37.000000000","message":"Thanks for your rework, LGTM.","commit_id":"3724240344d3ea492bf9bafc375d6f73cdcc5822"}],"setup.cfg":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"7580454285da7746a879383ab01f6d79058c28e7","unresolved":true,"context_lines":[{"line_number":114,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":115,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":116,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":117,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":118,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"},{"line_number":119,"context_line":"     vnflcm_op_fail \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:FailVnfLcmOp"},{"line_number":120,"context_line":"     vnflcm_op_retry \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RetryVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"4d71f68c_edac33e6","line":117,"range":{"start_line":117,"start_character":5,"end_line":117,"end_character":28},"updated":"2022-02-18 01:47:53.000000000","message":"I think it follows V1 and names this variable, but can it be modified to \"vnflcm_change_ext_conn\"?\nIt\u0027s trivial, so I think you can fix the V1 side in this patch as well.","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"dd7129cd0ba6c56dece8052daef647144b42a7d4","unresolved":true,"context_lines":[{"line_number":114,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":115,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":116,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":117,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":118,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"},{"line_number":119,"context_line":"     vnflcm_op_fail \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:FailVnfLcmOp"},{"line_number":120,"context_line":"     vnflcm_op_retry \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RetryVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"575d6d0f_3abc0a93","line":117,"range":{"start_line":117,"start_character":5,"end_line":117,"end_character":28},"in_reply_to":"300509a0_7e35c593","updated":"2022-02-21 08:42:35.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-ext-conn\"?","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"6e36c2eabcd27fc8f046d06fd58478e504916a0b","unresolved":true,"context_lines":[{"line_number":114,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":115,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":116,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":117,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":118,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"},{"line_number":119,"context_line":"     vnflcm_op_fail \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:FailVnfLcmOp"},{"line_number":120,"context_line":"     vnflcm_op_retry \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RetryVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"d0abc3ac_8f21abfe","line":117,"range":{"start_line":117,"start_character":5,"end_line":117,"end_character":28},"in_reply_to":"4d71f68c_edac33e6","updated":"2022-02-18 02:49:28.000000000","message":"OK, I will change it and same as v1.","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"e13a16ffbc56cbe34c525f94470d83d226d83b61","unresolved":true,"context_lines":[{"line_number":114,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":115,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":116,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":117,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":118,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"},{"line_number":119,"context_line":"     vnflcm_op_fail \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:FailVnfLcmOp"},{"line_number":120,"context_line":"     vnflcm_op_retry \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RetryVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"ce0ca33a_394314eb","line":117,"range":{"start_line":117,"start_character":5,"end_line":117,"end_character":28},"in_reply_to":"575d6d0f_3abc0a93","updated":"2022-02-21 22:42:22.000000000","message":"Oops, I intended to check CLI did not have any problem, but seemed to be wrong. I will change back.","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"43dbba4ccb4c0caf92f607b33d583472fc731d6f","unresolved":false,"context_lines":[{"line_number":114,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":115,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":116,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":117,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":118,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"},{"line_number":119,"context_line":"     vnflcm_op_fail \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:FailVnfLcmOp"},{"line_number":120,"context_line":"     vnflcm_op_retry \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RetryVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"818bf7ec_50d83938","line":117,"range":{"start_line":117,"start_character":5,"end_line":117,"end_character":28},"in_reply_to":"ce0ca33a_394314eb","updated":"2022-02-21 23:56:22.000000000","message":"Done","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"449a99888cb858cc57d44dd859860c9549a75b6d","unresolved":false,"context_lines":[{"line_number":114,"context_line":"     vnflcm_heal \u003d tackerclient.osc.v1.vnflcm.vnflcm:HealVnfLcm"},{"line_number":115,"context_line":"     vnflcm_update \u003d tackerclient.osc.v1.vnflcm.vnflcm:UpdateVnfLcm"},{"line_number":116,"context_line":"     vnflcm_scale \u003d tackerclient.osc.v1.vnflcm.vnflcm:ScaleVnfLcm"},{"line_number":117,"context_line":"     vnflcm_change-ext-conn \u003d tackerclient.osc.v1.vnflcm.vnflcm:ChangeExtConnVnfLcm"},{"line_number":118,"context_line":"     vnflcm_op_rollback \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RollbackVnfLcmOp"},{"line_number":119,"context_line":"     vnflcm_op_fail \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:FailVnfLcmOp"},{"line_number":120,"context_line":"     vnflcm_op_retry \u003d tackerclient.osc.v1.vnflcm.vnflcm_op_occs:RetryVnfLcmOp"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"300509a0_7e35c593","line":117,"range":{"start_line":117,"start_character":5,"end_line":117,"end_character":28},"in_reply_to":"d0abc3ac_8f21abfe","updated":"2022-02-18 05:10:19.000000000","message":"Done","commit_id":"d9fd5fd9744575966aa91f01721cbbc7f7c9eb43"}],"tackerclient/osc/v1/vnflcm/vnflcm.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"87e835ba2af4fd3bef29fa5a948ab8d244b5a023","unresolved":true,"context_lines":[{"line_number":255,"context_line":"            \u0027--additional-param-file\u0027,"},{"line_number":256,"context_line":"            metavar\u003d\"\u003cadditional-param-file\u003e\","},{"line_number":257,"context_line":"            help\u003d_(\"Additional parameters passed by the NFVO as input \""},{"line_number":258,"context_line":"                   \"to the heal process.\"))"},{"line_number":259,"context_line":"        return parser"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a918c69_2503a4c6","line":258,"range":{"start_line":258,"start_character":27,"end_line":258,"end_character":31},"updated":"2022-02-16 05:42:57.000000000","message":"\"healing\" may be better.","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"b5d1e411464a6f490578968dad43c734bdee7ca8","unresolved":true,"context_lines":[{"line_number":255,"context_line":"            \u0027--additional-param-file\u0027,"},{"line_number":256,"context_line":"            metavar\u003d\"\u003cadditional-param-file\u003e\","},{"line_number":257,"context_line":"            help\u003d_(\"Additional parameters passed by the NFVO as input \""},{"line_number":258,"context_line":"                   \"to the heal process.\"))"},{"line_number":259,"context_line":"        return parser"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"4043ce87_747b1944","line":258,"range":{"start_line":258,"start_character":27,"end_line":258,"end_character":31},"in_reply_to":"3a918c69_2503a4c6","updated":"2022-02-16 22:57:45.000000000","message":"OK, I will fix.","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"7580454285da7746a879383ab01f6d79058c28e7","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            \u0027--additional-param-file\u0027,"},{"line_number":256,"context_line":"            metavar\u003d\"\u003cadditional-param-file\u003e\","},{"line_number":257,"context_line":"            help\u003d_(\"Additional parameters passed by the NFVO as input \""},{"line_number":258,"context_line":"                   \"to the heal process.\"))"},{"line_number":259,"context_line":"        return parser"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    def args2body(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5581f54a_a01cc0b3","line":258,"range":{"start_line":258,"start_character":27,"end_line":258,"end_character":31},"in_reply_to":"4043ce87_747b1944","updated":"2022-02-18 01:47:53.000000000","message":"Done","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"08f055a169dc9993b61574391fe89293b7d5ac13","unresolved":true,"context_lines":[{"line_number":238,"context_line":""},{"line_number":239,"context_line":"    def get_parser(self, prog_name):"},{"line_number":240,"context_line":"        parser \u003d super(HealVnfLcm, self).get_parser(prog_name)"},{"line_number":241,"context_line":"        parser.add_argument("},{"line_number":242,"context_line":"            _VNF_INSTANCE,"},{"line_number":243,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":244,"context_line":"            help\u003d_(\"VNF instance ID to heal\"))"}],"source_content_type":"text/x-python","patch_set":4,"id":"1d491b99_851dbdbf","line":241,"updated":"2022-02-22 00:53:50.000000000","message":"The patch [1] that modify help message of heal command have been merged.\nHeal cli is a special case so usage message must be a fixed string.\n\nCould you rebase and add `additional-param-file` to `usage_message` [2]?\n\n[1] https://review.opendev.org/c/openstack/python-tackerclient/+/821819\n[2] https://review.opendev.org/c/openstack/python-tackerclient/+/821819/19/tackerclient/osc/v1/vnflcm/vnflcm.py#241","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"0381c07beeb93cae7bd37eab161ce1ab4c4004ed","unresolved":true,"context_lines":[{"line_number":238,"context_line":""},{"line_number":239,"context_line":"    def get_parser(self, prog_name):"},{"line_number":240,"context_line":"        parser \u003d super(HealVnfLcm, self).get_parser(prog_name)"},{"line_number":241,"context_line":"        parser.add_argument("},{"line_number":242,"context_line":"            _VNF_INSTANCE,"},{"line_number":243,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":244,"context_line":"            help\u003d_(\"VNF instance ID to heal\"))"}],"source_content_type":"text/x-python","patch_set":4,"id":"593d5687_3bb7c39f","line":241,"in_reply_to":"1d491b99_851dbdbf","updated":"2022-02-22 03:03:14.000000000","message":"I see. I will add.","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e3aee9bba01b119ba5401f8f3eea374d8eb8ce5d","unresolved":false,"context_lines":[{"line_number":238,"context_line":""},{"line_number":239,"context_line":"    def get_parser(self, prog_name):"},{"line_number":240,"context_line":"        parser \u003d super(HealVnfLcm, self).get_parser(prog_name)"},{"line_number":241,"context_line":"        parser.add_argument("},{"line_number":242,"context_line":"            _VNF_INSTANCE,"},{"line_number":243,"context_line":"            metavar\u003d\"\u003cvnf-instance\u003e\","},{"line_number":244,"context_line":"            help\u003d_(\"VNF instance ID to heal\"))"}],"source_content_type":"text/x-python","patch_set":4,"id":"8b5ae4ce_b5db58a4","line":241,"in_reply_to":"593d5687_3bb7c39f","updated":"2022-02-23 23:59:37.000000000","message":"I\u0027ve confirmed.","commit_id":"50dd2c769c3e28b5511ca27e5364e725263e5036"}],"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":"87e835ba2af4fd3bef29fa5a948ab8d244b5a023","unresolved":true,"context_lines":[{"line_number":30,"context_line":"from tackerclient.v1_0 import client as proxy_client"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"_heal_sample_param_file \u003d (\"./tackerclient/osc/v1/vnflcm/samples/\""},{"line_number":34,"context_line":"                           \"heal_vnf_instance_param_sample.json\")"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"0328682a_692663b6","line":33,"updated":"2022-02-16 05:42:57.000000000","message":"Since this is only used in the heal test, I think it\u0027s better to move it into the TestHealVnfLcm class.","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"b5d1e411464a6f490578968dad43c734bdee7ca8","unresolved":true,"context_lines":[{"line_number":30,"context_line":"from tackerclient.v1_0 import client as proxy_client"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"_heal_sample_param_file \u003d (\"./tackerclient/osc/v1/vnflcm/samples/\""},{"line_number":34,"context_line":"                           \"heal_vnf_instance_param_sample.json\")"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"158779d9_18a6ad0a","line":33,"in_reply_to":"0328682a_692663b6","updated":"2022-02-16 22:57:45.000000000","message":"I see. I will move it.","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"7580454285da7746a879383ab01f6d79058c28e7","unresolved":false,"context_lines":[{"line_number":30,"context_line":"from tackerclient.v1_0 import client as proxy_client"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"_heal_sample_param_file \u003d (\"./tackerclient/osc/v1/vnflcm/samples/\""},{"line_number":34,"context_line":"                           \"heal_vnf_instance_param_sample.json\")"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"168907a0_16397a16","line":33,"in_reply_to":"158779d9_18a6ad0a","updated":"2022-02-18 01:47:53.000000000","message":"Done","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"87e835ba2af4fd3bef29fa5a948ab8d244b5a023","unresolved":true,"context_lines":[{"line_number":360,"context_line":""},{"line_number":361,"context_line":"        self.assertRaises(exceptions.InvalidInput,"},{"line_number":362,"context_line":"                          self.heal_vnf_lcm.take_action, parsed_args)"},{"line_number":363,"context_line":""},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"@ddt.ddt"},{"line_number":366,"context_line":"class TestTerminateVnfLcm(TestVnfLcm):"}],"source_content_type":"text/x-python","patch_set":1,"id":"005012a4_d1407e03","line":363,"updated":"2022-02-16 05:42:57.000000000","message":"I think it\u0027s better to add the following assertion like the other exisiting tests(e.g. Scale), but what do you think?\n```\n        expected_msg \u003d (\"Invalid input: File %s does not exist \"\n                        \"or user does not have read privileges to it\")\n        self.assertEqual(expected_msg % sample_param_file, str(ex))\n```","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"b5d1e411464a6f490578968dad43c734bdee7ca8","unresolved":true,"context_lines":[{"line_number":360,"context_line":""},{"line_number":361,"context_line":"        self.assertRaises(exceptions.InvalidInput,"},{"line_number":362,"context_line":"                          self.heal_vnf_lcm.take_action, parsed_args)"},{"line_number":363,"context_line":""},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"@ddt.ddt"},{"line_number":366,"context_line":"class TestTerminateVnfLcm(TestVnfLcm):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5be068bc_08627622","line":363,"in_reply_to":"005012a4_d1407e03","updated":"2022-02-16 22:57:45.000000000","message":"I referenced to TestUpdateVnfLcm. But I see, it is better. I will add.","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"7580454285da7746a879383ab01f6d79058c28e7","unresolved":false,"context_lines":[{"line_number":360,"context_line":""},{"line_number":361,"context_line":"        self.assertRaises(exceptions.InvalidInput,"},{"line_number":362,"context_line":"                          self.heal_vnf_lcm.take_action, parsed_args)"},{"line_number":363,"context_line":""},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"@ddt.ddt"},{"line_number":366,"context_line":"class TestTerminateVnfLcm(TestVnfLcm):"}],"source_content_type":"text/x-python","patch_set":1,"id":"d9c6bb82_88268581","line":363,"in_reply_to":"5be068bc_08627622","updated":"2022-02-18 01:47:53.000000000","message":"Done","commit_id":"629d3dec4d228bc964507a501a2b1162ee1b0221"}]}