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

Issue 6964: get volume size from source PVC if it is invalid in VS #6976

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

Lyndon-Li
Copy link
Contributor

It is a valid case that the Status.RestoreSize field in VolumeSnapshot is not set, if so, get the volume size from the source PVC to create the backup PVC

@Lyndon-Li Lyndon-Li marked this pull request as ready for review October 19, 2023 03:51
@github-actions github-actions bot requested review from reasonerjt and sseago October 19, 2023 03:51
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Merging #6976 (329c128) into main (ad114f8) will increase coverage by 0.26%.
Report is 10 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #6976      +/-   ##
==========================================
+ Coverage   60.78%   61.05%   +0.26%     
==========================================
  Files         250      251       +1     
  Lines       26629    26845     +216     
==========================================
+ Hits        16187    16389     +202     
- Misses       9293     9303      +10     
- Partials     1149     1153       +4     
Files Coverage Δ
pkg/controller/data_upload_controller.go 68.50% <100.00%> (+0.05%) ⬆️
pkg/exposer/csi_snapshot.go 81.49% <100.00%> (+0.43%) ⬆️

... and 9 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants