Sahina Bose has uploaded a new change for review. Change subject: engine: Added username to geo-rep commands ......................................................................
engine: Added username to geo-rep commands To support non-root geo-rep session management Change-Id: I3fcdc7c29abb88916eacc048f2708ea68540134a Signed-off-by: Sahina Bose <sab...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGeoRepSessionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/PauseGlusterVolumeGeoRepSessionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResetDefaultGeoRepConfigCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResumeGeoRepSessionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/SetGeoRepConfigCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StartGlusterVolumeGeoRepCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopGeoRepSessionCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/gluster/GlusterVolumeGeoRepConfigVdsParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/DeleteGlusterVolumeGeoRepSessionVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionListVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionStatusVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/PauseGlusterVolumeGeoRepSessionVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/ResumeGeoRepSessionVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigDefaultVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StartGlusterVolumeGeoRepVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StopGlusterVolumeGeoRepSessionVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 22 files changed, 369 insertions(+), 131 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/85/39485/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGeoRepSessionCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGeoRepSessionCommand.java index 573335a..4d732e3 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGeoRepSessionCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGeoRepSessionCommand.java @@ -40,7 +40,7 @@ VDSCommandType.DeleteGlusterVolumeGeoRepSession, new GlusterVolumeGeoRepSessionVDSParameters(upServer.getId(), getGeoRepSession().getMasterVolumeName(), getGeoRepSession().getSlaveHostName(), - getGeoRepSession().getSlaveVolumeName())); + getGeoRepSession().getSlaveVolumeName(), getGeoRepSession().getUserName())); setSucceeded(returnValue.getSucceeded()); if (getSucceeded()) { getGlusterGeoRepDao().remove(getGeoRepSession().getId()); diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/PauseGlusterVolumeGeoRepSessionCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/PauseGlusterVolumeGeoRepSessionCommand.java index 1ab00a1..3a70759 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/PauseGlusterVolumeGeoRepSessionCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/PauseGlusterVolumeGeoRepSessionCommand.java @@ -46,6 +46,7 @@ session.getMasterVolumeName(), session.getSlaveHostName(), session.getSlaveVolumeName(), + session.getUserName(), getParameters().isForce())); setSucceeded(returnValue.getSucceeded()); if (!getSucceeded()) { diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResetDefaultGeoRepConfigCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResetDefaultGeoRepConfigCommand.java index 215371c..078c663 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResetDefaultGeoRepConfigCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResetDefaultGeoRepConfigCommand.java @@ -27,7 +27,9 @@ session.getMasterVolumeName(), session.getSlaveHostName(), session.getSlaveVolumeName(), - getParameters().getConfigKey())); + getParameters().getConfigKey(), + null, + session.getUserName())); setSucceeded(returnValue.getSucceeded()); if (!getSucceeded()) { handleVdsError(AuditLogType.GLUSTER_GEOREP_CONFIG_SET_DEFAULT_FAILED, returnValue.getVdsError() diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResumeGeoRepSessionCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResumeGeoRepSessionCommand.java index 7ea37d2..1634369 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResumeGeoRepSessionCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResumeGeoRepSessionCommand.java @@ -50,6 +50,7 @@ session.getMasterVolumeName(), session.getSlaveHostName(), session.getSlaveVolumeName(), + session.getUserName(), parameters.isForce())); setSucceeded(returnValue.getSucceeded()); if (!getSucceeded()) { diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/SetGeoRepConfigCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/SetGeoRepConfigCommand.java index dce0c43..44c9eeb 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/SetGeoRepConfigCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/SetGeoRepConfigCommand.java @@ -44,7 +44,8 @@ session.getSlaveHostName(), session.getSlaveVolumeName(), configKey, - configValue)); + configValue, + session.getUserName())); setSucceeded(returnValue.getSucceeded()); if (getSucceeded()) { GlusterGeoRepSessionConfiguration geoRepSessionConfig = new GlusterGeoRepSessionConfiguration(); diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StartGlusterVolumeGeoRepCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StartGlusterVolumeGeoRepCommand.java index fc5bee7..b4aa1ca 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StartGlusterVolumeGeoRepCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StartGlusterVolumeGeoRepCommand.java @@ -29,7 +29,13 @@ @Override protected void executeCommand() { GlusterGeoRepSession session = getGeoRepSession(); - VDSReturnValue returnValue = runVdsCommand(VDSCommandType.StartGlusterVolumeGeoRep, new GlusterVolumeGeoRepSessionVDSParameters(upServer.getId(), getGlusterVolumeName(), session.getSlaveHostName(), session.getSlaveVolumeName())); + VDSReturnValue returnValue = + runVdsCommand(VDSCommandType.StartGlusterVolumeGeoRep, + new GlusterVolumeGeoRepSessionVDSParameters(upServer.getId(), + getGlusterVolumeName(), + session.getSlaveHostName(), + session.getSlaveVolumeName(), + session.getUserName())); setSucceeded(returnValue.getSucceeded()); if (getSucceeded()) { session.setStatus(GeoRepSessionStatus.INITIALIZING); diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopGeoRepSessionCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopGeoRepSessionCommand.java index 40a4b6f..0eea76c 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopGeoRepSessionCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopGeoRepSessionCommand.java @@ -55,8 +55,11 @@ runVdsCommand( VDSCommandType.StopGlusterVolumeGeoRepSession, new GlusterVolumeGeoRepSessionVDSParameters(upServer.getId(), - getGeoRepSession().getMasterVolumeName(), getGeoRepSession().getSlaveHostName(), - getGeoRepSession().getSlaveVolumeName(), getParameters().isForce())); + getGeoRepSession().getMasterVolumeName(), + getGeoRepSession().getSlaveHostName(), + getGeoRepSession().getSlaveVolumeName(), + getGeoRepSession().getUserName(), + getParameters().isForce())); setSucceeded(returnValue.getSucceeded()); if (getSucceeded()) { diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/gluster/GlusterVolumeGeoRepConfigVdsParameters.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/gluster/GlusterVolumeGeoRepConfigVdsParameters.java index e7ac0d2..c0640c0 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/gluster/GlusterVolumeGeoRepConfigVdsParameters.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/gluster/GlusterVolumeGeoRepConfigVdsParameters.java @@ -17,6 +17,18 @@ this.configValue = configValue; } + public GlusterVolumeGeoRepConfigVdsParameters(Guid serverId, + String volumeName, + String slaveHost, + String slaveVolume, + String configKey, + String configValue, + String userName) { + super(serverId, volumeName, slaveHost, slaveVolume, userName); + this.configKey = configKey; + this.configValue = configValue; + } + public GlusterVolumeGeoRepConfigVdsParameters(Guid serverId, String volumeName, String slaveHost, String slaveVolume, String configKey) { super(serverId, volumeName, slaveHost, slaveVolume); this.configKey = configKey; diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/DeleteGlusterVolumeGeoRepSessionVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/DeleteGlusterVolumeGeoRepSessionVDSCommand.java index f96e803..7b73199 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/DeleteGlusterVolumeGeoRepSessionVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/DeleteGlusterVolumeGeoRepSessionVDSCommand.java @@ -16,7 +16,8 @@ protected void executeVdsBrokerCommand() { status = getBroker().glusterVolumeGeoRepSessionDelete(getParameters().getVolumeName(), getParameters().getSlaveHost(), - getParameters().getSlaveVolume()); + getParameters().getSlaveVolume(), + getParameters().getUserName()); // Handle errors if any proceedProxyReturnValue(); diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java index 8d0ca29..7b3d20e 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java @@ -21,7 +21,11 @@ @Override protected void executeVdsBrokerCommand() { - result = getBroker().glusterVolumeGeoRepConfigList(getParameters().getVolumeName(), getParameters().getSlaveHost(), getParameters().getSlaveVolume()); + result = + getBroker().glusterVolumeGeoRepConfigList(getParameters().getVolumeName(), + getParameters().getSlaveHost(), + getParameters().getSlaveVolume(), + getParameters().getUserName()); proceedProxyReturnValue(); List<GlusterGeoRepSessionConfiguration> sessionConfigs = result.getSessionConfig(); setReturnValue(sessionConfigs); diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionListVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionListVDSCommand.java index 34ac9c9..3e88b92 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionListVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionListVDSCommand.java @@ -26,7 +26,8 @@ result = getBroker().glusterVolumeGeoRepSessionList(parameter.getVolumeName(), parameter.getSlaveHost(), - parameter.getSlaveVolume()); + parameter.getSlaveVolume(), + parameter.getUserName()); } proceedProxyReturnValue(); if (getVDSReturnValue().getSucceeded()) { diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionStatusVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionStatusVDSCommand.java index f4a8089..a658c0b 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionStatusVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepSessionStatusVDSCommand.java @@ -19,8 +19,9 @@ protected void executeVdsBrokerCommand() { GlusterVolumeGeoRepSessionVDSParameters parameter = getParameters(); result = getBroker().glusterVolumeGeoRepSessionStatus(parameter.getVolumeName(), - parameter.getSlaveHost(), - parameter.getSlaveVolume()); + parameter.getSlaveHost(), + parameter.getSlaveVolume(), + parameter.getUserName()); proceedProxyReturnValue(); if (getVDSReturnValue().getSucceeded()) { setReturnValue(result.getGeoRepDetails()); diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/PauseGlusterVolumeGeoRepSessionVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/PauseGlusterVolumeGeoRepSessionVDSCommand.java index e5cb03e..5611c91 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/PauseGlusterVolumeGeoRepSessionVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/PauseGlusterVolumeGeoRepSessionVDSCommand.java @@ -11,7 +11,12 @@ @Override protected void executeVdsBrokerCommand() { GlusterVolumeGeoRepSessionVDSParameters parameters = getParameters(); - status = getBroker().glusterVolumeGeoRepSessionPause(parameters.getVolumeName(), parameters.getSlaveHost(), parameters.getSlaveVolume(), parameters.getForce()); + status = + getBroker().glusterVolumeGeoRepSessionPause(parameters.getVolumeName(), + parameters.getSlaveHost(), + parameters.getSlaveVolume(), + parameters.getUserName(), + parameters.getForce()); proceedProxyReturnValue(); } } diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/ResumeGeoRepSessionVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/ResumeGeoRepSessionVDSCommand.java index 9fca86b..ba07e8c 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/ResumeGeoRepSessionVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/ResumeGeoRepSessionVDSCommand.java @@ -11,7 +11,12 @@ @Override protected void executeVdsBrokerCommand() { GlusterVolumeGeoRepSessionVDSParameters parameters = getParameters(); - status = getBroker().glusterVolumeGeoRepSessionResume(parameters.getVolumeName(), parameters.getSlaveHost(), parameters.getSlaveVolume(), parameters.getForce()); + status = + getBroker().glusterVolumeGeoRepSessionResume(parameters.getVolumeName(), + parameters.getSlaveHost(), + parameters.getSlaveVolume(), + parameters.getUserName(), + parameters.getForce()); proceedProxyReturnValue(); } } diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigDefaultVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigDefaultVDSCommand.java index 8118487..e8afe0b 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigDefaultVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigDefaultVDSCommand.java @@ -10,7 +10,12 @@ @Override protected void executeVdsBrokerCommand() { - status = getBroker().glusterVolumeGeoRepConfigReset(getParameters().getVolumeName(), getParameters().getSlaveHost(), getParameters().getSlaveVolume(), getParameters().getConfigKey()); + status = + getBroker().glusterVolumeGeoRepConfigReset(getParameters().getVolumeName(), + getParameters().getSlaveHost(), + getParameters().getSlaveVolume(), + getParameters().getConfigKey(), + getParameters().getUserName()); proceedProxyReturnValue(); } diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigVDSCommand.java index 8d03426..d3f7725 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeGeoRepConfigVDSCommand.java @@ -10,7 +10,13 @@ @Override protected void executeVdsBrokerCommand() { - status = getBroker().glusterVolumeGeoRepConfigSet(getParameters().getVolumeName(), getParameters().getSlaveHost(), getParameters().getSlaveVolume(), getParameters().getConfigKey(), getParameters().getConfigValue()); + status = + getBroker().glusterVolumeGeoRepConfigSet(getParameters().getVolumeName(), + getParameters().getSlaveHost(), + getParameters().getSlaveVolume(), + getParameters().getConfigKey(), + getParameters().getConfigValue(), + getParameters().getUserName()); proceedProxyReturnValue(); } } diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StartGlusterVolumeGeoRepVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StartGlusterVolumeGeoRepVDSCommand.java index bdec25b..a5bdc37 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StartGlusterVolumeGeoRepVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StartGlusterVolumeGeoRepVDSCommand.java @@ -11,7 +11,12 @@ @Override protected void executeVdsBrokerCommand() { GlusterVolumeGeoRepSessionVDSParameters parameters = getParameters(); - status = getBroker().glusterVolumeGeoRepSessionStart(parameters.getVolumeName(), parameters.getSlaveHost(), parameters.getSlaveVolume(), parameters.getForce()); + status = + getBroker().glusterVolumeGeoRepSessionStart(parameters.getVolumeName(), + parameters.getSlaveHost(), + parameters.getSlaveVolume(), + parameters.getUserName(), + parameters.getForce()); proceedProxyReturnValue(); } diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StopGlusterVolumeGeoRepSessionVDSCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StopGlusterVolumeGeoRepSessionVDSCommand.java index d132cd2..a65a054 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StopGlusterVolumeGeoRepSessionVDSCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/StopGlusterVolumeGeoRepSessionVDSCommand.java @@ -14,9 +14,10 @@ @Override protected void executeVdsBrokerCommand() { status = getBroker().glusterVolumeGeoRepSessionStop(getParameters().getVolumeName(), - getParameters().getSlaveHost(), - getParameters().getSlaveVolume(), - getParameters().getForce()); + getParameters().getSlaveHost(), + getParameters().getSlaveVolume(), + getParameters().getUserName(), + getParameters().getForce()); // Handle errors if any proceedProxyReturnValue(); diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java index 010592f..f1cc661 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java @@ -1235,11 +1235,16 @@ } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionResume(String volumeName, String slaveHostName, String slaveVolumeName, boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionResume(String volumeName, + String slaveHostName, + String slaveVolumeName, + String slaveUserName, + boolean force) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionResume").withParameter("volumeName", volumeName) - .withParameter("remoteHost", slaveHostName) - .withParameter("remoteVolumeName", slaveVolumeName) - .withParameter("force", force).build(); + .withParameter("remoteHost", slaveHostName) + .withParameter("remoteVolumeName", slaveVolumeName) + .withOptionalParameter("remoteUserName", slaveUserName) + .withParameter("force", force).build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @@ -1284,24 +1289,33 @@ } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionDelete(String volumeName, String remoteHost, String remoteVolumeName) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionDelete(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionDelete") - .withParameter("volumeName", volumeName) - .withParameter("remoteHost", remoteHost) - .withParameter("remoteVolumeName", remoteVolumeName) - .build(); + .withParameter("volumeName", volumeName) + .withParameter("remoteHost", remoteHost) + .withParameter("remoteVolumeName", remoteVolumeName) + .withOptionalParameter("remoteUserName", remoteUserName) + .build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStop(String volumeName, String remoteHost, String remoteVolumeName, Boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStop(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionStop") - .withParameter("volumeName", volumeName) - .withParameter("remoteHost", remoteHost) - .withParameter("remoteVolumeName", remoteVolumeName) - .withParameter("force", force) - .build(); + .withParameter("volumeName", volumeName) + .withParameter("remoteHost", remoteHost) + .withParameter("remoteVolumeName", remoteVolumeName) + .withOptionalParameter("remoteUserName", remoteUserName) + .withParameter("force", force) + .build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @@ -1355,33 +1369,49 @@ } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionPause(String masterVolumeName, String slaveHost, String slaveVolumeName, boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionPause(String masterVolumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName, + boolean force) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionPause").withParameter("volumeName", masterVolumeName) - .withParameter("remoteHost", slaveHost) - .withParameter("remoteVolumeName", slaveVolumeName) - .withParameter("force", force).build(); + .withParameter("remoteHost", slaveHost) + .withParameter("remoteVolumeName", slaveVolumeName) + .withOptionalParameter("remoteUserName", slaveUserName) + .withParameter("force", force).build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStart(String volumeName, String remoteHost, String remoteVolumeName, Boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStart(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionStart").withParameter("volumeName", volumeName) - .withParameter("remoteHost", remoteHost) - .withParameter("remoteVolumeName", remoteVolumeName) - .withParameter("force", force).build(); + .withParameter("remoteHost", remoteHost) + .withParameter("remoteVolumeName", remoteVolumeName) + .withOptionalParameter("remoteUserName", remoteUserName) + .withParameter("force", force).build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigSet(String volumeName, String slaveHost, String slaveVolumeName, String configKey, String configValue) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigSet(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String configValue, + String slaveUserName) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepConfigSet") .withParameter("volumeName", volumeName) .withParameter("remoteHost", slaveHost) .withParameter("remoteVolumeName", slaveVolumeName) .withParameter("optionName", configKey) - .withParameter("optionValue", configValue).build(); + .withParameter("optionValue", configValue) + .withOptionalParameter("remoteUserName", slaveUserName).build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @@ -1390,24 +1420,30 @@ public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigReset(String volumeName, String slaveHost, String slaveVolumeName, - String configKey) { + String configKey, + String slaveUserName) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepConfigReset") - .withParameter("volumeName", volumeName) - .withParameter("remoteHost", slaveHost) - .withParameter("remoteVolumeName", slaveVolumeName) - .withParameter("optionName", configKey) - .build(); + .withParameter("volumeName", volumeName) + .withParameter("remoteHost", slaveHost) + .withParameter("remoteVolumeName", slaveVolumeName) + .withParameter("optionName", configKey) + .withOptionalParameter("remoteUserName", slaveUserName) + .build(); Map<String, Object> response = new FutureMap(this.client, request); return new StatusOnlyReturnForXmlRpc(response); } @Override - public GlusterVolumeGeoRepConfigListXmlRpc glusterVolumeGeoRepConfigList(String volumeName, String slaveHost, String slaveVolumeName) { + public GlusterVolumeGeoRepConfigListXmlRpc glusterVolumeGeoRepConfigList(String volumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepConfigList") - .withParameter("volumeName", volumeName) - .withParameter("remoteHost", slaveHost) - .withParameter("remoteVolumeName", slaveVolumeName) - .build(); + .withParameter("volumeName", volumeName) + .withParameter("remoteHost", slaveHost) + .withParameter("remoteVolumeName", slaveVolumeName) + .withOptionalParameter("remoteUserName", slaveUserName) + .build(); Map<String, Object> response = new FutureMap(this.client, request); return new GlusterVolumeGeoRepConfigListXmlRpc(response); } @@ -1609,11 +1645,13 @@ @Override public GlusterVolumeGeoRepStatusForXmlRpc glusterVolumeGeoRepSessionList(String volumeName, String slaveHost, - String slaveVolumeName) { + String slaveVolumeName, + String slaveUserName) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionList").withParameter("volumeName", volumeName) - .withParameter("remoteHost", slaveHost) - .withParameter("remoteVolumeName", slaveVolumeName).build(); + .withParameter("remoteHost", slaveHost) + .withParameter("remoteVolumeName", slaveVolumeName) + .withOptionalParameter("remoteUserName", slaveUserName).build(); Map<String, Object> response = new FutureMap(this.client, request).withIgnoreResponseKey(); return new GlusterVolumeGeoRepStatusForXmlRpc(response); } @@ -1621,11 +1659,13 @@ @Override public GlusterVolumeGeoRepStatusDetailForXmlRpc glusterVolumeGeoRepSessionStatus(String volumeName, String slaveHost, - String slaveVolumeName) { + String slaveVolumeName, + String slaveUserName) { JsonRpcRequest request = new RequestBuilder("GlusterVolume.geoRepSessionStatus").withParameter("volumeName", volumeName) - .withParameter("remoteHost", slaveHost) - .withParameter("remoteVolumeName", slaveVolumeName).build(); + .withParameter("remoteHost", slaveHost) + .withParameter("remoteVolumeName", slaveVolumeName) + .withOptionalParameter("remoteUserName", slaveUserName).build(); Map<String, Object> response = new FutureMap(this.client, request).withIgnoreResponseKey(); return new GlusterVolumeGeoRepStatusDetailForXmlRpc(response); } diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java index 5a386ee..431b3b1 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java @@ -279,18 +279,6 @@ GlusterServersListReturnForXmlRpc glusterServersList(); - GlusterHostsPubKeyReturnForXmlRpc glusterGeoRepKeysGet(); - - StatusOnlyReturnForXmlRpc glusterGeoRepKeysUpdate(List<String> geoRepPubKeys, String remoteUserName); - - StatusOnlyReturnForXmlRpc glusterGeoRepMountBrokerSetup(String remoteVolumeName, String remoteUserName, String remoteGroupName); - - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionCreate(String volumeName, String remoteHost, String remoteVolumeName, String remoteUserName, Boolean force); - - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionDelete(String volumeName, String remoteHost, String remoteVolumeName); - - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStop(String volumeName, String remoteHost, String remoteVolumeName, Boolean force); - StatusOnlyReturnForXmlRpc diskReplicateStart(String vmUUID, Map srcDisk, Map dstDisk); StatusOnlyReturnForXmlRpc diskReplicateFinish(String vmUUID, Map srcDisk, Map dstDisk); @@ -298,10 +286,6 @@ StatusOnlyReturnForXmlRpc glusterVolumeProfileStart(String volumeName); StatusOnlyReturnForXmlRpc glusterVolumeProfileStop(String volumeName); - - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStart(String volumeName, String remoteHost, String remoteVolumeName, Boolean force); - - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionPause(String masterVolumeName, String slaveHost, String slaveVolumeName, boolean force); GlusterVolumeStatusReturnForXmlRpc glusterVolumeStatus(Guid clusterId, String volumeName, @@ -338,23 +322,74 @@ GlusterVolumeTaskReturnForXmlRpc glusterVolumeRebalanceStatus(String volumeName); + GlusterHostsPubKeyReturnForXmlRpc glusterGeoRepKeysGet(); + + StatusOnlyReturnForXmlRpc glusterGeoRepKeysUpdate(List<String> geoRepPubKeys, String remoteUserName); + + StatusOnlyReturnForXmlRpc glusterGeoRepMountBrokerSetup(String remoteVolumeName, + String remoteUserName, + String remoteGroupName); + + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionCreate(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force); + + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionDelete(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName); + + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStop(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force); + GlusterVolumeGeoRepStatusForXmlRpc glusterVolumeGeoRepSessionList(); GlusterVolumeGeoRepStatusForXmlRpc glusterVolumeGeoRepSessionList(String volumeName); GlusterVolumeGeoRepStatusForXmlRpc glusterVolumeGeoRepSessionList(String volumeName, String slaveHost, - String slaveVolumeName); + String slaveVolumeName, + String slaveUserName); GlusterVolumeGeoRepStatusDetailForXmlRpc glusterVolumeGeoRepSessionStatus(String volumeName, String slaveHost, - String slaveVolumeName); + String slaveVolumeName, + String slaveUserName); - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigSet(String volumeName, String slaveHost, String slaveVolumeName, String configKey, String configValue); + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStart(String volumeName, + String remoteHost, + String remoteVolumeName, + String slaveUserName, + Boolean force); - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigReset(String volumeName, String slaveHost, String slaveVolumeName, String configKey); + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionPause(String masterVolumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName, + boolean force); - GlusterVolumeGeoRepConfigListXmlRpc glusterVolumeGeoRepConfigList(String volumeName, String slaveHost, String slaveVolumeName); + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigSet(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String configValue, + String slaveUserName); + + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigReset(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String slaveUserName); + + GlusterVolumeGeoRepConfigListXmlRpc glusterVolumeGeoRepConfigList(String volumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName); GlusterVolumeTaskReturnForXmlRpc glusterVolumeRemoveBrickStatus(String volumeName, String[] bricksList); @@ -364,7 +399,11 @@ VMListReturnForXmlRpc getExternalVmList(String uri, String username, String password); - StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionResume(String volumeName, String slaveHostName, String slaveVolumeName, boolean force); + StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionResume(String volumeName, + String slaveHostName, + String slaveVolumeName, + String slaveUserName, + boolean force); GlusterVolumeSnapshotInfoReturnForXmlRpc glusterSnapshotInfo(Guid clusterId, String volumeName); diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java index 3377ad9..e40bc96 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java @@ -235,16 +235,6 @@ public Map<String, Object> glusterVolumeRebalanceStart(String volumeName, Boolean fixLayoutOnly, Boolean force); - public Map<String, Object> glusterGeoRepKeysGet(); - - public Map<String, Object> glusterGeoRepKeysUpdate(String remoteUserName, List<String> geoRepPubKeys); - - public Map<String, Object> glusterGeoRepMountBrokerSetup(String remoteUserName, String remoteGroupName, String remoteVolumeName); - - public Map<String, Object> glusterVolumeGeoRepSessionCreate(String volumeName, String remoteHost, String remoteVolumeName, String remoteUserName, Boolean force); - - public Map<String, Object> glusterVolumeGeoRepSessionResume(String volumeName, String slaveHostName, String slaveVolumeName, boolean force); - public Map<String, Object> glusterVolumeRebalanceStop(String volumeName); public Map<String, Object> replaceGlusterVolumeBrickStart(String volumeName, String existingBrickDir, String newBrickDir); @@ -259,12 +249,6 @@ public Map<String, Object> glusterVolumesList(); - public Map<String, Object> glusterVolumeGeoRepSessionDelete(String volumeName, String remoteHost, String remoteVolumeName); - - public Map<String, Object> glusterVolumeGeoRepSessionStop(String volumeName, String remoteHost, String remoteVolumeName, Boolean force); - - public Map<String, Object> glusterVolumeGeoRepConfigSet(String volumeName, String slaveHost, String slaveVolumeName, String configKey, String configValue); - public Map<String, Object> ping(); @FutureCall(delegeteTo = "ping") @@ -277,10 +261,6 @@ public Map<String, Object> glusterVolumeProfileStart(String volumeName); public Map<String, Object> glusterVolumeProfileStop(String volumeName); - - public Map<String, Object> glusterVolumeGeoRepSessionStart(String volumeName, String remoteHost, String remoteVolumeName, Boolean force); - - public Map<String, Object> glusterVolumeGeoRepSessionPause(String masterVolumeName, String slaveHost, String slaveVolumeName, boolean force); public Map<String, Object> glusterVolumeStatus(String volumeName, String brickName, String volumeStatusOption); @@ -298,9 +278,18 @@ public Map<String, Object> glusterHookRead(String glusterCommand, String stage, String hookName); - public Map<String, Object> glusterHookUpdate(String glusterCommand, String stage, String hookName, String content, String checksum); + public Map<String, Object> glusterHookUpdate(String glusterCommand, + String stage, + String hookName, + String content, + String checksum); - public Map<String, Object> glusterHookAdd(String glusterCommand, String stage, String hookName, String content, String checksum, Boolean enabled); + public Map<String, Object> glusterHookAdd(String glusterCommand, + String stage, + String hookName, + String content, + String checksum, + Boolean enabled); public Map<String, Object> glusterHookRemove(String glusterCommand, String stage, String hookName); @@ -312,17 +301,80 @@ public Map<String, Object> glusterVolumeRebalanceStatus(String volumeName); + public Map<String, Object> glusterGeoRepKeysGet(); + + public Map<String, Object> glusterGeoRepKeysUpdate(String remoteUserName, List<String> geoRepPubKeys); + + public Map<String, Object> glusterGeoRepMountBrokerSetup(String remoteUserName, + String remoteGroupName, + String remoteVolumeName); + + public Map<String, Object> glusterVolumeGeoRepSessionCreate(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force); + + public Map<String, Object> glusterVolumeGeoRepSessionResume(String volumeName, + String slaveHostName, + String slaveVolumeName, + String slaveUserName, + boolean force); + + public Map<String, Object> glusterVolumeGeoRepSessionStart(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force); + + public Map<String, Object> glusterVolumeGeoRepSessionPause(String masterVolumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName, + boolean force); + public Map<String, Object> glusterVolumeGeoRepSessionList(); public Map<String, Object> glusterVolumeGeoRepSessionList(String volumeName); public Map<String, Object> glusterVolumeGeoRepSessionList(String volumeName, String slaveHost, - String slaveVolumeName); + String slaveVolumeName, + String slaveUserName); public Map<String, Object> glusterVolumeGeoRepSessionStatus(String volumeName, String slaveHost, - String slaveVolumeName); + String slaveVolumeName, + String slaveUserName); + + public Map<String, Object> glusterVolumeGeoRepConfigReset(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String slaveUserName); + + public Map<String, Object> glusterVolumeGeoRepConfigList(String volumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName); + + public Map<String, Object> glusterVolumeGeoRepSessionDelete(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName); + + public Map<String, Object> glusterVolumeGeoRepSessionStop(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force); + + public Map<String, Object> glusterVolumeGeoRepConfigSet(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String configValue, + String slaveUserName); public Map<String, Object> glusterVolumeRemoveBrickStatus(String volumeName, String[] bricksList); @@ -331,10 +383,6 @@ public Map<String, Object> updateVmPolicy(Map info); public Map<String, Object> getExternalVMs(String uri, String username, String password); - - public Map<String, Object> glusterVolumeGeoRepConfigReset(String volumeName, String slaveHost, String slaveVolumeName, String configKey); - - public Map<String, Object> glusterVolumeGeoRepConfigList(String volumeName, String slaveHost, String slaveVolumeName); public Map<String, Object> glusterSnapshotInfo(String snapshotName, String volumeName); diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java index 335f5ea..2db2096 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java @@ -1166,9 +1166,18 @@ } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionResume(String volumeName, String slaveHostName, String slaveVolumeName, boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionResume(String volumeName, + String slaveHostName, + String slaveVolumeName, + String slaveUserName, + boolean force) { try{ - Map<String, Object> xmlRpcReturnValue = vdsServer.glusterVolumeGeoRepSessionResume(volumeName, slaveHostName, slaveVolumeName, force); + Map<String, Object> xmlRpcReturnValue = + vdsServer.glusterVolumeGeoRepSessionResume(volumeName, + slaveHostName, + slaveVolumeName, + slaveUserName, + force); StatusOnlyReturnForXmlRpc wrapper = new StatusOnlyReturnForXmlRpc(xmlRpcReturnValue); return wrapper; } catch (UndeclaredThrowableException ute) { @@ -1272,18 +1281,34 @@ } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionPause(String masterVolumeName, String slaveHost, String slaveVolumeName, boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionPause(String masterVolumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName, + boolean force) { try { - return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepSessionPause(masterVolumeName, slaveHost, slaveVolumeName, force)); + return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepSessionPause(masterVolumeName, + slaveHost, + slaveVolumeName, + slaveUserName, + force)); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStart(String volumeName, String remoteHost, String remoteVolumeName, Boolean force) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStart(String volumeName, + String remoteHost, + String remoteVolumeName, + String remoteUserName, + Boolean force) { try { - return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepSessionStart(volumeName, remoteHost, remoteVolumeName, force)); + return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepSessionStart(volumeName, + remoteHost, + remoteVolumeName, + remoteUserName, + force)); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } @@ -1487,10 +1512,11 @@ @Override public GlusterVolumeGeoRepStatusForXmlRpc glusterVolumeGeoRepSessionList(String volumeName, String slaveHost, - String slaveVolumeName) { + String slaveVolumeName, + String slaveUserName) { try { Map<String, Object> response; - response = vdsServer.glusterVolumeGeoRepSessionList(volumeName, slaveHost, slaveVolumeName); + response = vdsServer.glusterVolumeGeoRepSessionList(volumeName, slaveHost, slaveVolumeName, slaveUserName); return new GlusterVolumeGeoRepStatusForXmlRpc(response); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); @@ -1500,10 +1526,11 @@ @Override public GlusterVolumeGeoRepStatusDetailForXmlRpc glusterVolumeGeoRepSessionStatus(String volumeName, String slaveHost, - String slaveVolumeName) { + String slaveVolumeName, + String slaveUserName) { try { Map<String, Object> response = - vdsServer.glusterVolumeGeoRepSessionStatus(volumeName, slaveHost, slaveVolumeName); + vdsServer.glusterVolumeGeoRepSessionStatus(volumeName, slaveHost, slaveVolumeName, slaveUserName); return new GlusterVolumeGeoRepStatusDetailForXmlRpc(response); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); @@ -1523,10 +1550,10 @@ @Override public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionDelete(String volumeName, String remoteHost, - String remoteVolumeName) { + String remoteVolumeName, String remoteUserName) { try { return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepSessionDelete(volumeName, - remoteHost, remoteVolumeName)); + remoteHost, remoteVolumeName, remoteUserName)); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } @@ -1534,37 +1561,61 @@ @Override public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepSessionStop(String volumeName, String remoteHost, - String remoteVolumeName, Boolean force) { + String remoteVolumeName, String remoteUserName, Boolean force) { try { return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepSessionStop(volumeName, - remoteHost, remoteVolumeName, force)); + remoteHost, remoteVolumeName, remoteUserName, force)); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigSet(String volumeName, String slaveHost, String slaveVolumeName, String configKey, String configValue) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigSet(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String configValue, + String slaveUserName) { try { - return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepConfigSet(volumeName, slaveHost, slaveVolumeName, configKey, configValue)); + return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepConfigSet(volumeName, + slaveHost, + slaveVolumeName, + configKey, + configValue, + slaveUserName)); } catch (UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } } @Override - public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigReset(String volumeName, String slaveHost, String slaveVolumeName, String configKey) { + public StatusOnlyReturnForXmlRpc glusterVolumeGeoRepConfigReset(String volumeName, + String slaveHost, + String slaveVolumeName, + String configKey, + String slaveUserName) { try { - return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepConfigReset(volumeName, slaveHost, slaveVolumeName, configKey)); + return new StatusOnlyReturnForXmlRpc(vdsServer.glusterVolumeGeoRepConfigReset(volumeName, + slaveHost, + slaveVolumeName, + configKey, + slaveUserName)); } catch(UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } } @Override - public GlusterVolumeGeoRepConfigListXmlRpc glusterVolumeGeoRepConfigList(String volumeName, String slaveHost, String slaveVolumeName) { + public GlusterVolumeGeoRepConfigListXmlRpc glusterVolumeGeoRepConfigList(String volumeName, + String slaveHost, + String slaveVolumeName, + String slaveUserName) { try { - return new GlusterVolumeGeoRepConfigListXmlRpc(vdsServer.glusterVolumeGeoRepConfigList(volumeName, slaveHost, slaveVolumeName)); + return new GlusterVolumeGeoRepConfigListXmlRpc(vdsServer.glusterVolumeGeoRepConfigList(volumeName, + slaveHost, + slaveVolumeName, + slaveUserName)); } catch(UndeclaredThrowableException ute) { throw new XmlRpcRunTimeException(ute); } -- To view, visit https://gerrit.ovirt.org/39485 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3fcdc7c29abb88916eacc048f2708ea68540134a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sab...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches