Moti Asayag has posted comments on this change.
Change subject: core: Add NetworkLinking support to UpdateVmCommand
......................................................................
Patch Set 10: (1 inline comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
Line 139: for (final VmNetworkInterface iface : interfaces) {
Line 140: Network net = LinqUtils.firstOrNull(networks, new
Predicate<Network>() {
Line 141: @Override
Line 142: public boolean eval(Network n) {
Line 143: return
n.getname().equals(iface.getNetworkName());
please use n.getName()
probably some cleanup patch will get rid of the Network.getname()
Line 144: }
Line 145: });
Line 146: // if network not exists in cluster we remove the
network to
Line 147: // interface connection
--
To view, visit http://gerrit.ovirt.org/9799
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I48c014629678f634593e81e584fcbbb6083fddc7
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
