| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Added a warning about KVM VM metadata preservation when unmanaged.
|
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. |
Sorry, something went wrong.
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. |
Sorry, something went wrong.
|
@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. |
Sorry, something went wrong.
There was a problem hiding this comment.
Sorry, mistakently approved this. @pavanaravapalli can you please check my comment.
Sorry, something went wrong.
| - The Instance must be a VMware Instance (as of CloudStack 4.19, it's also possible to unmanage a KVM-based Instances) | ||
|
|
||
| .. warning:: | ||
| Starting with the 4.22.0 release, when a KVM VM is unmanaged, its infrastructure metadata (zone, pod, cluster, and host) in the libvirt VM XML is preserved. |
There was a problem hiding this comment.
This information is already document in this PR https://github.com/apache/cloudstack-documentation/pull/562/files
is this something else ?
Sorry, something went wrong.
There was a problem hiding this comment.
This is different from https://github.com/apache/cloudstack-documentation/pull/562/files.
It focuses on retaining CloudStack metadata on libvirt XML even after the VM resource is unmanaged from ACS.
Sorry, something went wrong.
@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 |
Sorry, something went wrong.
@sureshanaparti |
Sorry, something went wrong.
@pavanaravapalli correct, version should be 4.22.1. no need of separate PR. |
Sorry, something went wrong.
| - The Instance must be a VMware Instance (as of CloudStack 4.19, it's also possible to unmanage a KVM-based Instances) | ||
|
|
||
| .. warning:: | ||
| Starting with the 4.22.0 release, when a KVM VM is unmanaged, its infrastructure metadata (zone, pod, cluster, and host) in the libvirt VM XML is preserved. |
There was a problem hiding this comment.
| Starting with the 4.22.0 release, when a KVM VM is unmanaged, its infrastructure metadata (zone, pod, cluster, and host) in the libvirt VM XML is preserved. | |
| Starting with the 4.22.1 release, when a KVM instance is unmanaged, the infrastructure metadata (zone, pod, cluster, and host details) is preserved in the libvirt VM XML. |
Note: if any change in the retaining behavior, mainly clearing the metadata while unmanaging instance is handled before 4.22.1, this warning message is not needed and has to be reverted. cc @DaanHoogland
Sorry, something went wrong.
There was a problem hiding this comment.
@sureshanaparti
Suresh, prior to 4.22.1, the preserved libvirt XML behavior was meant to support non-transient domains required for unmanaged VMs. During that process, no CloudStack metadata was stored — it only started being preserved after the libvirt metadata enhancement. I think this is worth mentioning in the release notes. Please correct me if I’m mistaken.
Sorry, something went wrong.
There was a problem hiding this comment.
@sureshanaparti
Changes applied as suggested — ready for review.
Sorry, something went wrong.
There was a problem hiding this comment.
@sureshanaparti Suresh, prior to 4.22.1, the preserved libvirt XML behavior was meant to support non-transient domains required for unmanaged VMs. During that process, no CloudStack metadata was stored — it only started being preserved after the libvirt metadata enhancement. I think this is worth mentioning in the release notes. Please correct me if I’m mistaken.
fine, we can mention that. I'm making a point that if any PR addresses the cleanup of the libvirt metadata in 4.22.1 itself, this note in the doc has to revisited.
Sorry, something went wrong.
|
@sureshanaparti a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. |
Sorry, something went wrong.
|
Verified the doc here: https://cloudstack-documentation--595.org.readthedocs.build/en/595/adminguide/virtual_machines/importing_unmanaging_vms.html
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Added a warning about KVM VM metadata preservation when unmanaged.
From the enhancement PR apache/cloudstack#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/