Igor Lvovsky has posted comments on this change.

Change subject: BZ#811807 Define network filter on libvirt
......................................................................


Patch Set 1: (1 inline comment)

Dan asked to put it in tools, isn't he?

....................................................
File vdsm/vdsmd.init.in
Line 452:             fi
Line 453:             sleep 1
Line 454:         done
Line 455:         # End workaround
Line 456: 
What is this? Why now?
How this related to nwfilter?
If we really need it, I think it should be in different patch
Line 457:         return 0
Line 458:     else
Line 459:         echo "$startout" >&2
Line 460:         return 1


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9f1708385dec6a87bc404e4ab25c4da8ab8a8acc
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to