Skip to content

Conversation

@blackpiglet
Copy link
Contributor

@blackpiglet blackpiglet commented Jun 24, 2025

Don't add VSClass in the additionalItems when it's empty.

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #9034

Please indicate you've done the following:

@blackpiglet blackpiglet force-pushed the 9034_fix branch 2 times, most recently from b39be24 to b01e071 Compare June 24, 2025 07:03
@blackpiglet blackpiglet changed the title Remove depedency with VolumeSnapshotClass in DataUpload. Remove dependency with VolumeSnapshotClass in DataUpload. Jun 24, 2025
@blackpiglet blackpiglet force-pushed the 9034_fix branch 2 times, most recently from d336b3a to 753b17f Compare June 24, 2025 07:30
@blackpiglet blackpiglet marked this pull request as ready for review June 24, 2025 07:32
@github-actions github-actions bot requested review from reasonerjt and sseago June 24, 2025 07:32
@codecov
Copy link

codecov bot commented Jun 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@37a22a3). Learn more about missing BASE report.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9040   +/-   ##
=======================================
  Coverage        ?   60.29%           
=======================================
  Files           ?      378           
  Lines           ?    42729           
  Branches        ?        0           
=======================================
  Hits            ?    25763           
  Misses          ?    15417           
  Partials        ?     1549           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

kaovilai
kaovilai previously approved these changes Jun 25, 2025
Don't add VSClass in the additionalItems when it's empty.

Signed-off-by: Xun Jiang <[email protected]>
Signed-off-by: xun.jiang <[email protected]>
@sseago sseago merged commit b0b5cc4 into vmware-tanzu:main Jun 25, 2025
46 checks passed
@MaloLelandais MaloLelandais mentioned this pull request Jul 28, 2025
3 tasks
MaloLelandais pushed a commit to MaloLelandais/velero that referenced this pull request Jul 28, 2025
…zu#9040)

Don't add VSClass in the additionalItems when it's empty.

Signed-off-by: Xun Jiang <[email protected]>
Signed-off-by: xun.jiang <[email protected]>
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.

Make sure data mover backup can work without VolumeSnapshotClass

4 participants