Tal Nisan has uploaded a new change for review. Change subject: restapi: Show full Gluster storage domain details ......................................................................
restapi: Show full Gluster storage domain details Before this patch the only storage detail of a Gluster displayed via REST was the type thus emitting important data such as address, path and mount options Change-Id: I2294ab6a0256f5c8bc5f48e04dada506500f79d5 Bug-Url: https://bugzilla.redhat.com/1211153 Signed-off-by: Tal Nisan <[email protected]> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java 2 files changed, 16 insertions(+), 11 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/10/39810/1 diff --git a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java index b088314..7b9ffff 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java +++ b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java @@ -360,7 +360,8 @@ case NFS: case LOCALFS: case POSIXFS: - mapNfsOrLocalOrPosix(model, entity); + case GLUSTERFS: + mapFileDomain(model, entity); break; } } @@ -368,7 +369,7 @@ return model; } - protected void mapNfsOrLocalOrPosix(StorageDomain model, + protected void mapFileDomain(StorageDomain model, org.ovirt.engine.core.common.businessentities.StorageDomain entity) { final Storage storage = model.getStorage(); StorageServerConnections cnx = getStorageServerConnection(entity.getStorage()); @@ -381,14 +382,17 @@ } storage.setMountOptions(cnx.getMountOptions()); storage.setVfsType(cnx.getVfsType()); - if (cnx.getNfsRetrans() != null) { - storage.setNfsRetrans(cnx.getNfsRetrans().intValue()); - } - if (cnx.getNfsTimeo() != null) { - storage.setNfsTimeo(cnx.getNfsTimeo().intValue()); - } - if (cnx.getNfsVersion() != null) { - storage.setNfsVersion(StorageDomainMapper.map(cnx.getNfsVersion(), null)); + + if (entity.getStorageType() == StorageType.NFS) { + if (cnx.getNfsRetrans() != null) { + storage.setNfsRetrans(cnx.getNfsRetrans().intValue()); + } + if (cnx.getNfsTimeo() != null) { + storage.setNfsTimeo(cnx.getNfsTimeo().intValue()); + } + if (cnx.getNfsVersion() != null) { + storage.setNfsVersion(StorageDomainMapper.map(cnx.getNfsVersion(), null)); + } } } diff --git a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java index b7b9045..338ff7e 100644 --- a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java +++ b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java @@ -290,7 +290,8 @@ model.setMountOptions(entity.getMountOptions()); } } - else if (entity.getstorage_type().equals(org.ovirt.engine.core.common.businessentities.storage.StorageType.POSIXFS)) { + else if (entity.getstorage_type().equals(org.ovirt.engine.core.common.businessentities.storage.StorageType.POSIXFS) + || entity.getstorage_type().equals(StorageType.GLUSTERFS)) { model.setMountOptions(entity.getMountOptions()); model.setVfsType(entity.getVfsType()); } -- To view, visit https://gerrit.ovirt.org/39810 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I2294ab6a0256f5c8bc5f48e04dada506500f79d5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
