-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update Velero to support restic restores of pods managed by DeploymentConfigs #340
Comments
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
Rotten issues close after 30d of inactivity. Reopen the issue by commenting /close |
@openshift-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
There has been a longstanding upstream issue with taking restic backups/restores of volumes which are mounted via pods that are owned by DeploymentConfigs. What ends up happening is at restore time, Velero restores the pod that mounts the volume and the DeploymentConfig which manages the pod.
This causes problems as the deploymentconfig adopts the restored pod, and redeploys a new version while terminating the old one. This is a problem because the restored pod contains the initContainer which waits for the restic restore to complete. Without this pod running with the initContainer, the restic restore process fails.
Related issues:
vmware-tanzu/velero#2073
vmware-tanzu/velero#1981
If we decide that we cannot change the upstream code to support this, then we will want to at a minimum document the process of creating 2 restores (first restore just pods+pvcs, second restore everything else) to workaround this.
The text was updated successfully, but these errors were encountered: