-
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
Update CSIVolumeSnapshotsCompleted in backup's status and the metric #7184
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #7184 +/- ##
==========================================
+ Coverage 61.83% 61.90% +0.07%
==========================================
Files 261 262 +1
Lines 28078 28175 +97
==========================================
+ Hits 17362 17442 +80
- Misses 9495 9499 +4
- Partials 1221 1234 +13 ☔ View full report in Codecov by Sentry. |
@kaovilai |
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 is cool, didnt know operations have resourceidentifier
Update CSIVolumeSnapshotsCompleted in backup's status and the metric during backup finalize stage according to async operations content. Signed-off-by: Xun Jiang <jxun@vmware.com>
Update CSIVolumeSnapshotsCompleted in backup's status and the metric during the backup finalize stage according to async operations content.
Thank you for contributing to Velero!
Please add a summary of your change
Does your change fix a particular issue?
Fixes #7163
Please indicate you've done the following:
/kind changelog-not-required
as a comment on this pull request.site/content/docs/main
.