Ramesh N has uploaded a new change for review.

Change subject: webadmin: add directory to brick mount point
......................................................................

webadmin: add directory to brick mount point

   Add a directory with name of the brick under brick mount point
in Add Brick Pop up. This required becase gluster requires a
directory under the mount point not the mount point itself as
a brick directory.

Change-Id: I44d0fe15988b8193f0866441097b178a98e88b93
Bug-Url: https://bugzilla.redhat.com/1209443
Signed-off-by: Ramesh Nachimuthu <rnach...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
1 file changed, 5 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/21/39821/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
index 31e622c..65afff4 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
@@ -146,13 +146,14 @@
                 @Override
                 public void onSuccess(Object model, Object returnValue) {
                     List<StorageDevice> bricks = (List<StorageDevice>) 
returnValue;
-                    List<String> lvNames = new ArrayList<String>();
+                    List<String> brickDirectories = new ArrayList<String>();
                     for (StorageDevice brick : bricks) {
-                        if (brick.getMountPoint() != null && 
!brick.getMountPoint().isEmpty()) {
-                            lvNames.add(brick.getMountPoint());
+                        String mountPoint = brick.getMountPoint();
+                        if (mountPoint != null && !mountPoint.isEmpty()) {
+                            brickDirectories.add(mountPoint + 
mountPoint.substring(mountPoint.lastIndexOf("/")));
                         }
                     }
-                    getBricksFromServer().setItems(lvNames);
+                    getBricksFromServer().setItems(brickDirectories);
                 }
             }), selectedServer.getId());
         }


-- 
To view, visit https://gerrit.ovirt.org/39821
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I44d0fe15988b8193f0866441097b178a98e88b93
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnach...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to