-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add velero annotations to all necessary deployments #45
Comments
One thing I've noticed is descheduler causes issues with Velero during backups if it re-schedules a pod during the backup phase, Velero offers a plugin system but its pre-backup and doesn't seem to do post-backup so adding a critical annotation to the pod doesn't seem plausable, got any thoughts on this ? |
After lots of debugging, it becomes clear that is it not possible to leverage velero/restic to backup volumes from annotated pods which are running on non-amd64 (i.e. arm) nodes. The reason for this is that the 'restic' daemonset needs to schedule pods on arm-based nodes in order for restic backup/restores to work properly. There are currently no multiarch velero images available. This issue and this PR suggest that it may soon? be possible but not today. This is sad :( |
Annotations are all created where possible. Also successfully tested velero backups against these annotations. Closing this issue. |
@billimek should the podAnnotation label value for velero contain the name of the actual PVC instead of the helm variable? For example values:
image:
repository: linuxserver/radarr
...
podAnnotations:
backup.velero.io/backup-volumes: radarr-config |
I think it's supposed to be the name of the volume in the deployment/pod. In the case of radarr, this is I see what you mean about using the name of the actual PVC and not the volume name. Indeed when I look in the velero restic backup logs, I see,
Maybe it is 'looking up' the PVC name based on the deployment volume definition? I certainly see the data being backed up when I look in the restic snapshot,
|
You're right, it seems to be smarter than I am. I thought I read it was the PVC name 🤦♂ |
Also, backing up the nzbget config is something to add. It only stores the |
See #15 for some background.
Deployments to annotate with velero restic backup stuff:
hass-mysqlnextcloudlokialertmanagerchronografprometheusThe text was updated successfully, but these errors were encountered: