-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 warning message for volume snapshotter in data mover case. #6377
Add warning message for volume snapshotter in data mover case. #6377
Conversation
c2051cc
to
c8fd96e
Compare
Codecov Report
@@ Coverage Diff @@
## main #6377 +/- ##
==========================================
- Coverage 48.35% 48.35% -0.01%
==========================================
Files 227 227
Lines 23346 23349 +3
==========================================
Hits 11290 11290
- Misses 11194 11196 +2
- Partials 862 863 +1
|
pkg/backup/item_backupper.go
Outdated
@@ -458,6 +458,13 @@ func (ib *itemBackupper) takePVSnapshot(obj runtime.Unstructured, log logrus.Fie | |||
return nil | |||
} | |||
|
|||
// TODO: Snapshot data mover is only supported for CSI plugin scenario. |
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.
This should be added after if features.IsEnabled(velerov1api.CSIFeatureFlag)
, if the snapshot request has been handled by CSI, we will not need this warning
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. Modified.
Signed-off-by: Xun Jiang <jxun@vmware.com>
c8fd96e
to
a5c28ad
Compare
// TODO: Snapshot data mover is only supported for CSI plugin scenario by now. | ||
// Need to add a mechanism to choose running which plugin for resources. | ||
// After that, this warning can be removed. | ||
if boolptr.IsSetToTrue(ib.backupRequest.Spec.SnapshotMoveData) { |
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.
@blackpiglet Maybe also create an issue for this TODO.
Thank you for contributing to Velero!
Please add a summary of your change
Does your change fix a particular issue?
Fixes #(issue)
Please indicate you've done the following:
/kind changelog-not-required
as a comment on this pull request.site/content/docs/main
.