----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22631/#review46452 -----------------------------------------------------------
Hi Punith, Just wanted to let you know that I have not forgotten about this review. I plan to check the code in once I finish up something else I'm working on over the next day or so. Thanks! Mike - Mike Tutkowski On June 17, 2014, 9:28 a.m., punith s wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22631/ > ----------------------------------------------------------- > > (Updated June 17, 2014, 9:28 a.m.) > > > Review request for cloudstack, amit kumar das, edison su, and Mike Tutkowski. > > > Repository: cloudstack-git > > > Description > ------- > > currently managed storage only supports iscsi protocol. > > hence now adding support for nfs protocol and also resizing the vdi to the SR > size, hence fully utilizing the SR space in managed storage. > > to support nfs, the getNfsSR method's signature has to changed similar to > existing getIscsiSR method. > > SR will be created based on the volume's protocoltype. > > jira id - CLOUDSTACK-6916 > > > Diffs > ----- > > api/src/com/cloud/agent/api/to/DiskTO.java 5901367 > > engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java > eea931e > > plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java > 9979802 > server/src/com/cloud/storage/VolumeApiServiceImpl.java 9b034e9 > > Diff: https://reviews.apache.org/r/22631/diff/ > > > Testing > ------- > > system testing done. > > > Thanks, > > punith s > >
