sureshanaparti commented on code in PR #595:
URL: 
https://github.com/apache/cloudstack-documentation/pull/595#discussion_r2517322088


##########
source/adminguide/virtual_machines/importing_unmanaging_vms.rst:
##########
@@ -360,6 +360,10 @@ The API has the following preconditions:
 - The Instance state must be 'Running’ or ‘Stopped’
 - 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.

Review 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.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to