Russell Bryant <russ...@ovn.org> writes: > On Mon, May 29, 2017 at 7:30 AM, Karthik Sundaravel <ksund...@redhat.com> > wrote: >> Hi >> >> Tested-by: ksund...@redhat.com >> I've tested the patch. Please see https://bugzilla.redhat. >> com/show_bug.cgi?id=1397299#c17. > > Thanks for testing and reporting back! > > Sorry for the slow review.
Still not the longest I've had to wait for a bugfix, so no problems :) > I've applied this to master and branch-2.7. Great! Thanks, Russell! >> >> Regards >> Karthik S >> >> On Fri, May 19, 2017 at 3:21 AM, Ben Pfaff <b...@ovn.org> wrote: >> >>> On Thu, May 18, 2017 at 05:11:03PM -0400, Aaron Conole wrote: >>> > Ben Pfaff <b...@ovn.org> writes: >>> > >>> > > On Sat, May 13, 2017 at 11:08:13AM -0400, Aaron Conole wrote: >>> > >> Aaron Conole writes: >>> > >> > It's possible to race with the udev service, such that dpdk ports >>> are >>> > >> > not finished being bound until after ovs-vswitchd has been started. >>> > >> > This means that attempts to use the port will fail. While it is >>> > >> > possible to work around this for some NICs using port hotplug, not >>> all >>> > >> > port types are supported (for instance vfio), and it requires manual >>> > >> > intervention. >>> > >> > >>> > >> > Fixes: 36af136b690c ("rhel-systemd: Delay shutting down the >>> services") >>> > >> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1397299 >>> > >> > Suggested-by: Flavio Leitner <f...@sysclose.org> >>> > >> > Signed-off-by: Aaron Conole <acon...@redhat.com> >>> >> Tested-by: ksund...@redhat.com >> >>> > >> > --- >>> > >> > rhel/usr_lib_systemd_system_ovs-vswitchd.service | 2 +- >>> > >> > 1 file changed, 1 insertion(+), 1 deletion(-) >>> > >> >>> > >> Ping? Should I rebase / resend this? I'm not sure if it still >>> applies >>> > >> to the tree. >>> > > >>> > > It still applies fine but I'm not comfortable reviewing it myself. Is >>> > > Flavio the right one to review it or should someone else? >>> > >>> > Flavio had actually suggested it. I'm CC'ing the tester. Maybe we can >>> > at least get a Tested-by tag. >>> >>> I didn't realize there was a tester. I guess that I should have looked >>> at the bugzilla URL. >>> >>> If Maxim tested it successfully, we don't really need him to give us the >>> Tested-by, we can add it ourselves. (But now that we've asked, we might >>> as well get an answer.) >>> >>> Thanks, >>> >>> Ben. >>> _______________________________________________ >>> dev mailing list >>> d...@openvswitch.org >>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >>> >> _______________________________________________ >> dev mailing list >> d...@openvswitch.org >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev