)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"54051143_84faceb0","updated":"2024-11-15 01:44:19.000000000","message":"- whitespace removed\n- line changed to under 79 characters","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"7f946b8390c97fdcd5e170582e60fae5e092a9e8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8242fcaa_c821bcd6","updated":"2024-11-18 01:34:45.000000000","message":"Abandoning this review to submit to master first.\nWill backport to stable/2024.1 after master merge.","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"d3f557ed863783d02ff274edcc7b17db8327eabe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7278692a_c98dfbd8","updated":"2024-11-15 12:49:01.000000000","message":"Stable backports need to be cherry picked from the commit that merged in the master branch.  See https://review.opendev.org/c/openstack/cinder/+/929502 as an example.  This pattern needs to be in place for all of your stable proposals so that reviewers can track origin.","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"882623030620f3ead82d6c38e2df9a7d3d136839","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f50ce687_19012f86","updated":"2024-11-15 04:30:16.000000000","message":"The CI failure appears to be due to server capacity issues (503 Service Unavailable).\nRequesting recheck.","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"98d12e02ed2bd49a6dde67ba2f012abdeb1fb326","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"018826f7_f7992e63","updated":"2024-11-15 04:30:50.000000000","message":"recheck - CI server returned 503 Service Unavailable error","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"ee4dcb146195525d0ab8a908f6a083a9290f01db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"08f2ae50_c0d8195d","updated":"2024-11-15 06:58:33.000000000","message":"recheck tempest-ipv6-only - Resource constraints/infrastructure issue","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"ad78c37dc0eb471c8cbef829529ea8afb8063ca6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4dddca50_dbc7bb7d","in_reply_to":"7278692a_c98dfbd8","updated":"2024-11-19 03:03:18.000000000","message":"Thank you. \nI have created a new review for master branch according to your guidance. \nPlease refer to the link below.\nhttps://review.opendev.org/c/openstack/cinder/+/935473","commit_id":"449c2cbae277ef68360a17790677f2c9c39abc84"}],"cinder/tests/unit/volume/flows/test_create_volume_flow.py":[{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2449,"context_line":"            mock_create_from_img_dl, mock_create_from_src,"},{"line_number":2450,"context_line":"            mock_handle_bootable, mock_fetch_img):"},{"line_number":2451,"context_line":"        \"\"\"Test PowerFlex volume creation from image with correct final size."},{"line_number":2452,"context_line":"        "},{"line_number":2453,"context_line":"        Scenario:"},{"line_number":2454,"context_line":"        1. Image virtual size: 10GiB"},{"line_number":2455,"context_line":"        2. Requested volume size: 24GiB"}],"source_content_type":"text/x-python","patch_set":1,"id":"24d850fe_31c25596","line":2452,"in_reply_to":"829d6535_3d3d44d7","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: W293 blank line contains whitespace\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2466,"context_line":""},{"line_number":2467,"context_line":"        # Disable image signature verification"},{"line_number":2468,"context_line":"        self.flags(verify_glance_signatures\u003d\u0027disabled\u0027)"},{"line_number":2469,"context_line":"        "},{"line_number":2470,"context_line":"        # Setup volume with 24GiB size"},{"line_number":2471,"context_line":"        volume \u003d fake_volume.fake_volume_obj("},{"line_number":2472,"context_line":"            self.ctxt,"}],"source_content_type":"text/x-python","patch_set":1,"id":"21d8af9a_e9603b4d","line":2469,"in_reply_to":"d9faa54d_0ec53786","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: W293 blank line contains whitespace\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2474,"context_line":"            size\u003d24,"},{"line_number":2475,"context_line":"            host\u003d\u0027host@powerflex#pool\u0027"},{"line_number":2476,"context_line":"        )"},{"line_number":2477,"context_line":"        "},{"line_number":2478,"context_line":"        # Mock the volume.save method"},{"line_number":2479,"context_line":"        volume.save \u003d mock.MagicMock(return_value\u003dNone)"},{"line_number":2480,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"50eaae20_3dccb6e6","line":2477,"in_reply_to":"3f699f33_111aad23","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: W293 blank line contains whitespace\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2491,"context_line":"        # Configure mocks for PowerFlex behavior"},{"line_number":2492,"context_line":"        self.mock_driver.clone_image.return_value \u003d (None, False)"},{"line_number":2493,"context_line":"        self.mock_cache.get_entry.return_value \u003d None"},{"line_number":2494,"context_line":"        "},{"line_number":2495,"context_line":"        # Mock the create_from_image_download to return volume with 16GiB size"},{"line_number":2496,"context_line":"        def mock_create_download(context, volume, image_location, image_meta, image_service):"},{"line_number":2497,"context_line":"            # Update volume size to 16 as PowerFlex rounds to 8GiB multiple"}],"source_content_type":"text/x-python","patch_set":1,"id":"69d24915_572285d5","line":2494,"in_reply_to":"cfd9944c_3537a794","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: W293 blank line contains whitespace\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2493,"context_line":"        self.mock_cache.get_entry.return_value \u003d None"},{"line_number":2494,"context_line":"        "},{"line_number":2495,"context_line":"        # Mock the create_from_image_download to return volume with 16GiB size"},{"line_number":2496,"context_line":"        def mock_create_download(context, volume, image_location, image_meta, image_service):"},{"line_number":2497,"context_line":"            # Update volume size to 16 as PowerFlex rounds to 8GiB multiple"},{"line_number":2498,"context_line":"            volume.size \u003d 16"},{"line_number":2499,"context_line":"            return {\u0027status\u0027: \u0027available\u0027, \u0027size\u0027: 16}"}],"source_content_type":"text/x-python","patch_set":1,"id":"ea30e790_3777612e","line":2496,"in_reply_to":"b1dfe9d1_045bf0eb","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: E501 line too long (93 \u003e 79 characters)\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2497,"context_line":"            # Update volume size to 16 as PowerFlex rounds to 8GiB multiple"},{"line_number":2498,"context_line":"            volume.size \u003d 16"},{"line_number":2499,"context_line":"            return {\u0027status\u0027: \u0027available\u0027, \u0027size\u0027: 16}"},{"line_number":2500,"context_line":"        "},{"line_number":2501,"context_line":"        mock_create_from_img_dl.side_effect \u003d mock_create_download"},{"line_number":2502,"context_line":""},{"line_number":2503,"context_line":"        # Mock extend_volume to update size to 24"}],"source_content_type":"text/x-python","patch_set":1,"id":"1e6757b9_8ee47c72","line":2500,"in_reply_to":"93391aa5_784e2854","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: W293 blank line contains whitespace\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"},{"author":{"_account_id":37469,"name":"DongHun, Kim","display_name":"DongHun, Kim","email":"bigheadzacky@gmail.com","username":"donghunkimzack"},"change_message_id":"d5b1a720a8e86c4abb884b3822a557311b83eec7","unresolved":false,"context_lines":[{"line_number":2504,"context_line":"        def mock_extend(volume, new_size):"},{"line_number":2505,"context_line":"            volume.size \u003d new_size"},{"line_number":2506,"context_line":"            return {\u0027size\u0027: new_size}"},{"line_number":2507,"context_line":"        "},{"line_number":2508,"context_line":"        self.mock_driver.extend_volume.side_effect \u003d mock_extend"},{"line_number":2509,"context_line":""},{"line_number":2510,"context_line":"        manager \u003d create_volume_manager.CreateVolumeFromSpecTask("}],"source_content_type":"text/x-python","patch_set":1,"id":"69643c58_b9266cfd","line":2507,"in_reply_to":"ae6193cf_55fa42fa","updated":"2024-11-15 01:44:19.000000000","message":"\u003e pep8: W293 blank line contains whitespace\n\nPlease fix.","commit_id":"42cfd66f5180ffbaadf7cc0262111281c3104de0"}]}