Move member VolumeSnapshotContents creation to the sidecar#1288
Move member VolumeSnapshotContents creation to the sidecar#1288leonardoce wants to merge 1 commit intokubernetes-csi:masterfrom
Conversation
This allows using the information from the CreateGroupSnapshot RPC call to fill the status of VolumeSnapshotContents, and to set the restoreSize field even when the ListSnapshots GRPC call is not implemented.
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: leonardoce The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
With this PR, member VolumeSnapshot objects have the What follows is an example using I also tested it using the CSI host path driver with the ListSnapshots GRPC call disabled. |
What type of PR is this?
/kind bug
What this PR does / why we need it:
It allows the use of information from the CreateGroupSnapshot RPC call to fill out the status of VolumeSnapshotContents and set the restoreSize field even when the ListSnapshots GRPC call is not implemented.
Which issue(s) this PR fixes:
Fixes #1271
Special notes for your reviewer:
A subsequent PR will thread a context to the function where the member VolumeSnapshotContents are created.
Does this PR introduce a user-facing change?: