Omer Frenkel has posted comments on this change.

Change subject: webadmin, userportal: cloud-init [5/6] - add to run vm once 
dialog
......................................................................


Patch Set 8: (1 inline comment)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/CloudInitModel.java
Line 465:                 }
Line 466:             }
Line 467: 
Line 468:             if (!networkMap.isEmpty()) {
Line 469:                 if (getDnsSerstartvers().getEntity() != null) {
thanks! dont know how it got in there :/
Line 470:                     for (String server : tokenizeString((String) 
getDnsServers().getEntity())) {
Line 471:                         if (!validateHidden(getDnsServers(), server, 
dnsServerListMessage,
Line 472:                                 new IValidation[] { new 
IpAddressValidation() })) {
Line 473:                             dnsIsValid = false;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c46ebb41ac97ecf08168c215911761e0fc497fd
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Greg Padgett <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[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