)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20733,"name":"Rajesh Tailor","email":"ratailor@redhat.com","username":"rajesht"},"change_message_id":"97ad28760a502b43f7d4d4422959b479b82e3657","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5ec8e039_d3a1cb39","updated":"2024-12-11 06:46:17.000000000","message":"LGTM","commit_id":"e10e826b190d19f189b6aa720ff3e7191d5aef89"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"fb2d46fd6615ef713b23f7b0082d7863384e8a87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"73888ecd_99fbf1ff","updated":"2025-01-29 17:34:57.000000000","message":"recheck unrelated failure","commit_id":"a48afecaff85df603826770814c737cc30723c85"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccb42e9f5a8a10252881a4c3ac01962ef6d880ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"9843ee14_eeac23b0","updated":"2025-05-22 10:19:14.000000000","message":"+1 is because i think this is correct but i think we ahave a docs bug","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d71a4437473fde8135a0e3db6b97c8503cb8b99c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7c88a537_681e47cb","updated":"2025-05-26 13:30:22.000000000","message":"+2 assuming other are OK with addressing the issues in https://review.opendev.org/c/openstack/nova/+/950867","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"90a788bfc152437ee3250c10873e8a8fa3d0d919","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"b94cce7f_3e664ec5","updated":"2025-05-26 13:05:57.000000000","message":"I pushed fixes for the issues noted so far to https://review.opendev.org/c/openstack/nova/+/950867/. If I rebase, I can squash those back in here.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e0f9287db8c4118908aefeb03693255a4c33ad7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7ccbd995_703dcfd6","updated":"2025-06-05 21:27:47.000000000","message":"lgtm","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"2a9785deed7a5e8c448cde3bd7d4dbf6584fcf75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"51dff7e2_43358375","updated":"2025-06-10 17:45:01.000000000","message":"recheck","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"}],"nova/api/openstack/compute/hypervisors.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e0f9287db8c4118908aefeb03693255a4c33ad7d","unresolved":true,"context_lines":[{"line_number":223,"context_line":"    @validation.query_schema(schema.index_query_v233, \u00272.33\u0027, \u00272.52\u0027)"},{"line_number":224,"context_line":"    @validation.query_schema(schema.index_query_v253, \u00272.53\u0027)"},{"line_number":225,"context_line":"    @validation.response_body_schema(schema.index_response, \u00272.1\u0027, \u00272.32\u0027)"},{"line_number":226,"context_line":"    @validation.response_body_schema(schema.index_response_v233, \u00272.33\u0027, \u00272.52\u0027)  # noqa: E501"},{"line_number":227,"context_line":"    @validation.response_body_schema(schema.index_response_v253, \u00272.53\u0027)"},{"line_number":228,"context_line":"    def index(self, req):"},{"line_number":229,"context_line":"        \"\"\"List hypervisors."}],"source_content_type":"text/x-python","patch_set":10,"id":"0f515625_7ffde10d","line":226,"range":{"start_line":226,"start_character":81,"end_line":226,"end_character":94},"updated":"2025-06-05 21:27:47.000000000","message":"noqa not required as such here, like below is not so un-readable\n\n    @validation.response_body_schema(schema.index_response_v233,\n                                     \u00272.33\u0027, \u00272.52\u0027)","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"}],"nova/api/openstack/compute/schemas/hypervisors.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccb42e9f5a8a10252881a4c3ac01962ef6d880ae","unresolved":true,"context_lines":[{"line_number":72,"context_line":"search_query \u003d {}"},{"line_number":73,"context_line":"servers_query \u003d {}"},{"line_number":74,"context_line":"uptime_query \u003d {}"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"_servers_response \u003d {"},{"line_number":77,"context_line":"    \u0027type\u0027: \u0027array\u0027,"},{"line_number":78,"context_line":"    \u0027items\u0027: {"}],"source_content_type":"text/x-python","patch_set":10,"id":"e7852155_f060257b","line":75,"updated":"2025-05-22 10:19:14.000000000","message":"so the indext endpoitn respocne is defiened here \nhttps://docs.openstack.org/api-ref/compute/#id293","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"af4649aa425ff45b736c089f3418a4dc4a5acfac","unresolved":false,"context_lines":[{"line_number":72,"context_line":"search_query \u003d {}"},{"line_number":73,"context_line":"servers_query \u003d {}"},{"line_number":74,"context_line":"uptime_query \u003d {}"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"_servers_response \u003d {"},{"line_number":77,"context_line":"    \u0027type\u0027: \u0027array\u0027,"},{"line_number":78,"context_line":"    \u0027items\u0027: {"}],"source_content_type":"text/x-python","patch_set":10,"id":"3fdf0f80_020d4f27","line":75,"in_reply_to":"e7852155_f060257b","updated":"2025-05-26 12:46:56.000000000","message":"Acknowledged","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccb42e9f5a8a10252881a4c3ac01962ef6d880ae","unresolved":true,"context_lines":[{"line_number":84,"context_line":"        \u0027required\u0027: [\u0027name\u0027, \u0027uuid\u0027],"},{"line_number":85,"context_line":"        \u0027additionalProperties\u0027: False,"},{"line_number":86,"context_line":"    },"},{"line_number":87,"context_line":"}"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"_hypervisor_response \u003d {"},{"line_number":90,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":10,"id":"f4011d6d_8a7b4cb5","line":87,"updated":"2025-05-22 10:19:14.000000000","message":"there server object was added in 2.53\n\nacording to the api ref servers is an array of object but the name and uuid are options.\n\nand the list is empty if no server are on the hypervior\n\nform 2.75 servers is required in the responce.\n\n\nthe schema and doc are not in allignment.\n\nin the request docs it states the servers are only condtionally included if you pass with_servers\u003dtrue\n\nwhich when combiend with this schema  impile sthe respocne  the doc shold mark the server array as optional not the server uuid and name\n\n\nso im inclidne to say this si is correct but the compute api ref is incorrect.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"af4649aa425ff45b736c089f3418a4dc4a5acfac","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        \u0027required\u0027: [\u0027name\u0027, \u0027uuid\u0027],"},{"line_number":85,"context_line":"        \u0027additionalProperties\u0027: False,"},{"line_number":86,"context_line":"    },"},{"line_number":87,"context_line":"}"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"_hypervisor_response \u003d {"},{"line_number":90,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":10,"id":"81e7c848_43ac5771","line":87,"in_reply_to":"f4011d6d_8a7b4cb5","updated":"2025-05-26 12:46:56.000000000","message":"\u003e which when combiend with this schema  impile sthe respocne  the doc shold mark the server array as optional not the server uuid and name\n\nYes, this is my understanding too and is what I was trying (and failing) to say [below](https://review.opendev.org/c/openstack/nova/+/937245/comment/f6745e11_875d30b6/). Will update.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccb42e9f5a8a10252881a4c3ac01962ef6d880ae","unresolved":true,"context_lines":[{"line_number":89,"context_line":"_hypervisor_response \u003d {"},{"line_number":90,"context_line":"    \u0027type\u0027: \u0027object\u0027,"},{"line_number":91,"context_line":"    \u0027properties\u0027: {"},{"line_number":92,"context_line":"        \u0027id\u0027: {\u0027type\u0027: \u0027integer\u0027},"},{"line_number":93,"context_line":"        \u0027hypervisor_hostname\u0027: {\u0027type\u0027: \u0027string\u0027},"},{"line_number":94,"context_line":"        \u0027state\u0027: {\u0027enum\u0027: [\u0027up\u0027, \u0027down\u0027]},"},{"line_number":95,"context_line":"        \u0027status\u0027: {\u0027enum\u0027: [\u0027enabled\u0027, \u0027disabled\u0027]},"}],"source_content_type":"text/x-python","patch_set":10,"id":"b8d8b18c_61c2a5cd","line":92,"range":{"start_line":92,"start_character":7,"end_line":92,"end_character":34},"updated":"2025-05-22 10:19:14.000000000","message":"this was converted to a uuid starting in 2.53 to supprot multple cells so yes thsi is correct for the base versions.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"af4649aa425ff45b736c089f3418a4dc4a5acfac","unresolved":false,"context_lines":[{"line_number":89,"context_line":"_hypervisor_response \u003d {"},{"line_number":90,"context_line":"    \u0027type\u0027: \u0027object\u0027,"},{"line_number":91,"context_line":"    \u0027properties\u0027: {"},{"line_number":92,"context_line":"        \u0027id\u0027: {\u0027type\u0027: \u0027integer\u0027},"},{"line_number":93,"context_line":"        \u0027hypervisor_hostname\u0027: {\u0027type\u0027: \u0027string\u0027},"},{"line_number":94,"context_line":"        \u0027state\u0027: {\u0027enum\u0027: [\u0027up\u0027, \u0027down\u0027]},"},{"line_number":95,"context_line":"        \u0027status\u0027: {\u0027enum\u0027: [\u0027enabled\u0027, \u0027disabled\u0027]},"}],"source_content_type":"text/x-python","patch_set":10,"id":"d571a037_320b04e9","line":92,"range":{"start_line":92,"start_character":7,"end_line":92,"end_character":34},"in_reply_to":"b8d8b18c_61c2a5cd","updated":"2025-05-26 12:46:56.000000000","message":"Acknowledged","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccb42e9f5a8a10252881a4c3ac01962ef6d880ae","unresolved":true,"context_lines":[{"line_number":98,"context_line":"    \u0027additionalProperties\u0027: False,"},{"line_number":99,"context_line":"}"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"_hypervisor_response_v253 \u003d copy.deepcopy(_hypervisor_response)"},{"line_number":102,"context_line":"_hypervisor_response_v253[\u0027properties\u0027].update({"},{"line_number":103,"context_line":"    \u0027id\u0027: {\u0027type\u0027: \u0027string\u0027, \u0027format\u0027: \u0027uuid\u0027},"},{"line_number":104,"context_line":"    \u0027servers\u0027: copy.deepcopy(_servers_response),"},{"line_number":105,"context_line":"})"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"index_response \u003d {"},{"line_number":108,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":10,"id":"530292d3_4e8bca1b","line":105,"range":{"start_line":101,"start_character":0,"end_line":105,"end_character":2},"updated":"2025-05-22 10:19:14.000000000","message":"this is really confusign to put here it would be better to put it on line 125\n\nwhen i read lines 125-132 it was very confusing to see the comment you left there when the encodeign of the server and id type are seperated like this","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"af4649aa425ff45b736c089f3418a4dc4a5acfac","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    \u0027additionalProperties\u0027: False,"},{"line_number":99,"context_line":"}"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"_hypervisor_response_v253 \u003d copy.deepcopy(_hypervisor_response)"},{"line_number":102,"context_line":"_hypervisor_response_v253[\u0027properties\u0027].update({"},{"line_number":103,"context_line":"    \u0027id\u0027: {\u0027type\u0027: \u0027string\u0027, \u0027format\u0027: \u0027uuid\u0027},"},{"line_number":104,"context_line":"    \u0027servers\u0027: copy.deepcopy(_servers_response),"},{"line_number":105,"context_line":"})"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"index_response \u003d {"},{"line_number":108,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":10,"id":"38193f1f_24502763","line":105,"range":{"start_line":101,"start_character":0,"end_line":105,"end_character":2},"in_reply_to":"530292d3_4e8bca1b","updated":"2025-05-26 12:46:56.000000000","message":"I\u0027m trying to keep all of the \"private\" or \"helper\" dictionaries up top so that they\u0027re obvious. There\u0027s just a lot of sharing going on here so it\u0027s difficult to do in a way that\u0027s sensible and doesn\u0027t result in loads of duplication.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccb42e9f5a8a10252881a4c3ac01962ef6d880ae","unresolved":false,"context_lines":[{"line_number":120,"context_line":"index_response_v233 \u003d copy.deepcopy(index_response)"},{"line_number":121,"context_line":"index_response_v233[\u0027properties\u0027].update({"},{"line_number":122,"context_line":"    \u0027hypervisors_links\u0027: response_types.collection_links,"},{"line_number":123,"context_line":"})"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# v2.53 adds the \u0027servers\u0027 field but only if a user requests it via the"},{"line_number":126,"context_line":"# \u0027with_servers\u0027 query arg. It also changes the \u0027id\u0027 field to a UUID. Note that"}],"source_content_type":"text/x-python","patch_set":10,"id":"8722d9b2_9991f19c","line":123,"updated":"2025-05-22 10:19:14.000000000","message":"ah yep tye were in 2.33 i intially misread that as 2.53 but no this is correct","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1d34a67d271dd76b1dff6431287fc4e8767e3141","unresolved":true,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# v2.53 adds the \u0027servers\u0027 field but only if a user requests it via the"},{"line_number":126,"context_line":"# \u0027with_servers\u0027 query arg. It also changes the \u0027id\u0027 field to a UUID. Note that"},{"line_number":127,"context_line":"# v2.75 makes the \u0027servers\u0027 property always present even if empty, but that\u0027s"},{"line_number":128,"context_line":"# not something we can capture with jsonschema so we don\u0027t try"},{"line_number":129,"context_line":"index_response_v253 \u003d copy.deepcopy(index_response_v233)"},{"line_number":130,"context_line":"index_response_v253[\u0027properties\u0027][\u0027hypervisors\u0027][\u0027items\u0027] \u003d ("},{"line_number":131,"context_line":"    _hypervisor_response_v253"}],"source_content_type":"text/x-python","patch_set":10,"id":"f6745e11_875d30b6","line":128,"range":{"start_line":127,"start_character":0,"end_line":128,"end_character":62},"updated":"2025-05-22 11:25:24.000000000","message":"im not sure why you cant have a 2.75 version of the index responce where you make it a required filed? can you explain?","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e0f9287db8c4118908aefeb03693255a4c33ad7d","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# v2.53 adds the \u0027servers\u0027 field but only if a user requests it via the"},{"line_number":126,"context_line":"# \u0027with_servers\u0027 query arg. It also changes the \u0027id\u0027 field to a UUID. Note that"},{"line_number":127,"context_line":"# v2.75 makes the \u0027servers\u0027 property always present even if empty, but that\u0027s"},{"line_number":128,"context_line":"# not something we can capture with jsonschema so we don\u0027t try"},{"line_number":129,"context_line":"index_response_v253 \u003d copy.deepcopy(index_response_v233)"},{"line_number":130,"context_line":"index_response_v253[\u0027properties\u0027][\u0027hypervisors\u0027][\u0027items\u0027] \u003d ("},{"line_number":131,"context_line":"    _hypervisor_response_v253"}],"source_content_type":"text/x-python","patch_set":10,"id":"b8b37493_166b5c31","line":128,"range":{"start_line":127,"start_character":0,"end_line":128,"end_character":62},"in_reply_to":"1a017e08_44fec4c0","updated":"2025-06-05 21:27:47.000000000","message":"yeah \u0027servers\u0027 field depends on either it is present or it is requested in query param so it is not always present so not adding it in required field is ok.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d71a4437473fde8135a0e3db6b97c8503cb8b99c","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# v2.53 adds the \u0027servers\u0027 field but only if a user requests it via the"},{"line_number":126,"context_line":"# \u0027with_servers\u0027 query arg. It also changes the \u0027id\u0027 field to a UUID. Note that"},{"line_number":127,"context_line":"# v2.75 makes the \u0027servers\u0027 property always present even if empty, but that\u0027s"},{"line_number":128,"context_line":"# not something we can capture with jsonschema so we don\u0027t try"},{"line_number":129,"context_line":"index_response_v253 \u003d copy.deepcopy(index_response_v233)"},{"line_number":130,"context_line":"index_response_v253[\u0027properties\u0027][\u0027hypervisors\u0027][\u0027items\u0027] \u003d ("},{"line_number":131,"context_line":"    _hypervisor_response_v253"}],"source_content_type":"text/x-python","patch_set":10,"id":"1a017e08_44fec4c0","line":128,"range":{"start_line":127,"start_character":0,"end_line":128,"end_character":62},"in_reply_to":"929656e8_bc4e715e","updated":"2025-05-26 13:30:22.000000000","message":"ack that makes sense why you took this approch.\ni agree i see you have updated teh comment in a follow up patch\nhttps://review.opendev.org/c/openstack/nova/+/950867\nso as long as we merge the hyperviors changes a block i think thats fine.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"af4649aa425ff45b736c089f3418a4dc4a5acfac","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# v2.53 adds the \u0027servers\u0027 field but only if a user requests it via the"},{"line_number":126,"context_line":"# \u0027with_servers\u0027 query arg. It also changes the \u0027id\u0027 field to a UUID. Note that"},{"line_number":127,"context_line":"# v2.75 makes the \u0027servers\u0027 property always present even if empty, but that\u0027s"},{"line_number":128,"context_line":"# not something we can capture with jsonschema so we don\u0027t try"},{"line_number":129,"context_line":"index_response_v253 \u003d copy.deepcopy(index_response_v233)"},{"line_number":130,"context_line":"index_response_v253[\u0027properties\u0027][\u0027hypervisors\u0027][\u0027items\u0027] \u003d ("},{"line_number":131,"context_line":"    _hypervisor_response_v253"}],"source_content_type":"text/x-python","patch_set":10,"id":"929656e8_bc4e715e","line":128,"range":{"start_line":127,"start_character":0,"end_line":128,"end_character":62},"in_reply_to":"f6745e11_875d30b6","updated":"2025-05-26 12:46:56.000000000","message":"Yeah, sorry, this wasn\u0027t clear.\n\nBefore v2.75, servers are always returned if present. From v2.75 onwards, servers are returned (a) if present and if the with_servers query string argument is not false, or (b) if the with_servers query string argument is true. I\u0027m saying that we can\u0027t model the relationship between the query string argument and the presence of the field. I\u0027ll reword to try clear it up somewhat.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1d34a67d271dd76b1dff6431287fc4e8767e3141","unresolved":true,"context_lines":[{"line_number":136,"context_line":"servers_response \u003d copy.deepcopy(index_response)"},{"line_number":137,"context_line":"servers_response[\u0027properties\u0027][\u0027hypervisors\u0027][\u0027items\u0027][\u0027properties\u0027].update({"},{"line_number":138,"context_line":"    \u0027servers\u0027: copy.deepcopy(_servers_response),"},{"line_number":139,"context_line":"})"}],"source_content_type":"text/x-python","patch_set":10,"id":"1c992148_00cb5bd4","line":139,"updated":"2025-05-22 11:25:24.000000000","message":"ok, looking at this again this is https://docs.openstack.org/api-ref/compute/#id308\nand until 2.52 when it became a flag on list\n\nwehn list and detail list were updated in 2.52 use use an int id this was not updated which is why you are usign the base index resonce.\n\nthat looks correct.","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"af4649aa425ff45b736c089f3418a4dc4a5acfac","unresolved":false,"context_lines":[{"line_number":136,"context_line":"servers_response \u003d copy.deepcopy(index_response)"},{"line_number":137,"context_line":"servers_response[\u0027properties\u0027][\u0027hypervisors\u0027][\u0027items\u0027][\u0027properties\u0027].update({"},{"line_number":138,"context_line":"    \u0027servers\u0027: copy.deepcopy(_servers_response),"},{"line_number":139,"context_line":"})"}],"source_content_type":"text/x-python","patch_set":10,"id":"7ebe4ec2_283e5dac","line":139,"in_reply_to":"1c992148_00cb5bd4","updated":"2025-05-26 12:46:56.000000000","message":"Acknowledged","commit_id":"531ede50a3356208d660a69b352a92912dc22a05"}]}