----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10124/#review18388 -----------------------------------------------------------
+1 Looks good, should fix the issue. Waiting for Edison to take a final look at it and give a ship it. Will assign the issue to Edison on JIRA. - Rohit Yadav On March 25, 2013, 8:41 p.m., Brian Spindler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10124/ > ----------------------------------------------------------- > > (Updated March 25, 2013, 8:41 p.m.) > > > Review request for cloudstack and edison su. > > > Description > ------- > > Fix for deleting all slashes from the storage path instead of just the first > slash. > > > This addresses bug CLOUDSTACK-1803. > > > Diffs > ----- > > > plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java > 2991574 > > Diff: https://reviews.apache.org/r/10124/diff/ > > > Testing > ------- > > Ubuntu 12.04, turned local storage on, added a physical host. > > > Thanks, > > Brian Spindler > >
