Hey Ben, Thanks for the review and for retesting this. I pushed this to master in the morning with changes to the commit message.
thanx! mehak On Thu, Jun 21, 2012 at 11:30 AM, Mehak Mahajan <mmaha...@nicira.com> wrote: > sure will do that.. > > > On Thu, Jun 21, 2012 at 10:12 AM, Ben Pfaff <b...@nicira.com> wrote: > >> On Wed, Jun 20, 2012 at 09:24:34PM -0700, Mehak Mahajan wrote: >> > This patch reapplies the changes that were reverted with the commit >> 59efa47 >> > (Revert DSCP update changes.). It also addresses the problem introduced >> by >> > the original commits, cd8fca2 ((jsonrpc: Correctly setting the dscp >> value >> > before reconnect.) and b2e18d (No need to restart DB / OVS on changing >> > dscp value.), that caused numerous unit test failures on some systems >> (as >> > diagnosed by valgrind). >> > >> > Signed-off-by: Mehak Mahajan <mmaha...@nicira.com> >> >> I tested it and it worked for me. Thank you. >> >> Please update the commit message, though, to include everything from >> the original commit message, rather than just referring to it. >> >> Thanks, >> >> Ben. >> > >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
