Michal Skrivanek has posted comments on this change.

Change subject: support for multiple heas in qxl display divice
......................................................................


Patch Set 6: (3 inline comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
Line 386:         }
Line 387:         return legal;
Line 388:     }
Line 389: 
Line 390:     public static boolean isSingleQslDeviceLegal(DisplayType 
displayType, int osId, List<String> reasons) {
isSingleQxlDeviceLegal?
Line 391:         if (displayType != DisplayType.qxl) {
Line 392:             
reasons.add(VdcBllMessages.ACTION_TYPE_FAILED_ILLEGAL_SINGLE_DEVICE_DISPLAY_TYPE.toString());
Line 393:             return false;
Line 394:         }


....................................................
Commit Message
Line 3: AuthorDate: 2013-07-10 15:31:37 +0300
Line 4: Commit:     Shahar Havivi <shav...@redhat.com>
Line 5: CommitDate: 2013-07-22 11:54:16 +0300
Line 6: 
Line 7: support for multiple heas in qxl display divice
fix typos
Line 8: 
Line 9: bug-url: https://bugzilla.redhat.com/787578
Line 10: Change-Id: If18b8ceba3d641f3e2e79947b34b549902f00b55


Line 5: CommitDate: 2013-07-22 11:54:16 +0300
Line 6: 
Line 7: support for multiple heas in qxl display divice
Line 8: 
Line 9: bug-url: https://bugzilla.redhat.com/787578
please Bug-Url:
Line 10: Change-Id: If18b8ceba3d641f3e2e79947b34b549902f00b55


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If18b8ceba3d641f3e2e79947b34b549902f00b55
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to