)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6fc8422e275ea0043e51abfdb6024c2b2184f530","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"eecbf1dc_63aece12","updated":"2022-08-22 17:22:30.000000000","message":"Could you add a release note? Otherwise, this looks very good. Thanks!","commit_id":"06cb59a81dabb383fd818e28aaae68c502caf7ad"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d2e451bea419f33d075156d0ddf25c6af98f41a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c7bc0198_7d8812cd","updated":"2022-08-24 09:34:27.000000000","message":"Still missing the release note 😞","commit_id":"455a239231ec590b465a69a39657e730e1fc2a5c"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"877478375ed72945c630295fa674d6b2f953a5cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9b5141dd_6a9963ae","in_reply_to":"c7bc0198_7d8812cd","updated":"2022-08-24 13:02:40.000000000","message":"Release note added.","commit_id":"455a239231ec590b465a69a39657e730e1fc2a5c"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"c01bf205b36e174293b2b493e8958e9f9f6b65ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3f146fdb_9ad26051","updated":"2023-03-24 10:01:35.000000000","message":"recheck","commit_id":"2142eb0ad8f10523c9f7b30ad5ed28c093515314"}],"openstack/compute/v2/server.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"fa8a7d45d0f3d20fe44d45ff69a0ec428f442962","unresolved":true,"context_lines":[{"line_number":488,"context_line":"        Unshelve -- Unshelve the server."},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"        :param availability_zone: If specified the instance will be unshelved"},{"line_number":491,"context_line":"                                  to the availability_zone."},{"line_number":492,"context_line":"                                  If None is passed the instance defined"},{"line_number":493,"context_line":"                                  availability_zone is unpin and the instance"},{"line_number":494,"context_line":"                                  will be scheduled to any availability_zone"}],"source_content_type":"text/x-python","patch_set":1,"id":"d7833f48_b4191988","line":491,"range":{"start_line":491,"start_character":13,"end_line":491,"end_character":34},"updated":"2022-08-22 17:23:10.000000000","message":"nit: You don\u0027t need the giant indent. 4 spaces is loads 😊","commit_id":"06cb59a81dabb383fd818e28aaae68c502caf7ad"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"033b951a4a1ba49b0b20d64a49d0fbdd4e00b980","unresolved":false,"context_lines":[{"line_number":488,"context_line":"        Unshelve -- Unshelve the server."},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"        :param availability_zone: If specified the instance will be unshelved"},{"line_number":491,"context_line":"                                  to the availability_zone."},{"line_number":492,"context_line":"                                  If None is passed the instance defined"},{"line_number":493,"context_line":"                                  availability_zone is unpin and the instance"},{"line_number":494,"context_line":"                                  will be scheduled to any availability_zone"}],"source_content_type":"text/x-python","patch_set":1,"id":"0e34f9fe_6de07bc4","line":491,"range":{"start_line":491,"start_character":13,"end_line":491,"end_character":34},"in_reply_to":"d7833f48_b4191988","updated":"2022-08-23 09:04:24.000000000","message":"Done","commit_id":"06cb59a81dabb383fd818e28aaae68c502caf7ad"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6fc8422e275ea0043e51abfdb6024c2b2184f530","unresolved":true,"context_lines":[{"line_number":508,"context_line":"            availability_zone is None or isinstance(availability_zone, str)"},{"line_number":509,"context_line":"        ) and host:"},{"line_number":510,"context_line":"            body[\"unshelve\"] \u003d {\"availability_zone\": availability_zone,"},{"line_number":511,"context_line":"                                \"host\": host}"},{"line_number":512,"context_line":""},{"line_number":513,"context_line":"        self._action(session, body)"},{"line_number":514,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"418ed2bc_d37c98eb","line":511,"updated":"2022-08-22 17:22:30.000000000","message":"nit:\n\n  data \u003d {}\n  if host:\n      data[\u0027host\u0027] \u003d host\n  if availability_zone is None or isinstance(availability_zone, str):\n      data[\u0027availability_zone\u0027] \u003d availability_zone\n  body \u003d {\u0027unshelve\u0027: data or None}\n\nwould do the same thing but nice, I think","commit_id":"06cb59a81dabb383fd818e28aaae68c502caf7ad"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"033b951a4a1ba49b0b20d64a49d0fbdd4e00b980","unresolved":false,"context_lines":[{"line_number":508,"context_line":"            availability_zone is None or isinstance(availability_zone, str)"},{"line_number":509,"context_line":"        ) and host:"},{"line_number":510,"context_line":"            body[\"unshelve\"] \u003d {\"availability_zone\": availability_zone,"},{"line_number":511,"context_line":"                                \"host\": host}"},{"line_number":512,"context_line":""},{"line_number":513,"context_line":"        self._action(session, body)"},{"line_number":514,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"b69a60a5_5adcc314","line":511,"in_reply_to":"418ed2bc_d37c98eb","updated":"2022-08-23 09:04:24.000000000","message":"Done","commit_id":"06cb59a81dabb383fd818e28aaae68c502caf7ad"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d2e451bea419f33d075156d0ddf25c6af98f41a2","unresolved":true,"context_lines":[{"line_number":70,"context_line":"        **tag.TagMixin._tag_query_parameters"},{"line_number":71,"context_line":"    )"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    _max_microversion \u003d \u00272.73\u0027"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    #: A list of dictionaries holding links relevant to this server."},{"line_number":76,"context_line":"    links \u003d resource.Body(\u0027links\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"8f155098_0a46ba92","line":73,"updated":"2022-08-24 09:34:27.000000000","message":"You need to bump this. This will set the microversion that openstacksdk will use by default for server commands, assuming the server supports it (openstacksdk will downgrade if it doesn\u0027t)","commit_id":"455a239231ec590b465a69a39657e730e1fc2a5c"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"877478375ed72945c630295fa674d6b2f953a5cb","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        **tag.TagMixin._tag_query_parameters"},{"line_number":71,"context_line":"    )"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    _max_microversion \u003d \u00272.73\u0027"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    #: A list of dictionaries holding links relevant to this server."},{"line_number":76,"context_line":"    links \u003d resource.Body(\u0027links\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"8b86f511_9b028229","line":73,"in_reply_to":"8f155098_0a46ba92","updated":"2022-08-24 13:02:40.000000000","message":"Done","commit_id":"455a239231ec590b465a69a39657e730e1fc2a5c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d2e451bea419f33d075156d0ddf25c6af98f41a2","unresolved":true,"context_lines":[{"line_number":502,"context_line":"            data[\"host\"] \u003d host"},{"line_number":503,"context_line":"        if availability_zone is None or isinstance(availability_zone, str):"},{"line_number":504,"context_line":"            data[\"availability_zone\"] \u003d availability_zone"},{"line_number":505,"context_line":"        if ("},{"line_number":506,"context_line":"            availability_zone is None or isinstance(availability_zone, str)"},{"line_number":507,"context_line":"        ) and host:"},{"line_number":508,"context_line":"            data[\"host\"] \u003d host"},{"line_number":509,"context_line":"            data[\"availability_zone\"] \u003d availability_zone"},{"line_number":510,"context_line":"        body \u003d {\u0027unshelve\u0027: data or None}"},{"line_number":511,"context_line":"        self._action(session, body)"},{"line_number":512,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"6824ec94_4fc8df3f","line":509,"range":{"start_line":505,"start_character":0,"end_line":509,"end_character":57},"updated":"2022-08-24 09:34:27.000000000","message":"This isn\u0027t necessary now. The two \u0027if\u0027 statements above will fill in this information, and the \u0027data or None\u0027 statement will ensure we don\u0027t send an empty dictionary if neither of these are set.","commit_id":"455a239231ec590b465a69a39657e730e1fc2a5c"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"877478375ed72945c630295fa674d6b2f953a5cb","unresolved":false,"context_lines":[{"line_number":502,"context_line":"            data[\"host\"] \u003d host"},{"line_number":503,"context_line":"        if availability_zone is None or isinstance(availability_zone, str):"},{"line_number":504,"context_line":"            data[\"availability_zone\"] \u003d availability_zone"},{"line_number":505,"context_line":"        if ("},{"line_number":506,"context_line":"            availability_zone is None or isinstance(availability_zone, str)"},{"line_number":507,"context_line":"        ) and host:"},{"line_number":508,"context_line":"            data[\"host\"] \u003d host"},{"line_number":509,"context_line":"            data[\"availability_zone\"] \u003d availability_zone"},{"line_number":510,"context_line":"        body \u003d {\u0027unshelve\u0027: data or None}"},{"line_number":511,"context_line":"        self._action(session, body)"},{"line_number":512,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3326b650_c9279f58","line":509,"range":{"start_line":505,"start_character":0,"end_line":509,"end_character":57},"in_reply_to":"6824ec94_4fc8df3f","updated":"2022-08-24 13:02:40.000000000","message":"oops 😮, you are completely right. Sorry about that.","commit_id":"455a239231ec590b465a69a39657e730e1fc2a5c"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b1cf6d7e89a6a6f3bf54bc266325b32f6767c7e6","unresolved":true,"context_lines":[{"line_number":483,"context_line":"        body \u003d {\"shelve\": None}"},{"line_number":484,"context_line":"        self._action(session, body)"},{"line_number":485,"context_line":""},{"line_number":486,"context_line":"    def unshelve(self, session, availability_zone\u003d_sentinel, host\u003dNone):"},{"line_number":487,"context_line":"        \"\"\""},{"line_number":488,"context_line":"        Unshelve -- Unshelve the server."},{"line_number":489,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ad42270c_2873f04a","line":486,"range":{"start_line":486,"start_character":50,"end_line":486,"end_character":59},"updated":"2022-10-05 10:23:47.000000000","message":"nit: you could use \u0027...\u0027 (Ellipsis) for this. It\u0027s not really meant for that, but it does work and looks better in auto-generated docs.","commit_id":"2142eb0ad8f10523c9f7b30ad5ed28c093515314"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b1cf6d7e89a6a6f3bf54bc266325b32f6767c7e6","unresolved":true,"context_lines":[{"line_number":500,"context_line":"        data \u003d {}"},{"line_number":501,"context_line":"        if host:"},{"line_number":502,"context_line":"            data[\"host\"] \u003d host"},{"line_number":503,"context_line":"        if availability_zone is None or isinstance(availability_zone, str):"},{"line_number":504,"context_line":"            data[\"availability_zone\"] \u003d availability_zone"},{"line_number":505,"context_line":"        body \u003d {\u0027unshelve\u0027: data or None}"},{"line_number":506,"context_line":"        self._action(session, body)"}],"source_content_type":"text/x-python","patch_set":3,"id":"f0870419_41f7b9eb","line":503,"updated":"2022-10-05 10:23:47.000000000","message":"nit:\n\n  if availability_zone is not _sentinel:","commit_id":"2142eb0ad8f10523c9f7b30ad5ed28c093515314"}]}