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

in GetAllBackups don't error if single backup is unreadable #66

Merged
merged 1 commit into from
Sep 5, 2017

Conversation

skriss
Copy link
Contributor

@skriss skriss commented Sep 1, 2017

Signed-off-by: Steve Kriss steve@heptio.com

fixes #53

@skriss skriss added this to the v0.4.0 milestone Sep 1, 2017
@ncdc
Copy link
Contributor

ncdc commented Sep 5, 2017

@skriss pls rebase

Signed-off-by: Steve Kriss <steve@heptio.com>
@skriss
Copy link
Contributor Author

skriss commented Sep 5, 2017

@ncdc done

@ncdc ncdc merged commit 4e59685 into vmware-tanzu:master Sep 5, 2017
@skriss skriss deleted the getallbackups-fix branch September 5, 2017 17:08
@ncdc ncdc mentioned this pull request Sep 6, 2017
4 tasks
alromeros pushed a commit to alromeros/velero that referenced this pull request Oct 25, 2024
* Handle correctly unfinished DataVolume

When DataVolume is unfinished it is included in the backup
together with corresponding PVC. In such case the pre-populated annotation
should not be present.

Signed-off-by: Bartosz Rybacki <brybacki@redhat.com>

* Skip PVC for DV in progress

Signed-off-by: Bartosz Rybacki <brybacki@redhat.com>

* Add test for DV clone in progress

Signed-off-by: Bartosz Rybacki <brybacki@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BackupService.GetAllBackups should be forgiving when there's partial data
2 participants