)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"5006b3e87519cddc7875cf50bee0d95e1a9dea87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"730e9665_b602038f","updated":"2025-07-17 14:44:52.000000000","message":"Change makes sense. Looks good to me.","commit_id":"bdb4964375c9f43a9109b3989b2589ebcd7abc6b"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"6e1147e0669735e3ca0ad60567ca64bcc93c3362","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b9460266_d8533784","updated":"2026-02-02 19:11:38.000000000","message":"+1 Change looks good.","commit_id":"5dcf7496456dd4922f5d10c2c627820b22f5e1be"}],"cinder/volume/drivers/remotefs.py":[{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"3f340256c05511fe396caf662b9c2316888196a0","unresolved":true,"context_lines":[{"line_number":882,"context_line":"                valid_ext \u003d \u0027\u0027"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"            if volume_name is None:"},{"line_number":885,"context_line":"                raise exception.RemoteFSException()"},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"            if ext_bf_template:"},{"line_number":888,"context_line":"                backing_file_template \u003d ext_bf_template.substitute("}],"source_content_type":"text/x-python","patch_set":3,"id":"8e59ac11_a4ed1915","line":885,"updated":"2025-07-17 14:33:02.000000000","message":"Raising an exception for the case volume_name is none doesn\u0027t mean we shouldn\u0027t accept a None value for that instead of changing it to Optional[str]?","commit_id":"bdb4964375c9f43a9109b3989b2589ebcd7abc6b"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"4db4a7f0c721ff7b36162a15579494929d57830d","unresolved":false,"context_lines":[{"line_number":882,"context_line":"                valid_ext \u003d \u0027\u0027"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"            if volume_name is None:"},{"line_number":885,"context_line":"                raise exception.RemoteFSException()"},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"            if ext_bf_template:"},{"line_number":888,"context_line":"                backing_file_template \u003d ext_bf_template.substitute("}],"source_content_type":"text/x-python","patch_set":3,"id":"cf48829f_912d3ff3","line":885,"in_reply_to":"8e59ac11_a4ed1915","updated":"2025-07-17 14:42:59.000000000","message":"Ok, I got it now. The lack of a volume name is only a problem in cases where we are dealing with images that have backing files. I\u0027m removing the -1.","commit_id":"bdb4964375c9f43a9109b3989b2589ebcd7abc6b"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"d396113c6154a10156158ca03afa675ef263b7ba","unresolved":true,"context_lines":[{"line_number":882,"context_line":"                valid_ext \u003d \u0027\u0027"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"            if volume_name is None:"},{"line_number":885,"context_line":"                raise exception.RemoteFSException()"},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"            if ext_bf_template:"},{"line_number":888,"context_line":"                backing_file_template \u003d ext_bf_template.substitute("}],"source_content_type":"text/x-python","patch_set":3,"id":"eca7b27c_a829af6c","line":885,"in_reply_to":"8e59ac11_a4ed1915","updated":"2025-07-17 14:41:51.000000000","message":"volume_name\u003dNone is only a problem when there is a backing_file - when there is no backing_file, this info is not required.  In the case where volume_name is None, the regex check below would have already failed.","commit_id":"bdb4964375c9f43a9109b3989b2589ebcd7abc6b"}]}