sureshanaparti commented on code in PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#discussion_r2149800340


##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = 
"host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = 
"host.instance.conversion";

Review Comment:
   @nvazquez this detail - 'host.instance.conversion' would be redundant when 
virt-v2v version is added in the details. so, can you remove this detail in 
upgrade path or when 'host.virtv2v.version' details is added?
   
   return host supports conversion when virtv2v version is there in detail.



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to