This looks pretty good!

Would you have the time to add the Recovery from VolumeSnapshots functionality as well?

It wouldn't want to add this as a backup method without adding a recovery path, as that could potentially make it more difficult to recover from.

Thanks @itay-grudev! I've added support for recovery from snapshots and tested this in a GKE environment.

The recovery config docs also mention being able to use a PVC as a recovery mechanism with the same block on the cluster, but that failed when I tested it, otherwise I would have added that too. It might be a nice to have when that capability stabilises in the operator.

Please let me know if there's anything else that could make this better.

Hi @itay-grudev please let me know what blockers if any there are to merging, we are really keen to start using this when it is released :)

I not think force user to decide between s3 and snapshot is good idea actually. Provider allow to utilize both S3 and snapshots, why we in Helm should cut off this capabilities, I see this option as: #359

I not think force user to decide between s3 and snapshot is good idea actually. Provider allow to utilize both S3 and snapshots, why we in Helm should cut off this capabilities, I see this option as: #359

Hi @dragoangel, I like your PR a lot. Whatever the maintainers prefer I am happy with, I just really want some volume snapshot capability. I've posted a review on your PR with two suggested changes. @itay-grudev please feel free to close this PR if you are going to merge #359 .