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

[cherry-pick][main]Make Windows build skip BlockMode code. #6989

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

blackpiglet
Copy link
Contributor

@blackpiglet blackpiglet commented Oct 20, 2023

PVC block mode backup and restore introduced some OS specific system calls. Those calls are not available for Windows, so add both non Windows version and Windows version code, and return error for block mode on the Windows platform.

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:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

PVC block mode backup and restore introduced some OS specific
system calls. Those calls are not available for Windows, so
add both non Windows version and Windows version code, and
return error for block mode on the Windows platform.

Signed-off-by: Xun Jiang <[email protected]>
@blackpiglet blackpiglet force-pushed the support_windows_build_main branch from 018c067 to ecc6e16 Compare October 20, 2023 11:39
@blackpiglet blackpiglet changed the title Make Windows build skip BlockMode code. [cherry-pick][main]Make Windows build skip BlockMode code. Oct 20, 2023
@blackpiglet blackpiglet marked this pull request as ready for review October 20, 2023 11:40
@codecov
Copy link

codecov bot commented Oct 20, 2023

Codecov Report

Merging #6989 (ecc6e16) into main (fd8350f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #6989   +/-   ##
=======================================
  Coverage   61.05%   61.05%           
=======================================
  Files         251      251           
  Lines       26845    26845           
=======================================
  Hits        16389    16389           
  Misses       9303     9303           
  Partials     1153     1153           
Files Coverage Δ
pkg/uploader/kopia/block_backup.go 30.00% <ø> (ø)
pkg/uploader/kopia/block_restore.go 21.27% <ø> (ø)

@ywk253100 ywk253100 merged commit 61d333a into vmware-tanzu:main Oct 24, 2023
26 of 27 checks passed
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