GaOrtiga commented on PR #12758: URL: https://github.com/apache/cloudstack/pull/12758#issuecomment-4134990086
> @JoaoJandre Thanks for the follow-ups. > > First, I think incremental backup is a valuable feature of the solution, but it may not be ideal to highlight it in the title. It is, or will be supported by most backup solutions. > > Second, it would be better to emphasize that secondary storage is used as the backup repository. > > With that in mind, a title such as “KVM VM Backup on Secondary Storage” could be more appropriate I think. I agree that "Incremental" doesn't need to be in the title, but I don't think "Secondary Storage" needs to be in it either. The title doesn't need to include every design choice, only the overall goal of the feature (in this case KVM backups). As long as both `KVM` and `backup` are in it, and the rest of the title does not include anything misleading, the title is correct from a technical standpoint and whether it's better or worse is just a matter of personal opinion. @weizhouapache Do you find any of the options provided in the list acceptable or is the missing "Secondary Storage" a deal breaker? @JoaoJandre Between the options in the list, I personally like `#5` and `#6` (KVM Backup Engine/Service). -- 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]
