Alon Bar-Lev has posted comments on this change.
Change subject: core: host-deploy: Using ssh username and password to install
and update host
......................................................................
Patch Set 10: (2 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java
Line 326: if (Config.<Boolean>
GetValue(ConfigValues.UseSecureConnectionWithServers)
Line 327: && !EngineEncryptionUtils.haveKey()) {
Line 328: returnValue =
failCanDoAction(VdcBllMessages.VDS_TRY_CREATE_SECURE_CERTIFICATE_NOT_FOUND);
Line 329: } else if (!getParameters().getAddPending()
Line 330: &&
StringUtils.isBlank(getParameters().getPassword())) {
Password should be used exact as-is, please revert.
Line 331: // We block vds installations if it's not a
RHEV-H and password is empty
Line 332: // Note that this may override local host SSH
policy. See BZ#688718.
Line 333: returnValue =
failCanDoAction(VdcBllMessages.VDS_CANNOT_INSTALL_EMPTY_PASSWORD);
Line 334: } else if (!isPowerManagementLegal()) {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java
Line 86: && _oldVds.getStatus() !=
VDSStatus.NonOperational
Line 87: && _oldVds.getStatus() !=
VDSStatus.InstallFailed) {
Line 88:
addCanDoActionMessage(VdcBllMessages.VDS_CANNOT_INSTALL_STATUS_ILLEGAL);
Line 89: } else if (getParameters().getInstallVds()
Line 90: &&
StringUtils.isBlank(getParameters().getPassword())
passwords should be used as-is, please revert.
Line 91: &&
getParameters().getVdsStaticData().getVdsType() == VDSType.VDS) {
Line 92:
addCanDoActionMessage(VdcBllMessages.VDS_CANNOT_INSTALL_EMPTY_PASSWORD);
Line 93: } else if (!getParameters().getInstallVds()
Line 94: && _oldVds.getPort() !=
getParameters().getVdsStaticData().getPort()) {
--
To view, visit http://gerrit.ovirt.org/16686
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3d9b23771314ebebb3e10686decb4f0d5ace6d3c
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
