)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Add the Fault Management interfaces and CLI to support AutoHeal."},{"line_number":13,"context_line":"Add the Performance Management interfaces and CLI to support"},{"line_number":14,"context_line":"AutoScale. The Fault Management and Performance Management"},{"line_number":15,"context_line":"interfaces are based on ETSI NFV-SOL 002 v3.3.1 and ETSI NFV-SOL"},{"line_number":16,"context_line":"003 v3.3.1, which are Version \"2.0.0\" API of Tacker. Add the"},{"line_number":17,"context_line":"Prometheus Plugin that has a interface between tacker and the"},{"line_number":18,"context_line":"External Monitoring Tool."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"777f3c86_81878034","line":16,"range":{"start_line":14,"start_character":11,"end_line":16,"end_character":52},"updated":"2022-08-30 15:54:02.000000000","message":"Just a question,\nthis describe seems to write \"FM and PM interface is Tacker v2 API\", but FM functions is implemented under osc/v1 folder.\nI\u0027m sorry for less understanding about python-tackerclient architecture, but why did you implement FM functions under the osc/v1 folder instead of the osc/v2 folder?","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Add the Fault Management interfaces and CLI to support AutoHeal."},{"line_number":13,"context_line":"Add the Performance Management interfaces and CLI to support"},{"line_number":14,"context_line":"AutoScale. The Fault Management and Performance Management"},{"line_number":15,"context_line":"interfaces are based on ETSI NFV-SOL 002 v3.3.1 and ETSI NFV-SOL"},{"line_number":16,"context_line":"003 v3.3.1, which are Version \"2.0.0\" API of Tacker. Add the"},{"line_number":17,"context_line":"Prometheus Plugin that has a interface between tacker and the"},{"line_number":18,"context_line":"External Monitoring Tool."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a04214f_f4452b29","line":16,"range":{"start_line":14,"start_character":11,"end_line":16,"end_character":52},"in_reply_to":"51a703af_c59e6fb2","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Add the Fault Management interfaces and CLI to support AutoHeal."},{"line_number":13,"context_line":"Add the Performance Management interfaces and CLI to support"},{"line_number":14,"context_line":"AutoScale. The Fault Management and Performance Management"},{"line_number":15,"context_line":"interfaces are based on ETSI NFV-SOL 002 v3.3.1 and ETSI NFV-SOL"},{"line_number":16,"context_line":"003 v3.3.1, which are Version \"2.0.0\" API of Tacker. Add the"},{"line_number":17,"context_line":"Prometheus Plugin that has a interface between tacker and the"},{"line_number":18,"context_line":"External Monitoring Tool."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"51a703af_c59e6fb2","line":16,"range":{"start_line":14,"start_character":11,"end_line":16,"end_character":52},"in_reply_to":"777f3c86_81878034","updated":"2022-09-02 01:18:06.000000000","message":"It\u0027s confusing that ETSI FM API has apiMajorVersion of \u0027v1\u0027,\nbut as we discussed, we use osc/v2 folder for FM functions.\nFixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3a162189_630d2199","updated":"2022-08-30 15:54:02.000000000","message":"Thanks for your patch, please kindly find my comments.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"1b4424922c734768b26d6c6e1394f552616c8d15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ac7e7dc7_de39e831","updated":"2022-09-01 12:03:17.000000000","message":"recheck","commit_id":"3a428d4a49447f4bf120ae81d7a2147fa6fb0e36"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"88d1b4ba6f45a52cc6d0b1e344bc29139cef67bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4770c16a_e9013580","updated":"2022-09-12 01:52:24.000000000","message":"Thank you for the review.","commit_id":"64d7a876703b98f6fb83d413a8c5c801a5e752d4"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"90c2e9cb1d9c91441a39276882e5303ca503a5fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"257dbd30_e3306782","updated":"2022-09-02 01:29:42.000000000","message":"Thanks for making a quick response!","commit_id":"64d7a876703b98f6fb83d413a8c5c801a5e752d4"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5f2bfb51_6e6fddef","updated":"2022-09-02 01:22:21.000000000","message":"Thanks for your rework, LGTM.","commit_id":"64d7a876703b98f6fb83d413a8c5c801a5e752d4"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"47133c5b3d208d3e331b4636f2f055803396cb15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"dca0bd99_d187d7c5","updated":"2022-09-02 01:06:49.000000000","message":"any update?","commit_id":"64d7a876703b98f6fb83d413a8c5c801a5e752d4"}],"tackerclient/osc/v1/vnffm/vnffm_alarm.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":96,"context_line":"        parser.add_argument("},{"line_number":97,"context_line":"            \"--filter\","},{"line_number":98,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":99,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":100,"context_line":"        )"},{"line_number":101,"context_line":"        return parser"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"c8d12403_118fbf2f","line":99,"range":{"start_line":99,"start_character":20,"end_line":99,"end_character":29},"updated":"2022-08-30 15:54:02.000000000","message":"typo: Attribute","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        parser.add_argument("},{"line_number":97,"context_line":"            \"--filter\","},{"line_number":98,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":99,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":100,"context_line":"        )"},{"line_number":101,"context_line":"        return parser"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f7d45665_d266efec","line":99,"range":{"start_line":99,"start_character":20,"end_line":99,"end_character":29},"in_reply_to":"3ad2b56b_6498699d","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":96,"context_line":"        parser.add_argument("},{"line_number":97,"context_line":"            \"--filter\","},{"line_number":98,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":99,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":100,"context_line":"        )"},{"line_number":101,"context_line":"        return parser"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3ad2b56b_6498699d","line":99,"range":{"start_line":99,"start_character":20,"end_line":99,"end_character":29},"in_reply_to":"c8d12403_118fbf2f","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"tackerclient/osc/v1/vnffm/vnffm_sub.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":111,"context_line":"        parser.add_argument("},{"line_number":112,"context_line":"            \"--filter\","},{"line_number":113,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":114,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":115,"context_line":"        )"},{"line_number":116,"context_line":"        return parser"},{"line_number":117,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"431df9a7_274146e1","line":114,"range":{"start_line":114,"start_character":20,"end_line":114,"end_character":29},"updated":"2022-08-30 15:54:02.000000000","message":"typo: Attribute","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        parser.add_argument("},{"line_number":112,"context_line":"            \"--filter\","},{"line_number":113,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":114,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":115,"context_line":"        )"},{"line_number":116,"context_line":"        return parser"},{"line_number":117,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9d0798df_f711be75","line":114,"range":{"start_line":114,"start_character":20,"end_line":114,"end_character":29},"in_reply_to":"39acb710_13b38be2","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":111,"context_line":"        parser.add_argument("},{"line_number":112,"context_line":"            \"--filter\","},{"line_number":113,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":114,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":115,"context_line":"        )"},{"line_number":116,"context_line":"        return parser"},{"line_number":117,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"39acb710_13b38be2","line":114,"range":{"start_line":114,"start_character":20,"end_line":114,"end_character":29},"in_reply_to":"431df9a7_274146e1","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":185,"context_line":"                     \"VNF FM subscriptions.\") % {\u0027error_count\u0027: error_count,"},{"line_number":186,"context_line":"                                                 \u0027total\u0027: total})"},{"line_number":187,"context_line":"            raise exceptions.CommandError(message\u003dmsg)"},{"line_number":188,"context_line":"        else:"},{"line_number":189,"context_line":"            if total \u003e 1:"},{"line_number":190,"context_line":"                print(_(\u0027All specified VNF FM subscriptions are deleted \u0027"},{"line_number":191,"context_line":"                        \u0027successfully\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"b1e345f6_e0c42edd","line":188,"range":{"start_line":188,"start_character":8,"end_line":188,"end_character":13},"updated":"2022-08-30 15:54:02.000000000","message":"This is unnecessary `else` after raise exceptions.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":185,"context_line":"                     \"VNF FM subscriptions.\") % {\u0027error_count\u0027: error_count,"},{"line_number":186,"context_line":"                                                 \u0027total\u0027: total})"},{"line_number":187,"context_line":"            raise exceptions.CommandError(message\u003dmsg)"},{"line_number":188,"context_line":"        else:"},{"line_number":189,"context_line":"            if total \u003e 1:"},{"line_number":190,"context_line":"                print(_(\u0027All specified VNF FM subscriptions are deleted \u0027"},{"line_number":191,"context_line":"                        \u0027successfully\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"7596da80_92f8522f","line":188,"range":{"start_line":188,"start_character":8,"end_line":188,"end_character":13},"in_reply_to":"017a550a_baa435fe","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":185,"context_line":"                     \"VNF FM subscriptions.\") % {\u0027error_count\u0027: error_count,"},{"line_number":186,"context_line":"                                                 \u0027total\u0027: total})"},{"line_number":187,"context_line":"            raise exceptions.CommandError(message\u003dmsg)"},{"line_number":188,"context_line":"        else:"},{"line_number":189,"context_line":"            if total \u003e 1:"},{"line_number":190,"context_line":"                print(_(\u0027All specified VNF FM subscriptions are deleted \u0027"},{"line_number":191,"context_line":"                        \u0027successfully\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"017a550a_baa435fe","line":188,"range":{"start_line":188,"start_character":8,"end_line":188,"end_character":13},"in_reply_to":"b1e345f6_e0c42edd","updated":"2022-09-02 01:18:06.000000000","message":"Fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"def jsonfile2body(file_path):"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    if file_path is not None and os.access(file_path, os.R_OK) is False:"},{"line_number":62,"context_line":"        msg \u003d _(\"File %s does not exist or user does not have read \""},{"line_number":63,"context_line":"                \"privileges to it\")"},{"line_number":64,"context_line":"        reason \u003d msg % file_path"},{"line_number":65,"context_line":"        raise exceptions.InvalidInput(reason\u003dreason)"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"310b857c_4bbf4072","line":63,"range":{"start_line":61,"start_character":4,"end_line":63,"end_character":35},"updated":"2022-08-31 07:56:05.000000000","message":"Why don\u0027t you separate the cases into each ones?","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"def jsonfile2body(file_path):"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    if file_path is not None and os.access(file_path, os.R_OK) is False:"},{"line_number":62,"context_line":"        msg \u003d _(\"File %s does not exist or user does not have read \""},{"line_number":63,"context_line":"                \"privileges to it\")"},{"line_number":64,"context_line":"        reason \u003d msg % file_path"},{"line_number":65,"context_line":"        raise exceptions.InvalidInput(reason\u003dreason)"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"d1ae9f78_7ad06433","line":63,"range":{"start_line":61,"start_character":4,"end_line":63,"end_character":35},"in_reply_to":"310b857c_4bbf4072","updated":"2022-09-02 01:18:06.000000000","message":"fixed at pathset 6","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    if not body:"},{"line_number":76,"context_line":"        reason \u003d _(\u0027The parameter file is empty\u0027)"},{"line_number":77,"context_line":"        raise exceptions.InvalidInput(reason\u003dreason)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    return body"},{"line_number":80,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"23f389e2_f798e171","line":77,"range":{"start_line":77,"start_character":25,"end_line":77,"end_character":37},"updated":"2022-08-31 07:56:05.000000000","message":"It\u0027s strange for me because the input must be correct in this case. Is there any appropriate exception other than this one?","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    if not body:"},{"line_number":76,"context_line":"        reason \u003d _(\u0027The parameter file is empty\u0027)"},{"line_number":77,"context_line":"        raise exceptions.InvalidInput(reason\u003dreason)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    return body"},{"line_number":80,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"27606267_b2af3ee9","line":77,"range":{"start_line":77,"start_character":25,"end_line":77,"end_character":37},"in_reply_to":"23f389e2_f798e171","updated":"2022-09-02 01:18:06.000000000","message":"I think it\u0027s better to use EmptyInput instead of InvalidInput.","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"}],"tackerclient/osc/v2/vnfpm/vnfpm_job.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":133,"context_line":"        parser.add_argument("},{"line_number":134,"context_line":"            \"--filter\","},{"line_number":135,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":136,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":137,"context_line":"        )"},{"line_number":138,"context_line":"        fields_exclusive_group \u003d parser.add_mutually_exclusive_group("},{"line_number":139,"context_line":"            required\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":3,"id":"17550f6e_e873e13b","line":136,"range":{"start_line":136,"start_character":20,"end_line":136,"end_character":29},"updated":"2022-08-30 15:54:02.000000000","message":"typo: Attribute","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":133,"context_line":"        parser.add_argument("},{"line_number":134,"context_line":"            \"--filter\","},{"line_number":135,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":136,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":137,"context_line":"        )"},{"line_number":138,"context_line":"        fields_exclusive_group \u003d parser.add_mutually_exclusive_group("},{"line_number":139,"context_line":"            required\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":3,"id":"88aabb44_33acfc61","line":136,"range":{"start_line":136,"start_character":20,"end_line":136,"end_character":29},"in_reply_to":"17550f6e_e873e13b","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":133,"context_line":"        parser.add_argument("},{"line_number":134,"context_line":"            \"--filter\","},{"line_number":135,"context_line":"            metavar\u003d\"\u003cfilter\u003e\","},{"line_number":136,"context_line":"            help\u003d_(\"Atrribute-based-filtering parameters\"),"},{"line_number":137,"context_line":"        )"},{"line_number":138,"context_line":"        fields_exclusive_group \u003d parser.add_mutually_exclusive_group("},{"line_number":139,"context_line":"            required\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":3,"id":"31221a97_203f7335","line":136,"range":{"start_line":136,"start_character":20,"end_line":136,"end_character":29},"in_reply_to":"88aabb44_33acfc61","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":322,"context_line":"                     \"VNF PM jobs.\") % {\u0027error_count\u0027: error_count,"},{"line_number":323,"context_line":"                                        \u0027total\u0027: total})"},{"line_number":324,"context_line":"            raise exceptions.CommandError(message\u003dmsg)"},{"line_number":325,"context_line":"        else:"},{"line_number":326,"context_line":"            if total \u003e 1:"},{"line_number":327,"context_line":"                print(_(\u0027All specified VNF PM jobs are deleted \u0027"},{"line_number":328,"context_line":"                        \u0027successfully\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"03134bd7_d56209c5","line":325,"range":{"start_line":325,"start_character":8,"end_line":325,"end_character":13},"updated":"2022-08-30 15:54:02.000000000","message":"This is unnecessary `else` after raise exceptions.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":322,"context_line":"                     \"VNF PM jobs.\") % {\u0027error_count\u0027: error_count,"},{"line_number":323,"context_line":"                                        \u0027total\u0027: total})"},{"line_number":324,"context_line":"            raise exceptions.CommandError(message\u003dmsg)"},{"line_number":325,"context_line":"        else:"},{"line_number":326,"context_line":"            if total \u003e 1:"},{"line_number":327,"context_line":"                print(_(\u0027All specified VNF PM jobs are deleted \u0027"},{"line_number":328,"context_line":"                        \u0027successfully\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"93635b3d_2462b521","line":325,"range":{"start_line":325,"start_character":8,"end_line":325,"end_character":13},"in_reply_to":"03134bd7_d56209c5","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":322,"context_line":"                     \"VNF PM jobs.\") % {\u0027error_count\u0027: error_count,"},{"line_number":323,"context_line":"                                        \u0027total\u0027: total})"},{"line_number":324,"context_line":"            raise exceptions.CommandError(message\u003dmsg)"},{"line_number":325,"context_line":"        else:"},{"line_number":326,"context_line":"            if total \u003e 1:"},{"line_number":327,"context_line":"                print(_(\u0027All specified VNF PM jobs are deleted \u0027"},{"line_number":328,"context_line":"                        \u0027successfully\u0027))"}],"source_content_type":"text/x-python","patch_set":3,"id":"15f2dd3a_e5f433aa","line":325,"range":{"start_line":325,"start_character":8,"end_line":325,"end_character":13},"in_reply_to":"93635b3d_2462b521","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"tackerclient/tests/unit/osc/v1/test_vnffm_alarm.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        actual_columns, data \u003d self.list_vnf_fm_alarms.take_action(parsed_args)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":80,"context_line":"            vnffm_alarm._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"9489e5b4_c7e11bd4","line":79,"range":{"start_line":79,"start_character":8,"end_line":79,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"unused variable, could you replace to `_` if possible?","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        actual_columns, data \u003d self.list_vnf_fm_alarms.take_action(parsed_args)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":80,"context_line":"            vnffm_alarm._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"24bd6250_15c35cf4","line":79,"range":{"start_line":79,"start_character":8,"end_line":79,"end_character":15},"in_reply_to":"2be06d77_a39a5aa7","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        actual_columns, data \u003d self.list_vnf_fm_alarms.take_action(parsed_args)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":80,"context_line":"            vnffm_alarm._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"2be06d77_a39a5aa7","line":79,"range":{"start_line":79,"start_character":8,"end_line":79,"end_character":15},"in_reply_to":"9489e5b4_c7e11bd4","updated":"2022-09-02 01:18:06.000000000","message":"Fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        actual_columns, data \u003d self.list_vnf_fm_alarms.take_action(parsed_args)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":107,"context_line":"            vnffm_alarm._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"c70d8375_b7300dd9","line":106,"range":{"start_line":106,"start_character":8,"end_line":106,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        actual_columns, data \u003d self.list_vnf_fm_alarms.take_action(parsed_args)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":107,"context_line":"            vnffm_alarm._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"6db22871_aceb752b","line":106,"range":{"start_line":106,"start_character":8,"end_line":106,"end_character":15},"in_reply_to":"0eddb528_dbe3bfe1","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        actual_columns, data \u003d self.list_vnf_fm_alarms.take_action(parsed_args)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":107,"context_line":"            vnffm_alarm._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"0eddb528_dbe3bfe1","line":106,"range":{"start_line":106,"start_character":8,"end_line":106,"end_character":15},"in_reply_to":"c70d8375_b7300dd9","updated":"2022-09-02 01:18:06.000000000","message":"Fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":174,"context_line":"        self.requests_mock.register_uri("},{"line_number":175,"context_line":"            \u0027GET\u0027, url, headers\u003dself.header, json\u003dvnffm_alarm_obj)"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        columns, data \u003d (self.show_vnf_fm_alarm.take_action(parsed_args))"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        self.assertCountEqual(_get_columns_vnffm_alarm(action\u003d\u0027show\u0027),"},{"line_number":180,"context_line":"                              columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"319dd2e8_e1330a7b","line":177,"range":{"start_line":177,"start_character":17,"end_line":177,"end_character":21},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":174,"context_line":"        self.requests_mock.register_uri("},{"line_number":175,"context_line":"            \u0027GET\u0027, url, headers\u003dself.header, json\u003dvnffm_alarm_obj)"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        columns, data \u003d (self.show_vnf_fm_alarm.take_action(parsed_args))"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        self.assertCountEqual(_get_columns_vnffm_alarm(action\u003d\u0027show\u0027),"},{"line_number":180,"context_line":"                              columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"db0a1858_76151dc0","line":177,"range":{"start_line":177,"start_character":17,"end_line":177,"end_character":21},"in_reply_to":"319dd2e8_e1330a7b","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":174,"context_line":"        self.requests_mock.register_uri("},{"line_number":175,"context_line":"            \u0027GET\u0027, url, headers\u003dself.header, json\u003dvnffm_alarm_obj)"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        columns, data \u003d (self.show_vnf_fm_alarm.take_action(parsed_args))"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        self.assertCountEqual(_get_columns_vnffm_alarm(action\u003d\u0027show\u0027),"},{"line_number":180,"context_line":"                              columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"8e9f0279_ae490ebd","line":177,"range":{"start_line":177,"start_character":17,"end_line":177,"end_character":21},"in_reply_to":"db0a1858_76151dc0","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"        self.assertCountEqual(expected_columns, actual_columns)"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        headers, columns \u003d vnffm_alarm._get_columns("},{"line_number":258,"context_line":"            vnffm_alarm_obj, action\u003d\u0027update\u0027)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        expected_data \u003d vnffm_alarm_fakes.get_vnffm_alarm_data("}],"source_content_type":"text/x-python","patch_set":3,"id":"f79870a1_94d119f4","line":257,"range":{"start_line":257,"start_character":8,"end_line":257,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"        self.assertCountEqual(expected_columns, actual_columns)"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        headers, columns \u003d vnffm_alarm._get_columns("},{"line_number":258,"context_line":"            vnffm_alarm_obj, action\u003d\u0027update\u0027)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        expected_data \u003d vnffm_alarm_fakes.get_vnffm_alarm_data("}],"source_content_type":"text/x-python","patch_set":3,"id":"d40dbdce_81e86ea3","line":257,"range":{"start_line":257,"start_character":8,"end_line":257,"end_character":15},"in_reply_to":"beacde27_4fe2b66b","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":254,"context_line":""},{"line_number":255,"context_line":"        self.assertCountEqual(expected_columns, actual_columns)"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        headers, columns \u003d vnffm_alarm._get_columns("},{"line_number":258,"context_line":"            vnffm_alarm_obj, action\u003d\u0027update\u0027)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"        expected_data \u003d vnffm_alarm_fakes.get_vnffm_alarm_data("}],"source_content_type":"text/x-python","patch_set":3,"id":"beacde27_4fe2b66b","line":257,"range":{"start_line":257,"start_character":8,"end_line":257,"end_character":15},"in_reply_to":"f79870a1_94d119f4","updated":"2022-09-02 01:18:06.000000000","message":"Fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"tackerclient/tests/unit/osc/v1/test_vnffm_sub.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":83,"context_line":"        actual_columns, data \u003d ("},{"line_number":84,"context_line":"            self.create_vnf_fm_sub.take_action(parsed_args))"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        headers, attributes \u003d vnffm_sub._get_columns(json)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        self.assertCountEqual(_get_columns_vnffm_sub(\"create\"),"},{"line_number":89,"context_line":"                              actual_columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"e35ff244_be2d004a","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"unused variable, could you replace to `_` if possible?","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        actual_columns, data \u003d ("},{"line_number":84,"context_line":"            self.create_vnf_fm_sub.take_action(parsed_args))"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        headers, attributes \u003d vnffm_sub._get_columns(json)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        self.assertCountEqual(_get_columns_vnffm_sub(\"create\"),"},{"line_number":89,"context_line":"                              actual_columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"0b4f093a_d22cf3b6","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":15},"in_reply_to":"b169a9e0_68f4af35","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":83,"context_line":"        actual_columns, data \u003d ("},{"line_number":84,"context_line":"            self.create_vnf_fm_sub.take_action(parsed_args))"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        headers, attributes \u003d vnffm_sub._get_columns(json)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        self.assertCountEqual(_get_columns_vnffm_sub(\"create\"),"},{"line_number":89,"context_line":"                              actual_columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"b169a9e0_68f4af35","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":15},"in_reply_to":"e35ff244_be2d004a","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        actual_columns, data \u003d self.list_vnffm_sub.take_action(parsed_args)"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":112,"context_line":"            vnffm_sub._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"de2d3bf1_ea9c35e0","line":111,"range":{"start_line":111,"start_character":8,"end_line":111,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        actual_columns, data \u003d self.list_vnffm_sub.take_action(parsed_args)"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":112,"context_line":"            vnffm_sub._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"a71deb1e_8368138f","line":111,"range":{"start_line":111,"start_character":8,"end_line":111,"end_character":15},"in_reply_to":"0c2b4c3e_5499f1a9","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        actual_columns, data \u003d self.list_vnffm_sub.take_action(parsed_args)"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":112,"context_line":"            vnffm_sub._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"0c2b4c3e_5499f1a9","line":111,"range":{"start_line":111,"start_character":8,"end_line":111,"end_character":15},"in_reply_to":"de2d3bf1_ea9c35e0","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        actual_columns, data \u003d self.list_vnffm_sub.take_action(parsed_args)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":140,"context_line":"            vnffm_sub._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"f56d85aa_e3593078","line":139,"range":{"start_line":139,"start_character":8,"end_line":139,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        actual_columns, data \u003d self.list_vnffm_sub.take_action(parsed_args)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":140,"context_line":"            vnffm_sub._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"898e1003_35f7b535","line":139,"range":{"start_line":139,"start_character":8,"end_line":139,"end_character":15},"in_reply_to":"a4d15ea3_1012e891","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        actual_columns, data \u003d self.list_vnffm_sub.take_action(parsed_args)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":140,"context_line":"            vnffm_sub._ATTR_MAP, long_listing\u003dTrue)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        expected_data \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"a4d15ea3_1012e891","line":139,"range":{"start_line":139,"start_character":8,"end_line":139,"end_character":15},"in_reply_to":"f56d85aa_e3593078","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":209,"context_line":"        self.requests_mock.register_uri("},{"line_number":210,"context_line":"            \u0027GET\u0027, url, headers\u003dself.header, json\u003dvnffm_sub_obj)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"        columns, data \u003d (self.show_vnf_fm_subs.take_action(parsed_args))"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        self.assertCountEqual(_get_columns_vnffm_sub(\u0027show\u0027),"},{"line_number":215,"context_line":"                              columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"d04f68e4_5b565d39","line":212,"range":{"start_line":212,"start_character":17,"end_line":212,"end_character":21},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":209,"context_line":"        self.requests_mock.register_uri("},{"line_number":210,"context_line":"            \u0027GET\u0027, url, headers\u003dself.header, json\u003dvnffm_sub_obj)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"        columns, data \u003d (self.show_vnf_fm_subs.take_action(parsed_args))"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        self.assertCountEqual(_get_columns_vnffm_sub(\u0027show\u0027),"},{"line_number":215,"context_line":"                              columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ddf50d27_512ce222","line":212,"range":{"start_line":212,"start_character":17,"end_line":212,"end_character":21},"in_reply_to":"853414a5_3c0b64d5","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":209,"context_line":"        self.requests_mock.register_uri("},{"line_number":210,"context_line":"            \u0027GET\u0027, url, headers\u003dself.header, json\u003dvnffm_sub_obj)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"        columns, data \u003d (self.show_vnf_fm_subs.take_action(parsed_args))"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        self.assertCountEqual(_get_columns_vnffm_sub(\u0027show\u0027),"},{"line_number":215,"context_line":"                              columns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"853414a5_3c0b64d5","line":212,"range":{"start_line":212,"start_character":17,"end_line":212,"end_character":21},"in_reply_to":"d04f68e4_5b565d39","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"tackerclient/tests/unit/osc/v2/test_vnfpm_job.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_job(),"},{"line_number":92,"context_line":"                              actual_columns)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        headers, attributes \u003d vnfpm_job._get_columns(json)"},{"line_number":95,"context_line":"        expected_data \u003d vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":96,"context_line":"            json, columns\u003dattributes)"},{"line_number":97,"context_line":"        self.assertListItemsEqual(expected_data, data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"dc68f8c5_72241652","line":94,"range":{"start_line":94,"start_character":8,"end_line":94,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"unused variable, could you replace to `_` if possible?","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_job(),"},{"line_number":92,"context_line":"                              actual_columns)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        headers, attributes \u003d vnfpm_job._get_columns(json)"},{"line_number":95,"context_line":"        expected_data \u003d vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":96,"context_line":"            json, columns\u003dattributes)"},{"line_number":97,"context_line":"        self.assertListItemsEqual(expected_data, data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"dd298a2f_063acaf3","line":94,"range":{"start_line":94,"start_character":8,"end_line":94,"end_character":15},"in_reply_to":"991b27d3_205485e2","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_job(),"},{"line_number":92,"context_line":"                              actual_columns)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        headers, attributes \u003d vnfpm_job._get_columns(json)"},{"line_number":95,"context_line":"        expected_data \u003d vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":96,"context_line":"            json, columns\u003dattributes)"},{"line_number":97,"context_line":"        self.assertListItemsEqual(expected_data, data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"991b27d3_205485e2","line":94,"range":{"start_line":94,"start_character":8,"end_line":94,"end_character":15},"in_reply_to":"dc68f8c5_72241652","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":150,"context_line":"            json\u003dself._vnf_pm_jobs, headers\u003dself.header)"},{"line_number":151,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":152,"context_line":"        expected_data \u003d []"},{"line_number":153,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":154,"context_line":"            self.list_vnf_pm_jobs.get_attributes(), long_listing\u003dTrue)"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"        for vnf_pm_job_obj in self._vnf_pm_jobs[\u0027vnf_pm_jobs\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"beca6ec6_1e3e5e22","line":153,"range":{"start_line":153,"start_character":8,"end_line":153,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":150,"context_line":"            json\u003dself._vnf_pm_jobs, headers\u003dself.header)"},{"line_number":151,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":152,"context_line":"        expected_data \u003d []"},{"line_number":153,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":154,"context_line":"            self.list_vnf_pm_jobs.get_attributes(), long_listing\u003dTrue)"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"        for vnf_pm_job_obj in self._vnf_pm_jobs[\u0027vnf_pm_jobs\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"45124b21_56a254ef","line":153,"range":{"start_line":153,"start_character":8,"end_line":153,"end_character":15},"in_reply_to":"1cd0e46d_a076c6d8","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":150,"context_line":"            json\u003dself._vnf_pm_jobs, headers\u003dself.header)"},{"line_number":151,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":152,"context_line":"        expected_data \u003d []"},{"line_number":153,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":154,"context_line":"            self.list_vnf_pm_jobs.get_attributes(), long_listing\u003dTrue)"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"        for vnf_pm_job_obj in self._vnf_pm_jobs[\u0027vnf_pm_jobs\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1cd0e46d_a076c6d8","line":153,"range":{"start_line":153,"start_character":8,"end_line":153,"end_character":15},"in_reply_to":"beca6ec6_1e3e5e22","updated":"2022-09-02 01:18:06.000000000","message":"Fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":172,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":173,"context_line":"        expected_data \u003d []"},{"line_number":174,"context_line":"        kwargs \u003d {arg: True}"},{"line_number":175,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":176,"context_line":"            self.list_vnf_pm_jobs.get_attributes(**kwargs), long_listing\u003dTrue)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"        for vnf_pm_job_obj in vnf_pm_jobs[\u0027vnf_pm_jobs\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"f2e39cba_63bca641","line":175,"range":{"start_line":175,"start_character":8,"end_line":175,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":172,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":173,"context_line":"        expected_data \u003d []"},{"line_number":174,"context_line":"        kwargs \u003d {arg: True}"},{"line_number":175,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":176,"context_line":"            self.list_vnf_pm_jobs.get_attributes(**kwargs), long_listing\u003dTrue)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"        for vnf_pm_job_obj in vnf_pm_jobs[\u0027vnf_pm_jobs\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"929d25cb_d7a352af","line":175,"range":{"start_line":175,"start_character":8,"end_line":175,"end_character":15},"in_reply_to":"18aa07f3_3111fba5","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":172,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":173,"context_line":"        expected_data \u003d []"},{"line_number":174,"context_line":"        kwargs \u003d {arg: True}"},{"line_number":175,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":176,"context_line":"            self.list_vnf_pm_jobs.get_attributes(**kwargs), long_listing\u003dTrue)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"        for vnf_pm_job_obj in vnf_pm_jobs[\u0027vnf_pm_jobs\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"18aa07f3_3111fba5","line":175,"range":{"start_line":175,"start_character":8,"end_line":175,"end_character":15},"in_reply_to":"f2e39cba_63bca641","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":204,"context_line":"        expected_data \u003d []"},{"line_number":205,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":206,"context_line":"            self.list_vnf_pm_jobs.get_attributes("},{"line_number":207,"context_line":"                exclude_fields\u003d[\u0027objectInstanceIds\u0027, \u0027criteria\u0027]),"},{"line_number":208,"context_line":"            long_listing\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"393d9a4c_53d28b82","line":205,"range":{"start_line":205,"start_character":8,"end_line":205,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":204,"context_line":"        expected_data \u003d []"},{"line_number":205,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":206,"context_line":"            self.list_vnf_pm_jobs.get_attributes("},{"line_number":207,"context_line":"                exclude_fields\u003d[\u0027objectInstanceIds\u0027, \u0027criteria\u0027]),"},{"line_number":208,"context_line":"            long_listing\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3e11e2f8_eaf0d8de","line":205,"range":{"start_line":205,"start_character":8,"end_line":205,"end_character":15},"in_reply_to":"393d9a4c_53d28b82","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":204,"context_line":"        expected_data \u003d []"},{"line_number":205,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":206,"context_line":"            self.list_vnf_pm_jobs.get_attributes("},{"line_number":207,"context_line":"                exclude_fields\u003d[\u0027objectInstanceIds\u0027, \u0027criteria\u0027]),"},{"line_number":208,"context_line":"            long_listing\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"68a7e312_375a8497","line":205,"range":{"start_line":205,"start_character":8,"end_line":205,"end_character":15},"in_reply_to":"3e11e2f8_eaf0d8de","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":252,"context_line":""},{"line_number":253,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":254,"context_line":"        expected_data \u003d []"},{"line_number":255,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":256,"context_line":"            self.list_vnf_pm_jobs.get_attributes("},{"line_number":257,"context_line":"                extra_fields\u003d[\u0027subObjectInstanceIds\u0027, \u0027criteria\u0027],"},{"line_number":258,"context_line":"                exclude_default\u003dTrue),"}],"source_content_type":"text/x-python","patch_set":3,"id":"b939cb0c_b7bb990a","line":255,"range":{"start_line":255,"start_character":8,"end_line":255,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":252,"context_line":""},{"line_number":253,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":254,"context_line":"        expected_data \u003d []"},{"line_number":255,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":256,"context_line":"            self.list_vnf_pm_jobs.get_attributes("},{"line_number":257,"context_line":"                extra_fields\u003d[\u0027subObjectInstanceIds\u0027, \u0027criteria\u0027],"},{"line_number":258,"context_line":"                exclude_default\u003dTrue),"}],"source_content_type":"text/x-python","patch_set":3,"id":"0b3e7a7b_dc435f9c","line":255,"range":{"start_line":255,"start_character":8,"end_line":255,"end_character":15},"in_reply_to":"171ed939_7edcba7d","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":252,"context_line":""},{"line_number":253,"context_line":"        actual_columns, data \u003d self.list_vnf_pm_jobs.take_action(parsed_args)"},{"line_number":254,"context_line":"        expected_data \u003d []"},{"line_number":255,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":256,"context_line":"            self.list_vnf_pm_jobs.get_attributes("},{"line_number":257,"context_line":"                extra_fields\u003d[\u0027subObjectInstanceIds\u0027, \u0027criteria\u0027],"},{"line_number":258,"context_line":"                exclude_default\u003dTrue),"}],"source_content_type":"text/x-python","patch_set":3,"id":"171ed939_7edcba7d","line":255,"range":{"start_line":255,"start_character":8,"end_line":255,"end_character":15},"in_reply_to":"b939cb0c_b7bb990a","updated":"2022-09-02 01:18:06.000000000","message":"Fixed at patchset6.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_job(\u0027show\u0027), columns)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        headers, attributes \u003d vnfpm_job._get_columns(vnfpm_job_obj)"},{"line_number":300,"context_line":"        self.assertListItemsEqual("},{"line_number":301,"context_line":"            vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":302,"context_line":"                vnfpm_job_obj, columns\u003dattributes), data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"2d15a7ab_a4a18927","line":299,"range":{"start_line":299,"start_character":8,"end_line":299,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_job(\u0027show\u0027), columns)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        headers, attributes \u003d vnfpm_job._get_columns(vnfpm_job_obj)"},{"line_number":300,"context_line":"        self.assertListItemsEqual("},{"line_number":301,"context_line":"            vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":302,"context_line":"                vnfpm_job_obj, columns\u003dattributes), data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"531d2209_835e800a","line":299,"range":{"start_line":299,"start_character":8,"end_line":299,"end_character":15},"in_reply_to":"2d15a7ab_a4a18927","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_job(\u0027show\u0027), columns)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        headers, attributes \u003d vnfpm_job._get_columns(vnfpm_job_obj)"},{"line_number":300,"context_line":"        self.assertListItemsEqual("},{"line_number":301,"context_line":"            vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":302,"context_line":"                vnfpm_job_obj, columns\u003dattributes), data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"f8cc8177_5f380757","line":299,"range":{"start_line":299,"start_character":8,"end_line":299,"end_character":15},"in_reply_to":"531d2209_835e800a","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":372,"context_line":"        expected_columns \u003d _get_columns_vnfpm_job(action\u003d\u0027update\u0027)"},{"line_number":373,"context_line":"        self.assertCountEqual(expected_columns, actual_columns)"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        headers, columns \u003d vnfpm_job._get_columns("},{"line_number":376,"context_line":"            vnfpm_job_obj, action\u003d\u0027update\u0027)"},{"line_number":377,"context_line":"        expected_data \u003d vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":378,"context_line":"            vnfpm_job_obj, columns\u003dcolumns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3ec2663c_f7ea580c","line":375,"range":{"start_line":375,"start_character":8,"end_line":375,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"ditto.","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":372,"context_line":"        expected_columns \u003d _get_columns_vnfpm_job(action\u003d\u0027update\u0027)"},{"line_number":373,"context_line":"        self.assertCountEqual(expected_columns, actual_columns)"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        headers, columns \u003d vnfpm_job._get_columns("},{"line_number":376,"context_line":"            vnfpm_job_obj, action\u003d\u0027update\u0027)"},{"line_number":377,"context_line":"        expected_data \u003d vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":378,"context_line":"            vnfpm_job_obj, columns\u003dcolumns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a28f39a_b7c5717b","line":375,"range":{"start_line":375,"start_character":8,"end_line":375,"end_character":15},"in_reply_to":"3ec2663c_f7ea580c","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":372,"context_line":"        expected_columns \u003d _get_columns_vnfpm_job(action\u003d\u0027update\u0027)"},{"line_number":373,"context_line":"        self.assertCountEqual(expected_columns, actual_columns)"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        headers, columns \u003d vnfpm_job._get_columns("},{"line_number":376,"context_line":"            vnfpm_job_obj, action\u003d\u0027update\u0027)"},{"line_number":377,"context_line":"        expected_data \u003d vnfpm_job_fakes.get_vnfpm_job_data("},{"line_number":378,"context_line":"            vnfpm_job_obj, columns\u003dcolumns)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5c5f90b2_41459c76","line":375,"range":{"start_line":375,"start_character":8,"end_line":375,"end_character":15},"in_reply_to":"9a28f39a_b7c5717b","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"tackerclient/tests/unit/osc/v2/test_vnfpm_report.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"02a1f79a35023be07bd6c18a1897ff0c683b5a54","unresolved":true,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_report(), columns)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        headers, attributes \u003d vnfpm_report._get_columns(vnfpm_report_obj)"},{"line_number":79,"context_line":"        expected_data \u003d vnfpm_report_fakes.get_vnfpm_report_data("},{"line_number":80,"context_line":"            vnfpm_report_obj, columns\u003dattributes)"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"4742064d_38868b78","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":15},"updated":"2022-08-30 15:54:02.000000000","message":"unused variable, could you replace to `_` if possible?","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7f4d4f673a7715c8191b68a2b2bd1e0b36c0a454","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_report(), columns)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        headers, attributes \u003d vnfpm_report._get_columns(vnfpm_report_obj)"},{"line_number":79,"context_line":"        expected_data \u003d vnfpm_report_fakes.get_vnfpm_report_data("},{"line_number":80,"context_line":"            vnfpm_report_obj, columns\u003dattributes)"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f5718b85_44cc262e","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":15},"in_reply_to":"21b155eb_961c93cb","updated":"2022-09-02 01:22:21.000000000","message":"Ack","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        self.assertCountEqual(_get_columns_vnfpm_report(), columns)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        headers, attributes \u003d vnfpm_report._get_columns(vnfpm_report_obj)"},{"line_number":79,"context_line":"        expected_data \u003d vnfpm_report_fakes.get_vnfpm_report_data("},{"line_number":80,"context_line":"            vnfpm_report_obj, columns\u003dattributes)"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"21b155eb_961c93cb","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":15},"in_reply_to":"4742064d_38868b78","updated":"2022-09-02 01:18:06.000000000","message":"fixed at patchset6","commit_id":"53d537c2d63cf139297f7a065dc32ff851e6560a"}],"tackerclient/tests/unit/osc/v2/vnfpm_job_fakes.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":129,"context_line":"                    vnf_pm_job[attribute])})"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    # return the list of data as per column order"},{"line_number":132,"context_line":"    if columns:"},{"line_number":133,"context_line":"        return tuple([vnf_pm_job[key] for key in columns])"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return tuple([vnf_pm_job[key] for key in sorted("},{"line_number":136,"context_line":"        vnf_pm_job.keys())])"}],"source_content_type":"text/x-python","patch_set":4,"id":"f418ba36_a84a294f","line":136,"range":{"start_line":132,"start_character":4,"end_line":136,"end_character":28},"updated":"2022-08-31 07:56:05.000000000","message":"It\u0027s just a little bit redundant. Why don\u0027t you make it simple as below example?\n\n```\nif columns is None:\n    columns \u003d sorted(vnf_pm_report.keys())\nreturn tuple([vnf_pm_report[key] for key in columns])","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":129,"context_line":"                    vnf_pm_job[attribute])})"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    # return the list of data as per column order"},{"line_number":132,"context_line":"    if columns:"},{"line_number":133,"context_line":"        return tuple([vnf_pm_job[key] for key in columns])"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"    return tuple([vnf_pm_job[key] for key in sorted("},{"line_number":136,"context_line":"        vnf_pm_job.keys())])"}],"source_content_type":"text/x-python","patch_set":4,"id":"a3ef9c94_55005768","line":136,"range":{"start_line":132,"start_character":4,"end_line":136,"end_character":28},"in_reply_to":"f418ba36_a84a294f","updated":"2022-09-02 01:18:06.000000000","message":"adopted your idea.","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"}],"tackerclient/tests/unit/osc/v2/vnfpm_report_fakes.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":22,"context_line":"    :param Dictionary attrs:"},{"line_number":23,"context_line":"        A dictionary with all attributes"},{"line_number":24,"context_line":"    :return:"},{"line_number":25,"context_line":"        A FakeVnfPmJob dict"},{"line_number":26,"context_line":"    \"\"\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    attrs \u003d attrs or {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"61861153_73f92bdd","line":25,"range":{"start_line":25,"start_character":8,"end_line":25,"end_character":27},"updated":"2022-08-31 07:56:05.000000000","message":"It\u0027s so confusing because it returns just a value of dict and not a dict of FakeVnfPmJob.","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":22,"context_line":"    :param Dictionary attrs:"},{"line_number":23,"context_line":"        A dictionary with all attributes"},{"line_number":24,"context_line":"    :return:"},{"line_number":25,"context_line":"        A FakeVnfPmJob dict"},{"line_number":26,"context_line":"    \"\"\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    attrs \u003d attrs or {}"}],"source_content_type":"text/x-python","patch_set":4,"id":"d6067102_7a3ee096","line":25,"range":{"start_line":25,"start_character":8,"end_line":25,"end_character":27},"in_reply_to":"61861153_73f92bdd","updated":"2022-09-02 01:18:06.000000000","message":"corrected to \"A pm report dict\"","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":60,"context_line":"    :return:"},{"line_number":61,"context_line":"        A tuple object sorted based on the name of the columns."},{"line_number":62,"context_line":"    \"\"\""},{"line_number":63,"context_line":"    complex_attributes \u003d [\u0027entries\u0027]"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    for attribute in complex_attributes:"},{"line_number":66,"context_line":"        if vnf_pm_report.get(attribute):"},{"line_number":67,"context_line":"            vnf_pm_report.update("},{"line_number":68,"context_line":"                {attribute: tacker_osc_utils.FormatComplexDataColumn("}],"source_content_type":"text/x-python","patch_set":4,"id":"f1ceec83_2e6a1073","line":65,"range":{"start_line":63,"start_character":4,"end_line":65,"end_character":39},"updated":"2022-08-31 07:56:05.000000000","message":"Is there any reason to have `complex_attributes` which is only used this two line?","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":60,"context_line":"    :return:"},{"line_number":61,"context_line":"        A tuple object sorted based on the name of the columns."},{"line_number":62,"context_line":"    \"\"\""},{"line_number":63,"context_line":"    complex_attributes \u003d [\u0027entries\u0027]"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    for attribute in complex_attributes:"},{"line_number":66,"context_line":"        if vnf_pm_report.get(attribute):"},{"line_number":67,"context_line":"            vnf_pm_report.update("},{"line_number":68,"context_line":"                {attribute: tacker_osc_utils.FormatComplexDataColumn("}],"source_content_type":"text/x-python","patch_set":4,"id":"ef4a69f4_a862b0c1","line":65,"range":{"start_line":63,"start_character":4,"end_line":65,"end_character":39},"in_reply_to":"f1ceec83_2e6a1073","updated":"2022-09-02 01:18:06.000000000","message":"I think it was redundant. fixed at patchset6.","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                {attribute: tacker_osc_utils.FormatComplexDataColumn("},{"line_number":69,"context_line":"                    vnf_pm_report[attribute])})"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    # return the list of data as per column order"},{"line_number":72,"context_line":"    if columns:"},{"line_number":73,"context_line":"        return tuple([vnf_pm_report[key] for key in columns])"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    return tuple([vnf_pm_report[key] for key in sorted("},{"line_number":76,"context_line":"        vnf_pm_report.keys())])"}],"source_content_type":"text/x-python","patch_set":4,"id":"98f8e6c5_292fbe72","line":76,"range":{"start_line":71,"start_character":4,"end_line":76,"end_character":31},"updated":"2022-08-31 07:56:05.000000000","message":"It\u0027s just a little bit redundant. Why don\u0027t you make it simple as below example?\n\n```\nif columns is None:\n    columns \u003d sorted(vnf_pm_report.keys())\nreturn tuple([vnf_pm_report[key] for key in columns])\n```","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":68,"context_line":"                {attribute: tacker_osc_utils.FormatComplexDataColumn("},{"line_number":69,"context_line":"                    vnf_pm_report[attribute])})"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    # return the list of data as per column order"},{"line_number":72,"context_line":"    if columns:"},{"line_number":73,"context_line":"        return tuple([vnf_pm_report[key] for key in columns])"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    return tuple([vnf_pm_report[key] for key in sorted("},{"line_number":76,"context_line":"        vnf_pm_report.keys())])"}],"source_content_type":"text/x-python","patch_set":4,"id":"49f74e38_eef5bba2","line":76,"range":{"start_line":71,"start_character":4,"end_line":76,"end_character":31},"in_reply_to":"98f8e6c5_292fbe72","updated":"2022-09-02 01:18:06.000000000","message":"adopted your idea.","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"}],"tackerclient/v1_0/client.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":1034,"context_line":"class VnfFMClient(ClientBase):"},{"line_number":1035,"context_line":"    \"\"\"Client for vnffm APIs."},{"line_number":1036,"context_line":""},{"line_number":1037,"context_line":"    Purpose of this class is to create required request url for vnffm"},{"line_number":1038,"context_line":"    APIs."},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"    headers \u003d {\u0027Version\u0027: \u00271.3.0\u0027}"}],"source_content_type":"text/x-python","patch_set":4,"id":"6acf5b76_e54ec74b","line":1038,"range":{"start_line":1037,"start_character":4,"end_line":1038,"end_character":9},"updated":"2022-08-31 07:56:05.000000000","message":"Although I understand you\u0027ve just followed the previous examples, this comment might be no need because it doesn\u0027t tell us nothing other than the title line at L.1037.","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":1034,"context_line":"class VnfFMClient(ClientBase):"},{"line_number":1035,"context_line":"    \"\"\"Client for vnffm APIs."},{"line_number":1036,"context_line":""},{"line_number":1037,"context_line":"    Purpose of this class is to create required request url for vnffm"},{"line_number":1038,"context_line":"    APIs."},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"    headers \u003d {\u0027Version\u0027: \u00271.3.0\u0027}"}],"source_content_type":"text/x-python","patch_set":4,"id":"779a12ca_46f8f00d","line":1038,"range":{"start_line":1037,"start_character":4,"end_line":1038,"end_character":9},"in_reply_to":"6acf5b76_e54ec74b","updated":"2022-09-02 01:18:06.000000000","message":"fixed at pathset 6","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"af33caabd8bc408cf7c7013f281fe435941d14cb","unresolved":true,"context_lines":[{"line_number":1090,"context_line":"class VnfPMClient(ClientBase):"},{"line_number":1091,"context_line":"    \"\"\"Client for vnfpm APIs."},{"line_number":1092,"context_line":""},{"line_number":1093,"context_line":"    Purpose of this class is to create required request url for vnfpm"},{"line_number":1094,"context_line":"    APIs."},{"line_number":1095,"context_line":"    \"\"\""},{"line_number":1096,"context_line":"    headers \u003d {\u0027Version\u0027: \u00272.1.0\u0027}"},{"line_number":1097,"context_line":"    vnf_pm_jobs_path \u003d \u0027/vnfpm/v2/pm_jobs\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"732235e9_ed918dd3","line":1094,"range":{"start_line":1093,"start_character":4,"end_line":1094,"end_character":9},"updated":"2022-08-31 07:56:05.000000000","message":"ditto","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"},{"author":{"_account_id":35016,"name":"Koji Shimizu","email":"shimizu.koji@fujitsu.com","username":"shimizu-koji"},"change_message_id":"8ff93600090d5e7c6f236f7493ff71fca907f11f","unresolved":true,"context_lines":[{"line_number":1090,"context_line":"class VnfPMClient(ClientBase):"},{"line_number":1091,"context_line":"    \"\"\"Client for vnfpm APIs."},{"line_number":1092,"context_line":""},{"line_number":1093,"context_line":"    Purpose of this class is to create required request url for vnfpm"},{"line_number":1094,"context_line":"    APIs."},{"line_number":1095,"context_line":"    \"\"\""},{"line_number":1096,"context_line":"    headers \u003d {\u0027Version\u0027: \u00272.1.0\u0027}"},{"line_number":1097,"context_line":"    vnf_pm_jobs_path \u003d \u0027/vnfpm/v2/pm_jobs\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"02d90a27_062d441c","line":1094,"range":{"start_line":1093,"start_character":4,"end_line":1094,"end_character":9},"in_reply_to":"732235e9_ed918dd3","updated":"2022-09-02 01:18:06.000000000","message":"fixed at pathset 6","commit_id":"a12d5ace2a9594bcf853114f9ad59a59f862a0fe"}]}
