)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"31532f072a6bddbb44520c08fb6d6fb328497139","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a902d1a6_89369182","updated":"2023-08-08 19:45:02.000000000","message":"May want to wait to recheck until after https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/890828 merges.","commit_id":"91f22dd2bc7b7dd70e5d0b8afa4a1ebb3be534b7"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cf2d3ba8fcc3a86c5cb258e93eb3e320759445b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b095609d_4b036263","updated":"2023-08-08 17:53:27.000000000","message":"cinder-tempest-plugin-lvm-lio-barbican hit two OOM errors:\nAug 08 16:19:05 np0034894611 kernel: Out of memory: Killed process 49892 (mysqld)\nand not surprisingly, after that there are a bunch of 500s from identity service.  System eventually recovers, and then:\nAug 08 16:39:26 np0034894611 kernel: Out of memory: Killed process 126697 (qemu-system-x86)\nand a bunch of timeouts during tests","commit_id":"91f22dd2bc7b7dd70e5d0b8afa4a1ebb3be534b7"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4b9cc582490bf1b641245044492dd4367a2b2ef6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2564e816_ca32ccfb","updated":"2023-08-14 18:09:11.000000000","message":"recheck because there is no other option","commit_id":"91f22dd2bc7b7dd70e5d0b8afa4a1ebb3be534b7"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0d27cbc842758bcd646d0194a348fe50efd11810","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f7b76d70_61b85d1f","updated":"2023-08-17 15:39:00.000000000","message":"recheck cinder-plugin-ceph-tempest job has been fixed","commit_id":"91f22dd2bc7b7dd70e5d0b8afa4a1ebb3be534b7"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"466b493334b69d7ff9c420e08e920edaae597935","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f25f15c9_9ed41e5f","updated":"2023-08-18 02:29:38.000000000","message":"recheck cinder-grenade-mn-sub-volbak failed during grenade setup","commit_id":"5ca435bed62cdeba35c1acb88eb38e578c305510"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"abf1e48f5d509eeb4d372f85625ee7557f10c883","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d48cea9f_23825f1e","updated":"2023-08-19 13:19:44.000000000","message":"All I did was add type annotations, correct a few typos, add a comment, and add the release note.  I did not modify any of the logic of the change, so I feel like I can still review this change.\n\nRaising the review priority because this is not a very nice regression.","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"16240efc9f5fa81130a4971d4d977bd6e53e2f4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"51c712d3_95a29024","updated":"2023-08-21 06:25:49.000000000","message":"The code looks good functionality wise, few comments inline for improvement that can be addressed as a followup.","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b28b963fbe5dde71d20052686f67b2df0ad798e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8da08054_956da544","updated":"2023-08-21 19:58:58.000000000","message":"recheck\n\ntempest-integrated-storage 89 tempest test failures","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"6a5dcdce1483fa5885ea7dab490636a436aa5e2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1863e946_4286c861","updated":"2023-08-22 17:48:47.000000000","message":"recheck\n\ntempest.api.compute.servers.test_server_actions.ServerActionsTestOtherA.test_resize_volume_backed_server_confirm ssh timeout","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5f59a011228273d45e4d02a131186281bbaf4649","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a08d5b08_e48ddda6","updated":"2023-08-22 22:49:34.000000000","message":"recheck cinder-grenade-mn-sub-volbak","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8ef0c6c61e207e0c656c39642b89a7929b45611e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f4e5e425_a852d3b1","updated":"2023-08-18 22:49:18.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican timeout in teardown class","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b2bb07ab85f48a40a7e5a685c90edcc8ce83b6b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"413d14ea_b3b1618d","updated":"2023-08-22 12:38:34.000000000","message":"recheck tempest-integrated-storage - bunch of DB connection errors, looks like mysql got killed by OOM reaper","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a550fdef0822e27121b90794452620387896a0ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f28ee5c0_49f88e30","updated":"2023-08-22 15:29:31.000000000","message":"recheck tempest-integrated-storage - tempest.api.volume.test_volumes_snapshots.VolumesSnapshotTestJSON.test_snapshot_backup timeout waiting for backup to reach available status","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"}],"cinder/scheduler/host_manager.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"16240efc9f5fa81130a4971d4d977bd6e53e2f4e","unresolved":true,"context_lines":[{"line_number":989,"context_line":"        # If the capability and value are not in the same type,"},{"line_number":990,"context_line":"        # we just convert them into string to compare them."},{"line_number":991,"context_line":"        return str(value) \u003d\u003d str(capability)"},{"line_number":992,"context_line":""},{"line_number":993,"context_line":"    def get_az(self,"},{"line_number":994,"context_line":"               volume: objects.Volume,"},{"line_number":995,"context_line":"               availability_zone: Union[str, None]) -\u003e Union[str, None]:"},{"line_number":996,"context_line":"        if availability_zone:"},{"line_number":997,"context_line":"            az \u003d availability_zone"},{"line_number":998,"context_line":"        elif volume:"},{"line_number":999,"context_line":"            az \u003d volume.availability_zone"},{"line_number":1000,"context_line":"        else:"},{"line_number":1001,"context_line":"            az \u003d None"},{"line_number":1002,"context_line":"        return az"},{"line_number":1003,"context_line":""},{"line_number":1004,"context_line":"    def get_backup_host(self,"},{"line_number":1005,"context_line":"                        volume: objects.Volume,"}],"source_content_type":"text/x-python","patch_set":7,"id":"f2f211cf_6ace0752","line":1002,"range":{"start_line":992,"start_character":0,"end_line":1002,"end_character":17},"updated":"2023-08-21 06:25:49.000000000","message":"nit: this logic can be shortened with\n\n    return availability_zone or volume.availability_zone\n\n\nalso maybe this logic makes sense here but it could be avoided if we just pass the above logic from cinder.backup.api","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"}],"releasenotes/notes/bug-1912624-bakup-a-z-regression-452f4bc9dfd41871.yaml":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"16240efc9f5fa81130a4971d4d977bd6e53e2f4e","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #1912624 \u003chttps://bugs.launchpad.net/cinder/+bug/1912624\u003e`_:"},{"line_number":5,"context_line":"    Corrected regression introduced by the refactoring of the backup"},{"line_number":6,"context_line":"    service in the ussuri release, which prevented the creation of"},{"line_number":7,"context_line":"    a volume backup in a different availability zone."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"13b108cd_eda6ae34","line":6,"range":{"start_line":5,"start_character":43,"end_line":6,"end_character":11},"updated":"2023-08-21 06:25:49.000000000","message":"nit: not exactly refactoring but by the change implementing the workflow to create a backup via scheduler\n\nrefactoring doesn\u0027t introduce functional changes, this can be corrected in a followup","commit_id":"ddcf394ae277692d7cab2e03b4b6ae67ba7f1cc3"}]}