You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by David Nalley <da...@gnsa.us> on 2013/03/27 02:07:22 UTC
Re: git commit: updated refs/heads/master to 99bcfd2
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, <ed...@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<su...@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 <da...@gmail.com>
> Authored: Mon Mar 25 13:20:27 2013 -0700
> Committer: Edison Su <su...@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();
>