Barak Azulay has posted comments on this change.

Change subject: restapi: API should expose hypervisor version (#829625)
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
Line 192:         OperatingSystem os = new OperatingSystem();
Line 193:         os.setType(hostOsInfo[0]);
Line 194:         os.setVersion(version);
Line 195:         return os;
Line 196:     }
Does it work also with ovirt-node?
Line 197: 
Line 198:     private static Integer getIntegerValue(String[] hostOsInfo, int 
indx) {
Line 199:         if (hostOsInfo.length <= indx) {
Line 200:             return null;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id8d3d090ae4c0ba489f3c7484d5c232c55f3c059
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to