cloudstack-documentation icon indicating copy to clipboard operation
cloudstack-documentation copied to clipboard

Document KVM VM metadata handling in unmanage process

Open pavanaravapalli opened this issue 1 month ago • 6 comments

Added a warning about KVM VM metadata preservation when unmanaged.

From the enhancement PR https://github.com/apache/cloudstack/pull/11061, as per discussion concluded to explicitly call out the VM behavior for unmanaged KVM VMs regarding the retention of libvirt metadata.

This PR adds a note to the documentation warning users that CloudStack does not cleanup the libvirt metadata for unamanged VMs, leaving it for user's choice.


📚 Documentation preview 📚: https://cloudstack-documentation--595.org.readthedocs.build/en/595/

pavanaravapalli avatar Nov 10 '25 14:11 pavanaravapalli

Correct me if I'm wrong, I think it may be phrased a bit shorter:

When a KVM VM is unmanaged, its metadata in the libvirt domain XML remains unchanged.
This information is left for the user’s discretion allowing them to either retain it or remove it themselves.

I specifically omitted the 4.22 version because the documentation is also versioned if I am not mistaken. So the older documentation versions will not contain this line.

phsm avatar Nov 10 '25 15:11 phsm

Correct me if I'm wrong, I think it may be phrased a bit shorter:

When a KVM VM is unmanaged, its metadata in the libvirt domain XML remains unchanged.
This information is left for the user’s discretion allowing them to either retain it or remove it themselves.

I specifically omitted the 4.22 version because the documentation is also versioned if I am not mistaken. So the older documentation versions will not contain this line.

@phsm

It's marked for 4.22.0 milestone Last time when i checked this PR https://github.com/apache/cloudstack/pull/11061. How ever this Doc PR goes to main branch.

pavanaravapalli avatar Nov 11 '25 07:11 pavanaravapalli

@pavanaravapalli @phsm, I think it does not hurt to leave the version in as a lot of people will read the “latest” documentation without caring to select a specific version.

If otherwise fine, I’m merging.

DaanHoogland avatar Nov 11 '25 08:11 DaanHoogland

Correct me if I'm wrong, I think it may be phrased a bit shorter:

When a KVM VM is unmanaged, its metadata in the libvirt domain XML remains unchanged.
This information is left for the user’s discretion allowing them to either retain it or remove it themselves.

I specifically omitted the 4.22 version because the documentation is also versioned if I am not mistaken. So the older documentation versions will not contain this line.

@phsm

It's marked for 4.22.0 milestone Last time when i checked this PR apache/cloudstack#11061. How ever this Doc PR goes to main branch.

@pavanaravapalli @phsm the PR https://github.com/apache/cloudstack/pull/11061 is merged after 4.22.0 release (earlier it was part of 4.22.0 milestone, but not merged before the release), so the doc PR should target 4.22 (as this should go with 4.22.1 release notes later, and also merged forward to main). cc @DaanHoogland

sureshanaparti avatar Nov 11 '25 09:11 sureshanaparti

Correct me if I'm wrong, I think it may be phrased a bit shorter:

When a KVM VM is unmanaged, its metadata in the libvirt domain XML remains unchanged.
This information is left for the user’s discretion allowing them to either retain it or remove it themselves.

I specifically omitted the 4.22 version because the documentation is also versioned if I am not mistaken. So the older documentation versions will not contain this line.

@phsm It's marked for 4.22.0 milestone Last time when i checked this PR apache/cloudstack#11061. How ever this Doc PR goes to main branch.

@pavanaravapalli @phsm the PR apache/cloudstack#11061 is merged after 4.22.0 release (earlier it was part of 4.22.0 milestone, but not merged before the release), so the doc PR should target 4.22 (as this should go with 4.22.1 release notes later, and also merged forward to main). cc @DaanHoogland

@sureshanaparti Should I update the version in the docs from 4.22.0 → 4.22.1? Will this get merged into both main and 4.22.1, or do I need to open separate PRs?

pavanaravapalli avatar Nov 11 '25 15:11 pavanaravapalli

@sureshanaparti Should I update the version in the docs from 4.22.0 → 4.22.1? Will this get merged into both main and 4.22.1, or do I need to open separate PRs?

@pavanaravapalli correct, version should be 4.22.1. no need of separate PR.

sureshanaparti avatar Nov 12 '25 07:11 sureshanaparti