Yaniv Bronhaim has posted comments on this change.

Change subject: API: host-deploy: Adding ssh username, port and fingerprint to 
host information
......................................................................


Patch Set 9: (1 inline comment)

....................................................
File 
backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/HostMapperTest.java
Line 230: 
Line 231:         VdsStatic mappedVdsStatic = HostMapper.map(sshConf, 
vdsStatic);
Line 232:         assertEquals(mappedVdsStatic.getSshPort(), 22);
Line 233:         assertEquals(mappedVdsStatic.getSshKeyFingerprint(), "1234");
Line 234:         assertEquals(mappedVdsStatic.getSshUsername(), "root");
authType is not mapped to vdsStatic, we don't persist this value, we use it 
only for actions.
Line 235:     }
Line 236: 
Line 237:     @Test
Line 238:     public void testUpdatePowerManagementHostFromAgents() {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic18e91f1602af42e7c73acea5d875c54545cb3c2
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: Ori Liel <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to