Skip to content
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

Upload progress monitoring for plugins after backup completion #3533

Open
dsu-igeek opened this issue Mar 5, 2021 · 5 comments
Open

Upload progress monitoring for plugins after backup completion #3533

dsu-igeek opened this issue Mar 5, 2021 · 5 comments
Assignees
Labels
Area/Plugins Issues related to plugin infra/internal plugins Enhancement/User End-User Enhancement to Velero Epic Icebox We see the value, but it is not slated for the next couple releases. kind/requirement
Projects

Comments

@dsu-igeek
Copy link
Contributor

Describe the problem/challenge you have

Volume snapshotter plug-in are used by Velero to take snapshots of persistent volume contents.
Depending on the underlying storage system, those snapshots may be available to use immediately,
they may be uploaded to stable storage internally by the plug-in or they may need to be uploaded after
the snapshot has been taken. We would like for Velero to continue on to the next part of the backup as quickly
as possible but we would also like the backup to not be marked as complete until it is a usable backup. We'd also
eventually like to bring the control of upload under the control of Velero and allow the user to make decisions
about the ultimate destination of backup data independent of the storage system they're using.

Describe the solution you'd like
Enable monitoring of uploads that are in-progress without blocking backups from starting.

Vote on this issue!

This is an invitation to the Velero community to vote on issues, you can see the project's top voted issues listed here.
Use the "reaction smiley face" up to the right of this comment to vote.

  • 👍 for "The project would be better with this feature added"
  • 👎 for "This feature will not enhance the project in a meaningful way"
@dsu-igeek dsu-igeek added P1 - Important Enhancement/User End-User Enhancement to Velero Area/Plugins Issues related to plugin infra/internal plugins labels Mar 5, 2021
@dsu-igeek dsu-igeek added this to the v1.7.0 milestone Mar 5, 2021
@dsu-igeek dsu-igeek self-assigned this Mar 5, 2021
@dsu-igeek dsu-igeek added this to To do in v1.7.0 via automation Mar 5, 2021
@dsu-igeek dsu-igeek moved this from New to Groomed in Velero Support Board Mar 8, 2021
@dsu-igeek dsu-igeek added the Epic label Mar 10, 2021
@dsu-igeek dsu-igeek removed this from Groomed in Velero Support Board Mar 10, 2021
@dsu-igeek dsu-igeek removed this from To do in v1.7.0 Apr 21, 2021
@dsu-igeek dsu-igeek added this to To do in Velero 1.7.0 via automation Apr 21, 2021
@dsu-igeek dsu-igeek moved this from To do to In progress in Velero 1.7.0 Apr 30, 2021
@dsu-igeek dsu-igeek removed this from In progress in Velero 1.7.0 Sep 8, 2021
@dsu-igeek dsu-igeek modified the milestones: v1.7.0, v1.8.0 Sep 8, 2021
@dsu-igeek dsu-igeek modified the milestones: v1.8.0, 1.9.0 Dec 21, 2021
@dsu-igeek dsu-igeek added this to To do in v1.9.0 via automation Dec 21, 2021
@dsu-igeek dsu-igeek removed this from To Do in v1.8.0 Dec 21, 2021
@reasonerjt reasonerjt added kind/requirement Reviewed Q2 2021 Area/Plugins Issues related to plugin infra/internal plugins Epic Enhancement/User End-User Enhancement to Velero and removed Enhancement/User End-User Enhancement to Velero Epic Area/Plugins Issues related to plugin infra/internal plugins Reviewed Q2 2021 labels May 20, 2022
@reasonerjt reasonerjt modified the milestones: 1.9.0, 1.10 Jun 9, 2022
@reasonerjt reasonerjt removed this from the 1.10 milestone Oct 12, 2022
@reasonerjt reasonerjt added the Icebox We see the value, but it is not slated for the next couple releases. label Oct 12, 2022
@reasonerjt
Copy link
Contributor

Moving this to icebox as we have decided to choose the alternative approach to track the progress: #5108

@arianitu
Copy link

@reasonerjt I just want to confirm that issue is not resolved in all released versions of velero. There are a lot of tasks that are closed that run into this particular issue (specifically restores failing because snapshots are not ready yet.)

We have patched our own version of velero that handles this case, I want to confirm that we should continue to maintain our own patched version until this is fixed via #5108?

Again, I want to mention for others that this issue may not be completely relevant if you're using CSI snapshots because they handle this case, here is my comment from a few years ago:

If anyone is curious, we talked to the Google folks managing the gke-csi diver and the CSI driver will handle this case for you. This may be relevant for you nrb since I don't think velero needs to wait for snapshots to be ready, so I guess you guys do not need to do anything.
Basically, the PVC provisioner will retry forever until the VolumeSnapshot is readyToUse. So in theory, the EBS and GKE volume snapshotter bugs are no longer relevant if you use the CSI snapshot.

I am unsure if CSI snapshots support restoring to another namespace, but our hope is one day to move to CSI snapshots to solve this issue.

Cheers,

@jglick
Copy link

jglick commented Jul 12, 2023

@blackpiglet
Copy link
Contributor

@jglick
Yeah. This design was already implemented in Velero v1.11.0.

@sseago
Copy link
Collaborator

sseago commented Jul 13, 2023

@blackpiglet @jglick Just a note, we've not yet implemented this for volumesnapshotter plugins (native aws, gcp, azure snapshots), but it is available for BackupItemAction/RestoreItemAction plugins as of 1.11, and will be used for the CSI plugin (and the new datamover) in 1.12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/Plugins Issues related to plugin infra/internal plugins Enhancement/User End-User Enhancement to Velero Epic Icebox We see the value, but it is not slated for the next couple releases. kind/requirement
Projects
v1.9.0
To do
Development

No branches or pull requests

8 participants