Skip to content

Conversation

@mergify
Copy link

@mergify mergify bot commented Mar 10, 2025

Summary

The backupVolumeSnapshotClassName should be grayed out directly for third-party storage.

PR Checklists

  • Do we need to backport this PR change to the Harvester Dashboard?
    • Yes, the relevant PR is at:
  • Are backend engineers aware of UI changes?

Related Issue

[BUG][GUI] Should Allow set empty for backupVolumeSnapshotClassName in Setting csi-driver-config #7737

Test screenshot/video

  • Navigate to Advanced → Settings → edit csi-driver-config
  • Click Add, and the Backup Volume Snapshot Class Name field should be disabled
  • Select the appropriate option, save and ensure it saves correctly

Screenshot 2025-03-06 at 6 52 41 PM (2)

Extra technical notes summary

N/A


This is an automatic backport of pull request #1282 done by [Mergify](https://mergify.com).

Signed-off-by: Yi-Ya Chen <yiya.chen@suse.com>
(cherry picked from commit bcba390)
(cherry picked from commit e0613d5)
@mergify mergify bot added the Backport to v1.4 Backport PR target v1.4 label Mar 10, 2025
@houhoucoop houhoucoop merged commit 409a639 into release-harvester-v1.4 Mar 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport to v1.4 Backport PR target v1.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants