)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"afdc231b7bd06b03d88ae33d726da054e860ce56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9485f8d_52ae41a9","updated":"2021-12-01 10:11:53.000000000","message":"This needs a lot of work, but it\u0027s a good start","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"008d55c76f1626ea17ac335c4ba9420b59eab21e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"64b087a5_6de9e3fb","updated":"2022-03-16 22:46:58.000000000","message":"Hi Thanks for reviewing this. Pending indentation issue is now fixed. ","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"b179a1139e45d67a528b6f654fce5ebeee8514ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b7ae6150_65f00246","updated":"2022-03-14 07:48:35.000000000","message":"Nit: Commit message rules: https://wiki.openstack.org/wiki/GitCommitMessages#Git_Commit_Good_Practice","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"5f07df8a5ff64afda51e056eb1c130c2bdd6d32f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"df251ebb_5c37082a","updated":"2022-03-17 07:00:36.000000000","message":"Please propose all changes in relation chain in one patch\nHere is guide: https://docs.opendev.org/opendev/infra-manual/latest/developers.html#updating-a-change","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"2accc16ba38d60df4bd2cc0406c6a5f3433ce263","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"83f9cc4c_9ef94646","updated":"2022-03-16 22:49:17.000000000","message":"Trying to reply regarding the allow_fetch. Per my test, it has to be set to True, otherwise the code would fail. ","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"127dca03_ddb9008f","updated":"2022-03-09 16:46:26.000000000","message":"just submitted another merge request for this one. Please review. allow_fetch has to be set, otherwise hitting this error: \nopenstack.exceptions.MethodNotSupported: The fetch method is not supported for openstack.compute.v2.server_actions.ServerActions","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"}],"openstack/compute/v2/_proxy.py":[{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from openstack.compute.v2 import limits"},{"line_number":23,"context_line":"from openstack.compute.v2 import server as _server"},{"line_number":24,"context_line":"from openstack.compute.v2 import server_diagnostics as _server_diagnostics"},{"line_number":25,"context_line":"from openstack.compute.v2 import server_osinstanceactions as _server_osinstanceactions"},{"line_number":26,"context_line":"from openstack.compute.v2 import server_group as _server_group"},{"line_number":27,"context_line":"from openstack.compute.v2 import server_interface as _server_interface"},{"line_number":28,"context_line":"from openstack.compute.v2 import server_ip"}],"source_content_type":"text/x-python","patch_set":2,"id":"48537fb9_37a62ef2","line":25,"in_reply_to":"0a229442_460a9859","updated":"2022-03-09 16:46:26.000000000","message":"\u003e pep8: E501 line too long (86 \u003e 79 characters)\n\nPlease fix.","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from openstack.compute.v2 import server as _server"},{"line_number":24,"context_line":"from openstack.compute.v2 import server_diagnostics as _server_diagnostics"},{"line_number":25,"context_line":"from openstack.compute.v2 import server_osinstanceactions as _server_osinstanceactions"},{"line_number":26,"context_line":"from openstack.compute.v2 import server_group as _server_group"},{"line_number":27,"context_line":"from openstack.compute.v2 import server_interface as _server_interface"},{"line_number":28,"context_line":"from openstack.compute.v2 import server_ip"},{"line_number":29,"context_line":"from openstack.compute.v2 import server_remote_console as _src"}],"source_content_type":"text/x-python","patch_set":2,"id":"de13cb34_59457773","line":26,"in_reply_to":"2a96db0f_88033b14","updated":"2022-03-09 16:46:26.000000000","message":"\u003e pep8: H306: imports not in alphabetical order (openstack.compute.v2.server_osinstanceactions, openstack.compute.v2.server_group)\n\nPlease fix.","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"afdc231b7bd06b03d88ae33d726da054e860ce56","unresolved":true,"context_lines":[{"line_number":1763,"context_line":"        return self._get(_server_diagnostics.ServerDiagnostics,"},{"line_number":1764,"context_line":"                         server_id\u003dserver_id, requires_id\u003dFalse)"},{"line_number":1765,"context_line":""},{"line_number":1766,"context_line":"    def get_server_osinstanceactions(self, server):"},{"line_number":1767,"context_line":"        \"\"\"Get a single server action logs"},{"line_number":1768,"context_line":""},{"line_number":1769,"context_line":"        :param server: This parameter need to be specified when ServerInterface"}],"source_content_type":"text/x-python","patch_set":2,"id":"dd896f1e_48f93f87","line":1766,"range":{"start_line":1766,"start_character":8,"end_line":1766,"end_character":36},"updated":"2021-12-01 10:11:53.000000000","message":"get_server_actions","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[{"line_number":1763,"context_line":"        return self._get(_server_diagnostics.ServerDiagnostics,"},{"line_number":1764,"context_line":"                         server_id\u003dserver_id, requires_id\u003dFalse)"},{"line_number":1765,"context_line":""},{"line_number":1766,"context_line":"    def get_server_osinstanceactions(self, server):"},{"line_number":1767,"context_line":"        \"\"\"Get a single server action logs"},{"line_number":1768,"context_line":""},{"line_number":1769,"context_line":"        :param server: This parameter need to be specified when ServerInterface"}],"source_content_type":"text/x-python","patch_set":2,"id":"68b9ddc9_d104c59d","line":1766,"range":{"start_line":1766,"start_character":8,"end_line":1766,"end_character":36},"in_reply_to":"dd896f1e_48f93f87","updated":"2022-03-09 16:46:26.000000000","message":"Done","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"b179a1139e45d67a528b6f654fce5ebeee8514ca","unresolved":true,"context_lines":[{"line_number":1870,"context_line":"        \"\"\"Get a single server action logs"},{"line_number":1871,"context_line":""},{"line_number":1872,"context_line":"        :param server: This parameter need to be specified when ServerInterface"},{"line_number":1873,"context_line":"                       ID is given as value. It can be either the ID of a"},{"line_number":1874,"context_line":"                       server or a :class:`~openstack.compute.v2.server.Server`"},{"line_number":1875,"context_line":"                       instance that the interface belongs to."},{"line_number":1876,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"e2b5dc28_6774118c","line":1873,"updated":"2022-03-14 07:48:35.000000000","message":"Please fix indents, same as above methods","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"008d55c76f1626ea17ac335c4ba9420b59eab21e","unresolved":false,"context_lines":[{"line_number":1870,"context_line":"        \"\"\"Get a single server action logs"},{"line_number":1871,"context_line":""},{"line_number":1872,"context_line":"        :param server: This parameter need to be specified when ServerInterface"},{"line_number":1873,"context_line":"                       ID is given as value. It can be either the ID of a"},{"line_number":1874,"context_line":"                       server or a :class:`~openstack.compute.v2.server.Server`"},{"line_number":1875,"context_line":"                       instance that the interface belongs to."},{"line_number":1876,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"26315b35_b1800aa8","line":1873,"in_reply_to":"e2b5dc28_6774118c","updated":"2022-03-16 22:46:58.000000000","message":"This is now fixed here: https://review.opendev.org/c/openstack/openstacksdk/+/834079/1.","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"}],"openstack/compute/v2/server_actions.py":[{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"5f07df8a5ff64afda51e056eb1c130c2bdd6d32f","unresolved":true,"context_lines":[{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    # List of the actions for the given instance."}],"source_content_type":"text/x-python","patch_set":3,"id":"9e8c9b3d_9bbd2501","line":20,"updated":"2022-03-17 07:00:36.000000000","message":"white space here after \u0027#\u0027","commit_id":"0ff51c8604cc7cded0de6a4a62f8b41dbdf10245"}],"openstack/compute/v2/server_osinstanceactions.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"afdc231b7bd06b03d88ae33d726da054e860ce56","unresolved":true,"context_lines":[{"line_number":13,"context_line":"from openstack import resource"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"class ServerOSInstanceActions(resource.Resource):"},{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7d656a4e_262a01a9","line":16,"updated":"2021-12-01 10:11:53.000000000","message":"Can we simply call this \u0027ServerActions\u0027?","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[{"line_number":13,"context_line":"from openstack import resource"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"class ServerOSInstanceActions(resource.Resource):"},{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"362de43b_290597ec","line":16,"in_reply_to":"7d656a4e_262a01a9","updated":"2022-03-09 16:46:26.000000000","message":"Done","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"afdc231b7bd06b03d88ae33d726da054e860ce56","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"class ServerOSInstanceActions(resource.Resource):"},{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"}],"source_content_type":"text/x-python","patch_set":2,"id":"c5addbe3_1d51c268","line":17,"updated":"2021-12-01 10:11:53.000000000","message":"This key determines how you remove the envelope from the response. Based on the example on the next page, this is \u0027instanceActions\u0027:\n\n  \"instanceActions\": {\n      ... actual fields ...\n  }\n\nAlso, this should \u0027resources_key\u0027 if you fix the allow_fetch thing below","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"class ServerOSInstanceActions(resource.Resource):"},{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"}],"source_content_type":"text/x-python","patch_set":2,"id":"86a7416f_5e1894c5","line":17,"in_reply_to":"c5addbe3_1d51c268","updated":"2022-03-09 16:46:26.000000000","message":"Done","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"008d55c76f1626ea17ac335c4ba9420b59eab21e","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"2f374759_6d6126da","line":20,"in_reply_to":"589a4791_c1805eaf","updated":"2022-03-16 22:46:58.000000000","message":"not sure what this means. I see there is \u0027#\u0027 in front of \u0027capabilities\u0027 above","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":true,"context_lines":[{"line_number":17,"context_line":"    resource_key \u003d \u0027os-instance-actions\u0027"},{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"}],"source_content_type":"text/x-python","patch_set":2,"id":"589a4791_c1805eaf","line":20,"in_reply_to":"7e3416ef_776efcb5","updated":"2022-03-09 16:46:26.000000000","message":"\u003e pep8: E265 block comment should start with \u0027# \u0027\n\nPlease fix.","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"afdc231b7bd06b03d88ae33d726da054e860ce56","unresolved":true,"context_lines":[{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"a70ec241_35f38383","line":21,"updated":"2021-12-01 10:11:53.000000000","message":"You _list_ server actions - there\u0027s no way to fetch an individual record","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"2accc16ba38d60df4bd2cc0406c6a5f3433ce263","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"e0a3ca5c_2a8757d5","line":21,"in_reply_to":"7b2ed44f_5c4bac7c","updated":"2022-03-16 22:49:17.000000000","message":"Done","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":true,"context_lines":[{"line_number":18,"context_line":"    base_path \u003d \u0027/servers/%(server_id)s/os-instance-actions\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7b2ed44f_5c4bac7c","line":21,"in_reply_to":"a70ec241_35f38383","updated":"2022-03-09 16:46:26.000000000","message":"This has to be set to True, otherwise it won\u0027t work for openstacksdk to retrieve actionlogs, and I am getting this error: \nopenstack.exceptions.MethodNotSupported: The fetch method is not supported for openstack.compute.v2.server_actions.ServerActions","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"afdc231b7bd06b03d88ae33d726da054e860ce56","unresolved":true,"context_lines":[{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    # List of the actions for the given instance."},{"line_number":26,"context_line":"    instanceActions \u003d resource.Body(\u0027instanceActions\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1b0ec12f_c53b4a3a","line":23,"updated":"2021-12-01 10:11:53.000000000","message":"This isn\u0027t relevant if you change the above","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"},{"author":{"_account_id":34257,"name":"hai wu","email":"haiwu.us@gmail.com","username":"hai"},"change_message_id":"81411b6aec4fea02ba05240ddcd21cf149b5bd17","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    #capabilities"},{"line_number":21,"context_line":"    allow_fetch \u003d True"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    requires_id \u003d False"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    # List of the actions for the given instance."},{"line_number":26,"context_line":"    instanceActions \u003d resource.Body(\u0027instanceActions\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"2f4e2002_dac88820","line":23,"in_reply_to":"1b0ec12f_c53b4a3a","updated":"2022-03-09 16:46:26.000000000","message":"Done","commit_id":"7867f6ea96ffebb3e5903fc86ae95df8b2872151"}]}