Ravi Nori has posted comments on this change.

Change subject: engine, webadmin: Re-work engine ovirt-node host-deploy sequence
......................................................................


Patch Set 6:

(1 comment)

@Gilad

Will fix the comments, replied to one of your comments about "admin" user name

http://gerrit.ovirt.org/#/c/25685/6/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java:

Line 57:     public static final String PmPortKey = "port"; //$NON-NLS-1$
Line 58:     public static final String PmSlotKey = "slot"; //$NON-NLS-1$
Line 59:     public static final String BeginTestStage = "BeginTest"; 
//$NON-NLS-1$
Line 60:     public static final String EndTestStage = "EndTest"; //$NON-NLS-1$
Line 61:     public static final String AdminUserName = "admin"; //$NON-NLS-1$
> can admin@internal be configured?
oVirt node does not let us login using user name "root". We have to login to 
oVirt Node using the user name admin, instead of root. So this variable is used 
to set the ssh user name incase of oVirt node installation using user name and 
password.
Line 62:     public static final String RootUserName = "root"; //$NON-NLS-1$
Line 63: 
Line 64:     private UICommand privateTestCommand;
Line 65:     UIConstants constants = 
ConstantsManager.getInstance().getConstants();


-- 
To view, visit http://gerrit.ovirt.org/25685
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7a897581e2b75f7cdc57db40418504a79b9d2787
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Ravi Nori <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to