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
don't error during backup when additional items returned by plugin don't exist #2595
Conversation
Signed-off-by: Steve Kriss <krisss@vmware.com>
Signed-off-by: Steve Kriss <krisss@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this! I think this covers the case where a cluster's state isn't what an administrator would expect, while also letting Velero finish a backup and signalling something needs to be looked at.
d.Printf("Errors:\t%d\n", status.Errors) | ||
d.Printf("Warnings:\t%d\n", status.Warnings) | ||
} | ||
d.Println() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this change. It is a good step towards unifying restore and backup describe output. (cc @sseago)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I tried to be consistent between the two, it's a lot easier for users that way.
…n't exist (vmware-tanzu#2595) * log a warning instead of erroring if additional item can't be found Signed-off-by: Steve Kriss <krisss@vmware.com> * always show backup warning/error count in get/describe Signed-off-by: Steve Kriss <krisss@vmware.com> * changelog Signed-off-by: Steve Kriss <krisss@vmware.com>
* setup ci in github actions Signed-off-by: Ashish Amarnath <ashisham@vmware.com> * remove travis config Signed-off-by: Ashish Amarnath <ashisham@vmware.com> * Add changelog for v1.4.2 Signed-off-by: Nolan Brubaker <brubakern@vmware.com> * update build status badge on readme Signed-off-by: Ashish Amarnath <ashisham@vmware.com> * don't error during backup when additional items returned by plugin don't exist (#2595) * log a warning instead of erroring if additional item can't be found Signed-off-by: Steve Kriss <krisss@vmware.com> * always show backup warning/error count in get/describe Signed-off-by: Steve Kriss <krisss@vmware.com> * changelog Signed-off-by: Steve Kriss <krisss@vmware.com> * Update changelogs Signed-off-by: Nolan Brubaker <brubakern@vmware.com> Co-authored-by: Ashish Amarnath <ashisham@vmware.com> Co-authored-by: Steve Kriss <krisss@vmware.com>
closes #1878
closes #2269
This PR changes the backup code to log a warning instead of returning an error if an additional item returned by a backup item action plugin can't be found in the Kubernetes API. It also changes
velero backup get/describe
to always display warning/error count, rather than doing it selectively based on phase.