abh1sar commented on PR #11884: URL: https://github.com/apache/cloudstack/pull/11884#issuecomment-3431811086
@daviftorres the code doesn't look like it is raising any events for the backup failure. I only see the change to not delete the backup entry in case of any error. Is this a work in progress? IMO, just keeping the backups db entry without any other information on the failure doesn't help much. If this could be tied with events so that the Backup in error state has an event which shows the cause of failure (specially for scheduled backups), would be a great addition. I have shared more on my thought process here : https://github.com/apache/cloudstack/issues/11727#issuecomment-3431779320 -- 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]
