-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20357/
-----------------------------------------------------------

(Updated April 16, 2014, 1:31 a.m.)


Review request for cloudstack.


Changes
-------

I changed the iteration method to be more proper, using alternating .next() and 
.prev() calls was hacky and prone to malfunction.


Bugs: CLOUDSTACK-5907 and CLOUDSTACK-6396
    https://issues.apache.org/jira/browse/CLOUDSTACK-5907
    https://issues.apache.org/jira/browse/CLOUDSTACK-6396


Repository: cloudstack-git


Description
-------

Fix for RBD (and potentially CLVM) volumes w/ KVM incorrectly identified as OVM 
volumes (not able to snapshot)


Diffs (updated)
-----

  server/src/com/cloud/api/ApiDBUtils.java 67e47f7 

Diff: https://reviews.apache.org/r/20357/diff/


Testing
-------

Applied on otherwise clean 4.4 branch and tested in a live testing environment 
with KVM hypervisors and RBD primary storage pool that would otherwise identify 
as OVM. No errors and no weird behavior. Just the expected result.


Thanks,

Tanner Danzey

Reply via email to