Arik Hadas has posted comments on this change.

Change subject: core: simplify iso path validation on run vm
......................................................................


Patch Set 2:

(1 comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/RunVmValidator.java
Line 170:         return false;
Line 171:     }
Line 172: 
Line 173:     @SuppressWarnings("unchecked")
Line 174:     private boolean isFloppyPathImageExists(String floppyPath, Guid 
storageDomainId) {
Done
Line 175:         VdcQueryReturnValue ret =
Line 176:                 
getBackend().runInternalQuery(VdcQueryType.GetImagesList,
Line 177:                         new GetImagesListParameters(storageDomainId, 
ImageFileType.Floppy));
Line 178:         if (ret != null && ret.getReturnValue() != null && 
ret.getSucceeded()) {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c1734497fdb22093ec507243ef1ae6bc5006220
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to