)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"854edf26d90482a7eaecd96be6e92b65e0a63304","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"43ea6c82_55df6cac","updated":"2026-05-25 12:27:53.000000000","message":"no tests added, \n\nplease add tests, the bug report is good with steps to reproduce.","commit_id":"d605fd1573c7dbf9e4a4ef3c1041c51dfe47de2b"},{"author":{"_account_id":38860,"name":"soheil","display_name":"soheil bagan","email":"soheillinred@hotmail.com","username":"soheillinred"},"change_message_id":"fcbd9b632fd6d4481b9fa3200165c82b67bba330","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d55c873a_f18a2c47","in_reply_to":"43ea6c82_55df6cac","updated":"2026-05-31 07:40:57.000000000","message":"Done","commit_id":"d605fd1573c7dbf9e4a4ef3c1041c51dfe47de2b"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"58d00be5568e9b4b3f73daac9bdb2fab000781f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9fd8f23d_d3020cd9","updated":"2026-06-02 09:44:47.000000000","message":"CI has valid issue and added tests did not ran.\n\nalso added test only verify config parameter is set, but do not verify if backups actually works with this patch.","commit_id":"2695739c4454feb190782f57ac10fa5c3390ccd5"},{"author":{"_account_id":38860,"name":"soheil","display_name":"soheil bagan","email":"soheillinred@hotmail.com","username":"soheillinred"},"change_message_id":"fcbd9b632fd6d4481b9fa3200165c82b67bba330","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2f1db7c2_b9fe539e","updated":"2026-05-31 07:40:57.000000000","message":"function test added","commit_id":"2695739c4454feb190782f57ac10fa5c3390ccd5"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"2b0231bc95611086ea63179f962f62f9b696fbf3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7937d6a1_46bb7ba5","updated":"2026-06-02 09:47:26.000000000","message":"on Separate note can you or your team look at this [1] email (in openstack-discuss mailing list) regarding `botocore.vendored` \nif you or your org using this we would like to get you opinion.\n\n[1] https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/HEEWHRVRYFWQNYEYBZ5SCRAF5ODSL3S6/","commit_id":"2695739c4454feb190782f57ac10fa5c3390ccd5"}],"cinder/tests/unit/backup/drivers/test_backup_s3.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"58d00be5568e9b4b3f73daac9bdb2fab000781f1","unresolved":true,"context_lines":[{"line_number":624,"context_line":"        # Test that request_checksum_calculation is set to \u0027when_required\u0027."},{"line_number":625,"context_line":"        volume_id \u003d \u0027b09b1ad4-5f0e-4d3f-8b9e-0000004f5ec2\u0027"},{"line_number":626,"context_line":"        container_name \u003d \u0027test-bucket\u0027"},{"line_number":627,"context_line":"        backup \u003d self._create_backup_db_entry(volume_id\u003dvolume_id,"},{"line_number":628,"context_line":"                                          container\u003dcontainer_name)"},{"line_number":629,"context_line":""},{"line_number":630,"context_line":"            # Mock the boto3 client creation to verify config"}],"source_content_type":"text/x-python","patch_set":4,"id":"498930c7_590c5a40","line":627,"updated":"2026-06-02 09:44:47.000000000","message":"its created but not used!! same for next test.\nits because its not required so this don\u0027t need saving.","commit_id":"2695739c4454feb190782f57ac10fa5c3390ccd5"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"58d00be5568e9b4b3f73daac9bdb2fab000781f1","unresolved":true,"context_lines":[{"line_number":640,"context_line":"            self.assertEqual(\u0027when_required\u0027, config[\u0027request_checksum_calculation\u0027])"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"    @mock_aws"},{"line_number":643,"context_line":"    def test_backup_with_default_checksum_config(self):"},{"line_number":644,"context_line":"        # Test that other config parameters remain unchanged."},{"line_number":645,"context_line":"        volume_id \u003d \u0027b09b1ad4-5f0e-4d3f-8b9e-0000004f5ec2\u0027"},{"line_number":646,"context_line":"        container_name \u003d \u0027test-bucket\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"0144a0e6_774df0d3","line":643,"in_reply_to":"58ee2e62_0e54c887","updated":"2026-06-02 09:44:47.000000000","message":"\u003e pep8: E999 IndentationError: unindent does not match any outer indentation level\n\nPlease fix, this test did not really ran in CI because of indentaion error\nyou can use `pre-commit run --all-fiels` before pushing for reviews that will help","commit_id":"2695739c4454feb190782f57ac10fa5c3390ccd5"}]}
