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

[Hyper-v platform] Delete disks on deletion of VM #3616

Merged
merged 2 commits into from
Feb 4, 2025

Conversation

SRIKKANTH
Copy link
Collaborator

When a VM is deleted, its attached disks are not automatically removed, which can lead to disk space issues. This change will address the problem by ensuring that both the OS and data disks attached to the VM are deleted across all storage controllers on VM deletion.

@SRIKKANTH
Copy link
Collaborator Author

@LiliDeng please let me know if anything is needed on this.

@SRIKKANTH SRIKKANTH force-pushed the smyakam/hyperv/cleanup_improvements_26_01_2025 branch from 85ca87b to 0ed59d4 Compare February 2, 2025 19:24
LiliDeng
LiliDeng previously approved these changes Feb 4, 2025
@LiliDeng
Copy link
Collaborator

LiliDeng commented Feb 4, 2025

@SRIKKANTH please fix the conflict.

When a VM is deleted, its attached disks are not automatically removed, which can lead to disk space issues. This change will address the problem by ensuring that both the OS and data disks attached to the VM are deleted across all storage controllers on VM deletion.
@SRIKKANTH SRIKKANTH force-pushed the smyakam/hyperv/cleanup_improvements_26_01_2025 branch from 0ed59d4 to daf76af Compare February 4, 2025 06:10
@SRIKKANTH
Copy link
Collaborator Author

@SRIKKANTH please fix the conflict.

@LiliDeng the conflicts are resolved.

@LiliDeng LiliDeng merged commit adcae19 into main Feb 4, 2025
45 checks passed
@LiliDeng LiliDeng deleted the smyakam/hyperv/cleanup_improvements_26_01_2025 branch February 4, 2025 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants