)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"72791f83b749e0c58bf04410f4e9815555893812","unresolved":true,"context_lines":[{"line_number":13,"context_line":"\u0027/vnflcm/v2\u0027 is used instead of \u0027/vnflcm/v1\u0027."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"New CLI \u0027openstack vnflcm versions\u0027 is added too."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I256a4010043d0b84ffe43b055c6a6a67a2d5d661"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"2c0d0ea3_6dd6d1be","line":16,"updated":"2021-08-26 01:36:30.000000000","message":"Could you add \"Implements:\" as well as https://review.opendev.org/c/openstack/tacker/+/803693.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"08de60e9953494a1ba36abe7d42ac011768eb55d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"\u0027/vnflcm/v2\u0027 is used instead of \u0027/vnflcm/v1\u0027."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"New CLI \u0027openstack vnflcm versions\u0027 is added too."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I256a4010043d0b84ffe43b055c6a6a67a2d5d661"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"e236c9f5_d46a9487","line":16,"in_reply_to":"1e047be7_d3467b64","updated":"2021-08-26 05:58:57.000000000","message":"Done","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7343c12200269e256d812dece9035f70cb9d5944","unresolved":true,"context_lines":[{"line_number":13,"context_line":"\u0027/vnflcm/v2\u0027 is used instead of \u0027/vnflcm/v1\u0027."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"New CLI \u0027openstack vnflcm versions\u0027 is added too."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I256a4010043d0b84ffe43b055c6a6a67a2d5d661"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1e047be7_d3467b64","line":16,"in_reply_to":"2c0d0ea3_6dd6d1be","updated":"2021-08-26 04:32:20.000000000","message":"I see. I will add it.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"6cf92a2d29136630b9a936f78244e0c8c753578e","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Add version \u00272\u0027 for support version of --os-tacker-api-version to"},{"line_number":10,"context_line":"\u0027openstack vnflcm\u0027 CLI."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"If \u0027--os-tacker-api-version 2\u0027 is specicied, REST API url"},{"line_number":13,"context_line":"\u0027/vnflcm/v2\u0027 is used instead of \u0027/vnflcm/v1\u0027."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"New CLI \u0027openstack vnflcm versions\u0027 is added too."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"619da80c_45f0a7c8","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":43},"updated":"2021-08-31 09:41:15.000000000","message":"nit: Misspelled here, \"specified\" is correct.","commit_id":"087e621b5662ea917f7ecabbad0bbe92421fff4f"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"0384657d6f2ee4e0759cd35e7e4d982d8949aebd","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add version \u00272\u0027 for support version of --os-tacker-api-version to"},{"line_number":10,"context_line":"\u0027openstack vnflcm\u0027 CLI."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"If \u0027--os-tacker-api-version 2\u0027 is specicied, REST API url"},{"line_number":13,"context_line":"\u0027/vnflcm/v2\u0027 is used instead of \u0027/vnflcm/v1\u0027."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"New CLI \u0027openstack vnflcm versions\u0027 is added too."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"6728e1e7_2e9659b5","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":43},"in_reply_to":"554fe02d_1b4cfc98","updated":"2021-08-31 23:32:10.000000000","message":"I\u0027ve confirmed.","commit_id":"087e621b5662ea917f7ecabbad0bbe92421fff4f"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"0082d245394595284410bde39f4572a61154f4c8","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Add version \u00272\u0027 for support version of --os-tacker-api-version to"},{"line_number":10,"context_line":"\u0027openstack vnflcm\u0027 CLI."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"If \u0027--os-tacker-api-version 2\u0027 is specicied, REST API url"},{"line_number":13,"context_line":"\u0027/vnflcm/v2\u0027 is used instead of \u0027/vnflcm/v1\u0027."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"New CLI \u0027openstack vnflcm versions\u0027 is added too."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"554fe02d_1b4cfc98","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":43},"in_reply_to":"619da80c_45f0a7c8","updated":"2021-08-31 22:20:57.000000000","message":"oops, I will fix.","commit_id":"087e621b5662ea917f7ecabbad0bbe92421fff4f"}],"tackerclient/osc/v2/vnflcm/vnflcm_versions.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"72791f83b749e0c58bf04410f4e9815555893812","unresolved":true,"context_lines":[{"line_number":34,"context_line":"    def take_action(self, parsed_args):"},{"line_number":35,"context_line":"        v \u003d None"},{"line_number":36,"context_line":"        if parsed_args.major_version:"},{"line_number":37,"context_line":"            if parsed_args.major_version not in [1, 2]:"},{"line_number":38,"context_line":"                msg \u003d _(\"Major version %d is not supported\")"},{"line_number":39,"context_line":"                reason \u003d msg % parsed_args.major_version"},{"line_number":40,"context_line":"                raise exceptions.InvalidInput(reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f3ec717_e76562ae","line":37,"updated":"2021-08-26 01:36:30.000000000","message":"I think it will be clearer if you define the constants as follows.\n---\nSUPPORTED_VERSIONS \u003d [1, 2]","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"08de60e9953494a1ba36abe7d42ac011768eb55d","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    def take_action(self, parsed_args):"},{"line_number":35,"context_line":"        v \u003d None"},{"line_number":36,"context_line":"        if parsed_args.major_version:"},{"line_number":37,"context_line":"            if parsed_args.major_version not in [1, 2]:"},{"line_number":38,"context_line":"                msg \u003d _(\"Major version %d is not supported\")"},{"line_number":39,"context_line":"                reason \u003d msg % parsed_args.major_version"},{"line_number":40,"context_line":"                raise exceptions.InvalidInput(reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":4,"id":"10783755_66e309a6","line":37,"in_reply_to":"096932ce_482bd4df","updated":"2021-08-26 05:58:57.000000000","message":"Done","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7343c12200269e256d812dece9035f70cb9d5944","unresolved":true,"context_lines":[{"line_number":34,"context_line":"    def take_action(self, parsed_args):"},{"line_number":35,"context_line":"        v \u003d None"},{"line_number":36,"context_line":"        if parsed_args.major_version:"},{"line_number":37,"context_line":"            if parsed_args.major_version not in [1, 2]:"},{"line_number":38,"context_line":"                msg \u003d _(\"Major version %d is not supported\")"},{"line_number":39,"context_line":"                reason \u003d msg % parsed_args.major_version"},{"line_number":40,"context_line":"                raise exceptions.InvalidInput(reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":4,"id":"096932ce_482bd4df","line":37,"in_reply_to":"7f3ec717_e76562ae","updated":"2021-08-26 04:32:20.000000000","message":"I see. I will fix.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"a1e1e21dc12b3bf6c50dd0fc46f5f3950da68a3b","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2021 Nippon Telegraph and Telephone Corporation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"80d13797_bcb092d7","line":1,"updated":"2021-08-27 02:56:35.000000000","message":"I think this file can also be moved by creating the following directories, but do you think it\u0027s worth it?\n---\ntackerclient/osc/common/vnflcm/vnflcm_versions.py","commit_id":"42303e585fce02c68dec505d088b93a46d742358"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f13114817971d1485bb3eae1f7b0085e26615bea","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2021 Nippon Telegraph and Telephone Corporation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"c1545f28_e5e484db","line":1,"in_reply_to":"80d13797_bcb092d7","updated":"2021-08-27 03:57:47.000000000","message":"Hmm, certainly this is not v2 specific. I will try to move in next patch.","commit_id":"42303e585fce02c68dec505d088b93a46d742358"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"9ae27d6f6b55ec13a8367baf43e2621dd1e8fe1c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2021 Nippon Telegraph and Telephone Corporation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"751a5593_c6e21c8a","line":1,"in_reply_to":"c1545f28_e5e484db","updated":"2021-08-27 04:13:53.000000000","message":"Thanks! I confirmed!","commit_id":"42303e585fce02c68dec505d088b93a46d742358"}],"tackerclient/tests/unit/osc/common/test_vnflcm_versions.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"9ae27d6f6b55ec13a8367baf43e2621dd1e8fe1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"287c87c9_c3d04a3e","line":48,"range":{"start_line":48,"start_character":33,"end_line":48,"end_character":46},"in_reply_to":"874c6e9e_1bc019d3","updated":"2021-08-27 04:13:53.000000000","message":"Done","commit_id":"42303e585fce02c68dec505d088b93a46d742358"}],"tackerclient/tests/unit/osc/v2/test_vnflcm.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"72791f83b749e0c58bf04410f4e9815555893812","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    def setUp(self):"},{"line_number":25,"context_line":"        super(TestVnfLcmV2, self).setUp()"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def test_client_v2(self):"},{"line_number":28,"context_line":"        self.assertEqual(self.cs.vnf_lcm_client.headers,"},{"line_number":29,"context_line":"                         {\u0027Version\u0027: \u00272.0.0\u0027})"},{"line_number":30,"context_line":"        self.assertEqual(self.cs.vnf_lcm_client.vnf_instances_path,"}],"source_content_type":"text/x-python","patch_set":4,"id":"afc0b55f_d98fc9d0","line":27,"range":{"start_line":27,"start_character":8,"end_line":27,"end_character":22},"updated":"2021-08-26 01:36:30.000000000","message":"Could you add a similar test to \"tests/unit/osc/v1/test_vnflcm.py\" for test consistency?","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"08de60e9953494a1ba36abe7d42ac011768eb55d","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    def setUp(self):"},{"line_number":25,"context_line":"        super(TestVnfLcmV2, self).setUp()"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def test_client_v2(self):"},{"line_number":28,"context_line":"        self.assertEqual(self.cs.vnf_lcm_client.headers,"},{"line_number":29,"context_line":"                         {\u0027Version\u0027: \u00272.0.0\u0027})"},{"line_number":30,"context_line":"        self.assertEqual(self.cs.vnf_lcm_client.vnf_instances_path,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5571354a_37eb57ec","line":27,"range":{"start_line":27,"start_character":8,"end_line":27,"end_character":22},"in_reply_to":"0b6a7c00_628ae463","updated":"2021-08-26 05:58:57.000000000","message":"Done","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7343c12200269e256d812dece9035f70cb9d5944","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    def setUp(self):"},{"line_number":25,"context_line":"        super(TestVnfLcmV2, self).setUp()"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    def test_client_v2(self):"},{"line_number":28,"context_line":"        self.assertEqual(self.cs.vnf_lcm_client.headers,"},{"line_number":29,"context_line":"                         {\u0027Version\u0027: \u00272.0.0\u0027})"},{"line_number":30,"context_line":"        self.assertEqual(self.cs.vnf_lcm_client.vnf_instances_path,"}],"source_content_type":"text/x-python","patch_set":4,"id":"0b6a7c00_628ae463","line":27,"range":{"start_line":27,"start_character":8,"end_line":27,"end_character":22},"in_reply_to":"afc0b55f_d98fc9d0","updated":"2021-08-26 04:32:20.000000000","message":"I see. I will add a test.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"}],"tackerclient/tests/unit/osc/v2/test_vnflcm_versions.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"a1e1e21dc12b3bf6c50dd0fc46f5f3950da68a3b","unresolved":true,"context_lines":[{"line_number":45,"context_line":"        self.vnflcm_versions \u003d vnflcm_versions.VnfLcmVersions("},{"line_number":46,"context_line":"            self.app, self.app_args, cmd_name\u003d\u0027vnflcm versions\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def _versions_response(self, major_version):"},{"line_number":49,"context_line":"        if major_version is None:"},{"line_number":50,"context_line":"            return {\"uriPrefix\": \"/vnflcm\","},{"line_number":51,"context_line":"                    \"apiVersions\": [{\"version\": \"1.3.0\","}],"source_content_type":"text/x-python","patch_set":5,"id":"b3e335f2_556446c9","line":48,"range":{"start_line":48,"start_character":33,"end_line":48,"end_character":46},"updated":"2021-08-27 02:56:35.000000000","message":"I think \"major_version\u003dNone\" is better because you can ommit the argument in L75.","commit_id":"42303e585fce02c68dec505d088b93a46d742358"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f13114817971d1485bb3eae1f7b0085e26615bea","unresolved":true,"context_lines":[{"line_number":45,"context_line":"        self.vnflcm_versions \u003d vnflcm_versions.VnfLcmVersions("},{"line_number":46,"context_line":"            self.app, self.app_args, cmd_name\u003d\u0027vnflcm versions\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def _versions_response(self, major_version):"},{"line_number":49,"context_line":"        if major_version is None:"},{"line_number":50,"context_line":"            return {\"uriPrefix\": \"/vnflcm\","},{"line_number":51,"context_line":"                    \"apiVersions\": [{\"version\": \"1.3.0\","}],"source_content_type":"text/x-python","patch_set":5,"id":"874c6e9e_1bc019d3","line":48,"range":{"start_line":48,"start_character":33,"end_line":48,"end_character":46},"in_reply_to":"b3e335f2_556446c9","updated":"2021-08-27 03:57:47.000000000","message":"I see. I will fix.","commit_id":"42303e585fce02c68dec505d088b93a46d742358"}],"tackerclient/v1_0/client.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"72791f83b749e0c58bf04410f4e9815555893812","unresolved":true,"context_lines":[{"line_number":219,"context_line":"        query \u003d query + urlparse.urlencode(keyval_encoded, doseq\u003d1)"},{"line_number":220,"context_line":"        return query.strip(\u0027\u0026\u0027)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def do_request(self, method, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":223,"context_line":"        action \u003d self.build_action(action)"},{"line_number":224,"context_line":"        # Add format and tenant_id"},{"line_number":225,"context_line":"        if type(params) is dict and params:"}],"source_content_type":"text/x-python","patch_set":4,"id":"191613c4_486a3c4e","line":222,"range":{"start_line":222,"start_character":60,"end_line":222,"end_character":62},"updated":"2021-08-26 01:36:30.000000000","message":"Just a question.\nIs there any reason why you changed the default value of \"headers\" from \"None\" to \"{}\"?","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7343c12200269e256d812dece9035f70cb9d5944","unresolved":true,"context_lines":[{"line_number":219,"context_line":"        query \u003d query + urlparse.urlencode(keyval_encoded, doseq\u003d1)"},{"line_number":220,"context_line":"        return query.strip(\u0027\u0026\u0027)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def do_request(self, method, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":223,"context_line":"        action \u003d self.build_action(action)"},{"line_number":224,"context_line":"        # Add format and tenant_id"},{"line_number":225,"context_line":"        if type(params) is dict and params:"}],"source_content_type":"text/x-python","patch_set":4,"id":"e159de58_a77c834d","line":222,"range":{"start_line":222,"start_character":60,"end_line":222,"end_character":62},"in_reply_to":"191613c4_486a3c4e","updated":"2021-08-26 04:32:20.000000000","message":"Previously headers is not used (see. L233) actually so the value is not problem. Now headers is used as an argument of self.httpclient.do_request. {} is correct and None causes an exception.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"08de60e9953494a1ba36abe7d42ac011768eb55d","unresolved":false,"context_lines":[{"line_number":219,"context_line":"        query \u003d query + urlparse.urlencode(keyval_encoded, doseq\u003d1)"},{"line_number":220,"context_line":"        return query.strip(\u0027\u0026\u0027)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def do_request(self, method, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":223,"context_line":"        action \u003d self.build_action(action)"},{"line_number":224,"context_line":"        # Add format and tenant_id"},{"line_number":225,"context_line":"        if type(params) is dict and params:"}],"source_content_type":"text/x-python","patch_set":4,"id":"8f7a9a26_6966aa3d","line":222,"range":{"start_line":222,"start_character":60,"end_line":222,"end_character":62},"in_reply_to":"e159de58_a77c834d","updated":"2021-08-26 05:58:57.000000000","message":"Thank you for your answer. I see.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"72791f83b749e0c58bf04410f4e9815555893812","unresolved":true,"context_lines":[{"line_number":980,"context_line":"            path \u003d \"/vnflcm/api_versions\""},{"line_number":981,"context_line":"        else:"},{"line_number":982,"context_line":"            path \u003d \"/vnflcm/{}/api_versions\".format(major_version)"},{"line_number":983,"context_line":"        return self.get(path, headers\u003d{\u0027Version\u0027: \u00272.0.0\u0027})"},{"line_number":984,"context_line":""},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"class Client(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"d069d8dc_c359b1b4","line":983,"range":{"start_line":983,"start_character":38,"end_line":983,"end_character":58},"updated":"2021-08-26 01:36:30.000000000","message":"Why don\u0027t you use \"self.headers\" for this header?","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7343c12200269e256d812dece9035f70cb9d5944","unresolved":true,"context_lines":[{"line_number":980,"context_line":"            path \u003d \"/vnflcm/api_versions\""},{"line_number":981,"context_line":"        else:"},{"line_number":982,"context_line":"            path \u003d \"/vnflcm/{}/api_versions\".format(major_version)"},{"line_number":983,"context_line":"        return self.get(path, headers\u003d{\u0027Version\u0027: \u00272.0.0\u0027})"},{"line_number":984,"context_line":""},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"class Client(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"d47856c7_d4d9c2f9","line":983,"range":{"start_line":983,"start_character":38,"end_line":983,"end_character":58},"in_reply_to":"d069d8dc_c359b1b4","updated":"2021-08-26 04:32:20.000000000","message":"This may be called with any combinations (ex. --os-tacker-api-verson \u003d\u003d 1 and major_version \u003d\u003d 2). Specifying headers\u003d{\u0027Version\u0027: \u00272.0.0\u0027} is most simple to make all cases OK.\nI will add a comment.","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"08de60e9953494a1ba36abe7d42ac011768eb55d","unresolved":false,"context_lines":[{"line_number":980,"context_line":"            path \u003d \"/vnflcm/api_versions\""},{"line_number":981,"context_line":"        else:"},{"line_number":982,"context_line":"            path \u003d \"/vnflcm/{}/api_versions\".format(major_version)"},{"line_number":983,"context_line":"        return self.get(path, headers\u003d{\u0027Version\u0027: \u00272.0.0\u0027})"},{"line_number":984,"context_line":""},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"class Client(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"64e0ecc1_375a30d5","line":983,"range":{"start_line":983,"start_character":38,"end_line":983,"end_character":58},"in_reply_to":"d47856c7_d4d9c2f9","updated":"2021-08-26 05:58:57.000000000","message":"Done","commit_id":"726960a2b3e0a07403f26bb69062850154e62178"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"a1e1e21dc12b3bf6c50dd0fc46f5f3950da68a3b","unresolved":true,"context_lines":[{"line_number":880,"context_line":"        self.headers \u003d {\u0027Version\u0027: \u00271.3.0\u0027}"},{"line_number":881,"context_line":"        sol_api_version \u003d \u0027v1\u0027"},{"line_number":882,"context_line":"        if api_version \u003d\u003d \u00272\u0027:"},{"line_number":883,"context_line":"            sol_api_version \u003d \u0027v2\u0027"},{"line_number":884,"context_line":"            self.headers \u003d {\u0027Version\u0027: \u00272.0.0\u0027}"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"        self.vnf_instances_path \u003d ("},{"line_number":887,"context_line":"            \u0027/vnflcm/{}/vnf_instances\u0027.format(sol_api_version))"}],"source_content_type":"text/x-python","patch_set":5,"id":"390b3233_804e8a8e","line":884,"range":{"start_line":883,"start_character":0,"end_line":884,"end_character":47},"updated":"2021-08-27 02:56:35.000000000","message":"Nit: It\u0027s better to unify the order.\n---\n            self.headers \u003d {\u0027Version\u0027: \u00272.0.0\u0027}\n            sol_api_version \u003d \u0027v2\u0027","commit_id":"42303e585fce02c68dec505d088b93a46d742358"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"9ae27d6f6b55ec13a8367baf43e2621dd1e8fe1c","unresolved":false,"context_lines":[{"line_number":880,"context_line":"        self.headers \u003d {\u0027Version\u0027: \u00271.3.0\u0027}"},{"line_number":881,"context_line":"        sol_api_version \u003d \u0027v1\u0027"},{"line_number":882,"context_line":"        if api_version \u003d\u003d \u00272\u0027:"},{"line_number":883,"context_line":"            sol_api_version \u003d \u0027v2\u0027"},{"line_number":884,"context_line":"            self.headers \u003d {\u0027Version\u0027: \u00272.0.0\u0027}"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"        self.vnf_instances_path \u003d ("},{"line_number":887,"context_line":"            \u0027/vnflcm/{}/vnf_instances\u0027.format(sol_api_version))"}],"source_content_type":"text/x-python","patch_set":5,"id":"a8b68671_211f8bf8","line":884,"range":{"start_line":883,"start_character":0,"end_line":884,"end_character":47},"in_reply_to":"0f30457e_64cb9a9f","updated":"2021-08-27 04:13:53.000000000","message":"Done","commit_id":"42303e585fce02c68dec505d088b93a46d742358"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"f13114817971d1485bb3eae1f7b0085e26615bea","unresolved":true,"context_lines":[{"line_number":880,"context_line":"        self.headers \u003d {\u0027Version\u0027: \u00271.3.0\u0027}"},{"line_number":881,"context_line":"        sol_api_version \u003d \u0027v1\u0027"},{"line_number":882,"context_line":"        if api_version \u003d\u003d \u00272\u0027:"},{"line_number":883,"context_line":"            sol_api_version \u003d \u0027v2\u0027"},{"line_number":884,"context_line":"            self.headers \u003d {\u0027Version\u0027: \u00272.0.0\u0027}"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"        self.vnf_instances_path \u003d ("},{"line_number":887,"context_line":"            \u0027/vnflcm/{}/vnf_instances\u0027.format(sol_api_version))"}],"source_content_type":"text/x-python","patch_set":5,"id":"0f30457e_64cb9a9f","line":884,"range":{"start_line":883,"start_character":0,"end_line":884,"end_character":47},"in_reply_to":"390b3233_804e8a8e","updated":"2021-08-27 03:57:47.000000000","message":"I see. I will fix.","commit_id":"42303e585fce02c68dec505d088b93a46d742358"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":219,"context_line":"        query \u003d query + urlparse.urlencode(keyval_encoded, doseq\u003d1)"},{"line_number":220,"context_line":"        return query.strip(\u0027\u0026\u0027)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def do_request(self, method, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":223,"context_line":"        action \u003d self.build_action(action)"},{"line_number":224,"context_line":"        # Add format and tenant_id"},{"line_number":225,"context_line":"        if type(params) is dict and params:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5ef2f0f8_93205536","line":222,"updated":"2021-08-30 07:48:57.000000000","message":"I think it\u0027s better to avoid using a mutable object as a default argument [1][2].\nIt\u0027s not a big deal in this case, but I believe it\u0027s better to avoid it as possible because other developers might do the same thing at a different place where this implementation causes a problem.\nIsn\u0027t possible to write the following inside ``do_request`` and change all ``headers\u003d{}`` to ``headers\u003dNone``?\n\n```\nif headers is None:\n    headers \u003d {}\n```\n\n\n[1] http://pylint-messages.wikidot.com/messages:w0102\n[2] https://stackoverflow.com/questions/41686829/warning-about-mutable-default-argument-in-pycharm","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"8150240b7a830539029a4abdf4e6e5112afe494b","unresolved":true,"context_lines":[{"line_number":219,"context_line":"        query \u003d query + urlparse.urlencode(keyval_encoded, doseq\u003d1)"},{"line_number":220,"context_line":"        return query.strip(\u0027\u0026\u0027)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def do_request(self, method, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":223,"context_line":"        action \u003d self.build_action(action)"},{"line_number":224,"context_line":"        # Add format and tenant_id"},{"line_number":225,"context_line":"        if type(params) is dict and params:"}],"source_content_type":"text/x-python","patch_set":6,"id":"f1ea1bf8_f9fabdcd","line":222,"in_reply_to":"5ef2f0f8_93205536","updated":"2021-08-30 22:10:02.000000000","message":"Thank you for the comment. I see. I will fix.","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":219,"context_line":"        query \u003d query + urlparse.urlencode(keyval_encoded, doseq\u003d1)"},{"line_number":220,"context_line":"        return query.strip(\u0027\u0026\u0027)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def do_request(self, method, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":223,"context_line":"        action \u003d self.build_action(action)"},{"line_number":224,"context_line":"        # Add format and tenant_id"},{"line_number":225,"context_line":"        if type(params) is dict and params:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5d61880f_d74b4755","line":222,"in_reply_to":"f1ea1bf8_f9fabdcd","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":307,"context_line":"            return \"application/%s\" % (_format)"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"    def retry_request(self, method, action, body\u003dNone,"},{"line_number":310,"context_line":"                      headers\u003d{}, params\u003dNone):"},{"line_number":311,"context_line":"        \"\"\"Call do_request with the default retry configuration."},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"        Only idempotent requests should retry failed connection attempts."}],"source_content_type":"text/x-python","patch_set":6,"id":"0c11d17a_72e6eddc","line":310,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":307,"context_line":"            return \"application/%s\" % (_format)"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"    def retry_request(self, method, action, body\u003dNone,"},{"line_number":310,"context_line":"                      headers\u003d{}, params\u003dNone):"},{"line_number":311,"context_line":"        \"\"\"Call do_request with the default retry configuration."},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"        Only idempotent requests should retry failed connection attempts."}],"source_content_type":"text/x-python","patch_set":6,"id":"93b6fd0b_fabba77f","line":310,"in_reply_to":"0c11d17a_72e6eddc","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":334,"context_line":""},{"line_number":335,"context_line":"        raise exceptions.ConnectionFailed(reason\u003dmsg)"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"    def delete(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":338,"context_line":"        return self.retry_request(\"DELETE\", action, body\u003dbody,"},{"line_number":339,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":340,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"57c89832_e1679e20","line":337,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":334,"context_line":""},{"line_number":335,"context_line":"        raise exceptions.ConnectionFailed(reason\u003dmsg)"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"    def delete(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":338,"context_line":"        return self.retry_request(\"DELETE\", action, body\u003dbody,"},{"line_number":339,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":340,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"788c330c_b2bbbccb","line":337,"in_reply_to":"57c89832_e1679e20","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":338,"context_line":"        return self.retry_request(\"DELETE\", action, body\u003dbody,"},{"line_number":339,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    def get(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":342,"context_line":"        return self.retry_request(\"GET\", action, body\u003dbody,"},{"line_number":343,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":344,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"c7b9da48_0ebbf2cb","line":341,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":338,"context_line":"        return self.retry_request(\"DELETE\", action, body\u003dbody,"},{"line_number":339,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    def get(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":342,"context_line":"        return self.retry_request(\"GET\", action, body\u003dbody,"},{"line_number":343,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":344,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"8c4517ef_69575e9b","line":341,"in_reply_to":"c7b9da48_0ebbf2cb","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":342,"context_line":"        return self.retry_request(\"GET\", action, body\u003dbody,"},{"line_number":343,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"    def post(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":346,"context_line":"        # Do not retry POST requests to avoid the orphan objects problem."},{"line_number":347,"context_line":"        return self.do_request(\"POST\", action, body\u003dbody,"},{"line_number":348,"context_line":"                               headers\u003dheaders, params\u003dparams)"}],"source_content_type":"text/x-python","patch_set":6,"id":"573c7fc0_f2317c59","line":345,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":342,"context_line":"        return self.retry_request(\"GET\", action, body\u003dbody,"},{"line_number":343,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"    def post(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":346,"context_line":"        # Do not retry POST requests to avoid the orphan objects problem."},{"line_number":347,"context_line":"        return self.do_request(\"POST\", action, body\u003dbody,"},{"line_number":348,"context_line":"                               headers\u003dheaders, params\u003dparams)"}],"source_content_type":"text/x-python","patch_set":6,"id":"206bb147_5f54eaf7","line":345,"in_reply_to":"573c7fc0_f2317c59","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":347,"context_line":"        return self.do_request(\"POST\", action, body\u003dbody,"},{"line_number":348,"context_line":"                               headers\u003dheaders, params\u003dparams)"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    def put(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":351,"context_line":"        return self.retry_request(\"PUT\", action, body\u003dbody,"},{"line_number":352,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":353,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"4ad787f1_c0d612b8","line":350,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":347,"context_line":"        return self.do_request(\"POST\", action, body\u003dbody,"},{"line_number":348,"context_line":"                               headers\u003dheaders, params\u003dparams)"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    def put(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":351,"context_line":"        return self.retry_request(\"PUT\", action, body\u003dbody,"},{"line_number":352,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":353,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"dace3ddb_a2b8276a","line":350,"in_reply_to":"4ad787f1_c0d612b8","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":351,"context_line":"        return self.retry_request(\"PUT\", action, body\u003dbody,"},{"line_number":352,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"    def patch(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":355,"context_line":"        return self.retry_request(\"PATCH\", action, body\u003dbody,"},{"line_number":356,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":357,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"85f91c86_a8bfd68d","line":354,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":351,"context_line":"        return self.retry_request(\"PUT\", action, body\u003dbody,"},{"line_number":352,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"    def patch(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":355,"context_line":"        return self.retry_request(\"PATCH\", action, body\u003dbody,"},{"line_number":356,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":357,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"2c8b94f4_e17436b4","line":354,"in_reply_to":"85f91c86_a8bfd68d","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":355,"context_line":"        return self.retry_request(\"PATCH\", action, body\u003dbody,"},{"line_number":356,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"    def list(self, collection, path, retrieve_all\u003dTrue, headers\u003d{},"},{"line_number":359,"context_line":"             **params):"},{"line_number":360,"context_line":"        if retrieve_all:"},{"line_number":361,"context_line":"            res \u003d []"}],"source_content_type":"text/x-python","patch_set":6,"id":"4e566dd8_bff3d503","line":358,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":355,"context_line":"        return self.retry_request(\"PATCH\", action, body\u003dbody,"},{"line_number":356,"context_line":"                                  headers\u003dheaders, params\u003dparams)"},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"    def list(self, collection, path, retrieve_all\u003dTrue, headers\u003d{},"},{"line_number":359,"context_line":"             **params):"},{"line_number":360,"context_line":"        if retrieve_all:"},{"line_number":361,"context_line":"            res \u003d []"}],"source_content_type":"text/x-python","patch_set":6,"id":"8d62ed02_fc34c21c","line":358,"in_reply_to":"4e566dd8_bff3d503","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":1014,"context_line":""},{"line_number":1015,"context_line":"    # LegacyClient methods"},{"line_number":1016,"context_line":""},{"line_number":1017,"context_line":"    def delete(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1018,"context_line":"        return self.legacy_client.delete(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1019,"context_line":"                                         params\u003dparams)"},{"line_number":1020,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"18ee1349_53eac395","line":1017,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":1014,"context_line":""},{"line_number":1015,"context_line":"    # LegacyClient methods"},{"line_number":1016,"context_line":""},{"line_number":1017,"context_line":"    def delete(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1018,"context_line":"        return self.legacy_client.delete(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1019,"context_line":"                                         params\u003dparams)"},{"line_number":1020,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"f08ed62f_7edb8753","line":1017,"in_reply_to":"18ee1349_53eac395","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":1018,"context_line":"        return self.legacy_client.delete(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1019,"context_line":"                                         params\u003dparams)"},{"line_number":1020,"context_line":""},{"line_number":1021,"context_line":"    def get(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1022,"context_line":"        return self.legacy_client.get(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1023,"context_line":"                                      params\u003dparams)"},{"line_number":1024,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5a5f44bb_ebf3d3e1","line":1021,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":1018,"context_line":"        return self.legacy_client.delete(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1019,"context_line":"                                         params\u003dparams)"},{"line_number":1020,"context_line":""},{"line_number":1021,"context_line":"    def get(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1022,"context_line":"        return self.legacy_client.get(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1023,"context_line":"                                      params\u003dparams)"},{"line_number":1024,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"361dabd3_b0addba3","line":1021,"in_reply_to":"5a5f44bb_ebf3d3e1","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":1022,"context_line":"        return self.legacy_client.get(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1023,"context_line":"                                      params\u003dparams)"},{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"    def post(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1026,"context_line":"        return self.legacy_client.post(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1027,"context_line":"                                       params\u003dparams)"},{"line_number":1028,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"72c48d0e_37e48399","line":1025,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":1022,"context_line":"        return self.legacy_client.get(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1023,"context_line":"                                      params\u003dparams)"},{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"    def post(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1026,"context_line":"        return self.legacy_client.post(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1027,"context_line":"                                       params\u003dparams)"},{"line_number":1028,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"0ab5b751_ce283155","line":1025,"in_reply_to":"72c48d0e_37e48399","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"e05beb22c6173c46a5ce66f4b942ff0d0a59402d","unresolved":true,"context_lines":[{"line_number":1026,"context_line":"        return self.legacy_client.post(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1027,"context_line":"                                       params\u003dparams)"},{"line_number":1028,"context_line":""},{"line_number":1029,"context_line":"    def put(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1030,"context_line":"        return self.legacy_client.put(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1031,"context_line":"                                      params\u003dparams)"},{"line_number":1032,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"63a89dc8_12ccbe86","line":1029,"updated":"2021-08-30 07:48:57.000000000","message":"ditto","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"3254d9beefe6317d734b9130105e773b4d99d4ad","unresolved":false,"context_lines":[{"line_number":1026,"context_line":"        return self.legacy_client.post(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1027,"context_line":"                                       params\u003dparams)"},{"line_number":1028,"context_line":""},{"line_number":1029,"context_line":"    def put(self, action, body\u003dNone, headers\u003d{}, params\u003dNone):"},{"line_number":1030,"context_line":"        return self.legacy_client.put(action, body\u003dbody, headers\u003dheaders,"},{"line_number":1031,"context_line":"                                      params\u003dparams)"},{"line_number":1032,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"e48e4b93_109b623b","line":1029,"in_reply_to":"63a89dc8_12ccbe86","updated":"2021-08-31 03:18:43.000000000","message":"Done","commit_id":"99819abed86e32be997082dfdd0ab37a37d9c3e3"}]}
