cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Fang Wang" <>
Subject Re: Review Request 13547: Extend listCapabilities API support to verify KVM snapshot enabled or not
Date Wed, 14 Aug 2013 00:46:09 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Aug. 14, 2013, 12:46 a.m.)

Review request for cloudstack and Venkata Siva Vijayendra Bhamidipati.

Bugs: cloudstack-4308

Repository: cloudstack-git


For 6.3 KVM, by default, snapshot is not supported. This is managed by a global setting KVM.snapshot.enabled.
UI needs the API support for listCapabilities response, to stop user to set recurring snapshot
when snapshot is not supported for KVM 6.3. 


  api/src/org/apache/cloudstack/api/command/user/config/ a30e26c 
  api/src/org/apache/cloudstack/api/response/ c2996f0 
  server/src/com/cloud/server/ 622e167 


Testing (updated)

Before the fix, listCapabilities does not have the KVMSnapshotEnabled field:
 "listcapabilitiesresponse": { 
        "capability": { 
            "securitygroupsenabled": false, 
            "cloudstackversion": "4.2.0-SNAPSHOT", 
            "userpublictemplateenabled": true, 
            "supportELB": "false", 
            "projectinviterequired": false, 
            "allowusercreateprojects": true, 
            "customdiskofferingmaxsize": 1024 

I tested now with my fix, it shows:
 "listcapabilitiesresponse" :  { "capability" : {"securitygroupsenabled":true,"cloudstackversion":"4.2.0-SNAPSHOT","userpublictemplateenabled":true,"supportELB":"false","projectinviterequired":false,"allowusercreateprojects":true,"customdiskofferingmaxsize":1024,"KVMsnapshotenabled":false}
<--- default value is false;

I then set the default value to TRUE on MS, and call listCapabilities command again:
{ "listcapabilitiesresponse" :  { "capability" : {"securitygroupsenabled":true,"cloudstackversion":"4.2.0-SNAPSHOT","userpublictemplateenabled":true,"supportELB":"false","projectinviterequired":false,"allowusercreateprojects":true,"customdiskofferingmaxsize":1024,"KVMsnapshotenabled":true}
}  <---- reflect the true value.

Verified the fix through the tests. 


Fang Wang

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message