jmsperu commented on PR #12900: URL: https://github.com/apache/cloudstack/pull/12900#issuecomment-5396229571
Thanks @DaanHoogland. Pushed 3fa88ea addressing all of your comments plus the two remaining Copilot ones. The multi-management-server question is answered in-thread: runs were already serialised by a cluster-wide GlobalLock, and each server now writes to its own `infra-backup/<ms-hostname>/` directory with retention counted per server, so configs and certificates of different servers never overwrite or rotate each other out. InfrastructureBackupTaskTest is at 14 tests, including two new ones for the per-server layout and per-server retention. Now that #13074 has landed, could this one go back into the queue? -- 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]
