cv4pve-barc
cv4pve-barc copied to clipboard
[Discussion] Remove Snapshot merging / legacy Code?
Since there are currently two approaches in the Codebase, I want to discuss if you want to keep the ability to merge snapshots into each other. I personally wouldn't dare to go on and take one Base Snapshot in a year, and then only merge onto it. And: There's no way to merge into the Basecopy, so the volatile data would accumulate in the first Differential file after the Basecopy.
what do you propose?
Add option --disable-merge-snapshot to disable merge.
There's no need to do, because it won't merge snapshots in --renew mode. in --keep mode it does behave like it always did.
What idea do you have for this?
The idea was to compeltely remove all merge logic since it you might not detect if something goes wrong until you try to restore stuff. For now let's keep it in, and keep backwards compatibility..
Ok for me good idea remove all merge logic