Is there a reason the commit message isn't what was in the review request? Did you use git am?
--David On Tue, Mar 26, 2013 at 8:38 PM, <edi...@apache.org> wrote: > Updated Branches: > refs/heads/master 5df32e6d7 -> 99bcfd24c > > > All slashes were being removed from the storage path which caused adding the > pool to break. > Reviewed-by: Edison Su<sudi...@gmail.com> > > > Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo > Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/99bcfd24 > Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/99bcfd24 > Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/99bcfd24 > > Branch: refs/heads/master > Commit: 99bcfd24c70d751d0222944d15a468a6017fb660 > Parents: 5df32e6 > Author: Brian Spindler <darn...@gmail.com> > Authored: Mon Mar 25 13:20:27 2013 -0700 > Committer: Edison Su <sudi...@gmail.com> > Committed: Tue Mar 26 17:36:29 2013 -0700 > > ---------------------------------------------------------------------- > .../CloudStackPrimaryDataStoreLifeCycleImpl.java | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > ---------------------------------------------------------------------- > > > http://git-wip-us.apache.org/repos/asf/cloudstack/blob/99bcfd24/plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java > ---------------------------------------------------------------------- > diff --git > a/plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java > > b/plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java > index 2991574..b8b0859 100644 > --- > a/plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java > +++ > b/plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java > @@ -204,7 +204,7 @@ public class CloudStackPrimaryDataStoreLifeCycleImpl > implements > String hostPath = uri.getPath(); > Object localStorage = dsInfos.get("localStorage"); > if (localStorage != null) { > - hostPath = hostPath.replace("/", ""); > + hostPath = hostPath.replaceFirst("/", ""); > } > String userInfo = uri.getUserInfo(); > int port = uri.getPort(); >
