daviftorres commented on issue #11727: URL: https://github.com/apache/cloudstack/issues/11727#issuecomment-3429008333
Looking at the code (https://github.com/apache/cloudstack/blob/f63118c011ecd90a81adee5fd043a966b4822c47/plugins/backup/nas/src/main/java/org/apache/cloudstack/backup/NASBackupProvider.java) I see the error handlers: - Line 223 sets status to `Failed` when connection fails, - Line 228 sets status to `Failed` when times out, - Line 251 sets status to `Failed` when fails for any other reason. Note that in line 247 sets status to `Error` when the cleanup fails, and that is the only time it updates and leave the backup with a failing statatus behind. Al the other scenarios it removed right after setting status. <img width="1056" height="1294" alt="Image" src="https://github.com/user-attachments/assets/4cfe921e-4887-4acb-a7cf-a7dd36db00e2" /> Why not keeping the failed tentative in the logs? Can we generate an events / alerts for those failed conditions? (see example below) <img width="1208" height="261" alt="Image" src="https://github.com/user-attachments/assets/b56b2d41-f541-4a7b-9575-8e80fcefebff" /> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
