)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b7bbd459a8802dbd5fc5342a9e6387bc1496819a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f3087fa3_ca8d2ade","updated":"2025-05-13 03:09:49.000000000","message":"recheck\n\nwe fixed the manila-api app failures","commit_id":"7658154cb26b011fc3bde884a4688a4a8509e729"},{"author":{"_account_id":36761,"name":"za","display_name":"zgoggin","email":"zacharygoggin@coenin.co.uk","username":"za","status":"Manila share backup enthusiast :)"},"change_message_id":"0b34397b18424ee8bcd46892753d77d4911d16f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"91e3bd17_275dbc31","updated":"2025-04-28 15:02:33.000000000","message":"recheck - failures appear to be related to issues instancing devstack","commit_id":"7658154cb26b011fc3bde884a4688a4a8509e729"}],"manila/data/manager.py":[{"author":{"_account_id":36761,"name":"za","display_name":"zgoggin","email":"zacharygoggin@coenin.co.uk","username":"za","status":"Manila share backup enthusiast :)"},"change_message_id":"0f3cba4f6961aa300c541caed8e1688be0cb7e4a","unresolved":true,"context_lines":[{"line_number":685,"context_line":"                result \u003d {}"},{"line_number":686,"context_line":"                try:"},{"line_number":687,"context_line":"                    if self.backup_driver.use_data_manager is False:"},{"line_number":688,"context_line":"                        # REVIEW - zgoggin: this \\ feels unclean. seek style"},{"line_number":689,"context_line":"                        # feedback for trying to \u003d\u003d\u003d pep8 here better..."},{"line_number":690,"context_line":"                        backup_values \u003d self.backup_driver.\\"},{"line_number":691,"context_line":"                            get_restore_progress(context, backup, share)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f02864ad_45bcfca6","line":688,"range":{"start_line":688,"start_character":24,"end_line":688,"end_character":32},"updated":"2025-04-26 08:44:49.000000000","message":"Highlighting for self / others.","commit_id":"c12de208156f17c77d08f2dd8b989830d7c4fd09"}],"manila/tests/data/test_manager.py":[{"author":{"_account_id":36761,"name":"za","display_name":"zgoggin","email":"zacharygoggin@coenin.co.uk","username":"za","status":"Manila share backup enthusiast :)"},"change_message_id":"0f3cba4f6961aa300c541caed8e1688be0cb7e4a","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        self.share \u003d db_utils.create_share()"},{"line_number":52,"context_line":"        manager.CONF.set_default(\u0027mount_tmp_location\u0027, \u0027/tmp/\u0027)"},{"line_number":53,"context_line":"        manager.CONF.set_default(\u0027backup_mount_tmp_location\u0027, \u0027/tmp/\u0027)"},{"line_number":54,"context_line":"        # REVIEW: - zgoggin: If the intention bellow is to use the fake in the"},{"line_number":55,"context_line":"        # tests, this will not work, while the CONF value is set properly,"},{"line_number":56,"context_line":"        # the manager defaults/loads the NFS driver prior when instanced above."},{"line_number":57,"context_line":"        # This can be seen with print(vars(self.manager.backup_driver)) within"}],"source_content_type":"text/x-python","patch_set":1,"id":"776e6419_79973613","line":54,"range":{"start_line":54,"start_character":9,"end_line":54,"end_character":16},"updated":"2025-04-26 08:44:49.000000000","message":"Highlighting for self / others.","commit_id":"c12de208156f17c77d08f2dd8b989830d7c4fd09"}]}
