Only get VolumeSnapshotClass when DataUpload exists.

[blackpiglet]

Thank you for contributing to Velero!

Please add a summary of your change

Does your change fix a particular issue?

Fixes #7823

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.

[codecov]

Codecov Report

Attention: Patch coverage is 89.18919% with 8 lines in your changes missing coverage. Please review.

Project coverage is 59.09%. Comparing base (8fde4a0) to head (1d9fbcf).
Report is 36 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/volume/volumes_information.go	89.18%	6 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7974      +/-   ##
==========================================
+ Coverage   59.02%   59.09%   +0.07%     
==========================================
  Files         364      364              
  Lines       30272    30340      +68     
==========================================
+ Hits        17867    17929      +62     
- Misses      10959    10968       +9     
+ Partials     1446     1443       -3     

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

[sseago]

Don't we also need VSClass for CSI snapshot backups without datamover?

[blackpiglet]

Don't we also need VSClass for CSI snapshot backups without datamover?

Yes, but for the CSI plugin scenario, because the VolumeSnapshot-related CRs are persisted, so VolumeSnapshotClasses are already passed in by parameters.

velero/internal/volume/volumes_information.go

Lines 321 to 340 in 1ec52be

	func (v *BackupVolumesInformation) Result(
	csiVolumeSnapshots []snapshotv1api.VolumeSnapshot,
	csiVolumeSnapshotContents []snapshotv1api.VolumeSnapshotContent,
	csiVolumesnapshotClasses []snapshotv1api.VolumeSnapshotClass,
	crClient kbclient.Client,
	logger logrus.FieldLogger,
	) []*BackupVolumeInfo {
	v.logger = logger
	v.crClient = crClient
	v.volumeSnapshots = csiVolumeSnapshots
	v.volumeSnapshotContents = csiVolumeSnapshotContents
	v.volumeSnapshotClasses = csiVolumesnapshotClasses
	v.generateVolumeInfoForSkippedPV()
	v.generateVolumeInfoForVeleroNativeSnapshot()
	v.generateVolumeInfoForCSIVolumeSnapshot()
	v.generateVolumeInfoFromPVB()
	v.generateVolumeInfoFromDataUpload()
	return v.volumeInfos

[blackpiglet]

@kaovilai @shubham-pampattiwar
Modified according to comments. PTAL.