)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"1de16f57f5142560e2f4b57cdc61878855c927c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0eee96e8_339cd700","updated":"2026-04-30 17:19:50.000000000","message":"Thank you for the review everyone. Answers inline :)","commit_id":"b3517c576a26d1d8ca0676e51e4ecaaa1cb4213a"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"ef275650622251fbff1420de364e5036f70a74e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"280fc3cd_59b691a6","updated":"2026-04-30 13:03:55.000000000","message":"Well written change, LGTM","commit_id":"b3517c576a26d1d8ca0676e51e4ecaaa1cb4213a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a25e2b1093cc3afa247258cf88754a9b88857bde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"562ae0ed_b47446f6","updated":"2026-04-30 17:16:08.000000000","message":"almost lgtm but a few more things to address:\n- please add release notes for the new client\n- add unit tests in https://github.com/openstack/tempest/tree/master/tempest/tests/lib/services/compute\n- also, can you please point me to the change (manila tests if you have) where they are used so that we can know it works fine.","commit_id":"b3517c576a26d1d8ca0676e51e4ecaaa1cb4213a"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"1de16f57f5142560e2f4b57cdc61878855c927c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c0234456_6ca6e7dd","in_reply_to":"562ae0ed_b47446f6","updated":"2026-04-30 17:19:50.000000000","message":"working on the release notes and the unit tests now, thank you! The change that is making use of it is: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/984884, producing this job run: https://zuul.opendev.org/t/openstack/build/f761d9016979435abe69c849a645ebc2","commit_id":"b3517c576a26d1d8ca0676e51e4ecaaa1cb4213a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"68706a7b69111c3debc766b6b85929de3ff649f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c97a326d_01cd13da","in_reply_to":"7dc87ce8_84df066e","updated":"2026-05-01 19:47:53.000000000","message":"perfect, thanks","commit_id":"b3517c576a26d1d8ca0676e51e4ecaaa1cb4213a"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"8373b2f336b91a3f10c952ded1b5a4613dfd22ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7dc87ce8_84df066e","in_reply_to":"c0234456_6ca6e7dd","updated":"2026-04-30 20:02:09.000000000","message":"Added unit tests and the release notes. I realized there isn\u0027t a well defined way to do the microversion testing in the nova client unit tests, so I went for a new class, which I think looks cleaner and more maintainable. Please let me know in case I should use the mocking annotation as other tests do.","commit_id":"b3517c576a26d1d8ca0676e51e4ecaaa1cb4213a"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"92be533f542079754f521a5e5d4424403b0d723f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"553a3709_10fa9662","updated":"2026-05-06 11:34:15.000000000","message":"LGTM","commit_id":"b38b7d916f939a362d501feaa36859afa3e0ff84"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"9b0219df756ec4dad5bbe212c5d6ab84bf9fc63a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d6669cbd_f22e4d62","updated":"2026-05-04 18:57:43.000000000","message":"Thanks for the reviews!\n\nrecheck - nova-ceph-multistore failed for an unrelated reason","commit_id":"b38b7d916f939a362d501feaa36859afa3e0ff84"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"b812b5a501ced5f5cd06a2612fb1bbbe7650805f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d613022c_bb60798a","updated":"2026-05-01 12:51:59.000000000","message":"Yeah all gates except ironic-tempest-bios-ipmi-autodetect are passing, and that one fails due to a timeout","commit_id":"b38b7d916f939a362d501feaa36859afa3e0ff84"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"621414274606ae67a235a597ea9fd6abb9a49c5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"65d18054_3a2d4bf2","updated":"2026-05-01 09:24:32.000000000","message":"recheck\nNeither ironic or ceph jobs seem to be related to the code I\u0027m touching. Let\u0027s give it another shot.","commit_id":"b38b7d916f939a362d501feaa36859afa3e0ff84"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"68706a7b69111c3debc766b6b85929de3ff649f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b95b2515_f5909633","updated":"2026-05-01 19:47:53.000000000","message":"recheck ironic job. change lgtm","commit_id":"b38b7d916f939a362d501feaa36859afa3e0ff84"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"94ea3ad63ed00d269aa50939738470baecebe272","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5e8ed55f_93b9095a","updated":"2026-05-01 12:51:26.000000000","message":"recheck ironic-tempest-bios-ipmi-autodetect","commit_id":"b38b7d916f939a362d501feaa36859afa3e0ff84"}]}