Omer Frenkel has posted comments on this change.

Change subject: backend: Control virtio rng device
......................................................................


Patch Set 25:

(1 comment)

http://gerrit.ovirt.org/#/c/18176/25/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmRngDevice.java
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmRngDevice.java:

Line 66:     public static final String BYTES_STRING = "bytes";
Line 67:     public static final String PERIOD_STRING = "period";
Line 68:     public static final String SOURCE_STRING = "source";
Line 69: 
Line 70:     public static VmRngDevice fromVmDevice(VmDevice dev) {
please replace this as well with a regular ctor that get vmDevice and just 
call: this(dev.get(),dev.getSpe..)
Line 71:         return new VmRngDevice(dev.getId(), dev.getSpecParams());
Line 72:     }
Line 73: 
Line 74:     // because of gwt


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifc26a484c84244074426f4b0c790e2cc4ebb8da6
Gerrit-PatchSet: 25
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Frank Kobzik <[email protected]>
Gerrit-Reviewer: Martin Betak <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Roy Golan <[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