Hi Andy, thanks for fixing this. On 9 March 2016 at 18:05, Andy Zhou <az...@ovn.org> wrote: > From: Daniele Di Proietto <diproiet...@vmware.com>
As a general rule, it's nice to have the commit id direct in the message to show the upstream commit being backported. I've been doing it like this at the start of the message: Upstream: e88b97cbbec3497a2272ace805255d1c87230743 This helps reviewers compare the original commit with the backport to look for discrepancies. > Recently some testcases have been failing in travis because of a warning > related to the use of an L3 device (OpenVZ specific) inside the workers. > > To get travis tests working again we can move to the newer container > infrastructure: this commit does that. > > The disadvantage is that there's no sudo access anymore, but we can > install packages with the apt plugin, and we shouldn't use root for > anything else > > Also, since we're building DPDK with vhost-user (not vhost-cuse), > libfuse-dev is not needed anymore. > > Tested-at: https://travis-ci.org/ddiproietto/ovs/builds/81764972 > > [back ported to branch-2.4 by azhou] > > Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> > CC: Joe Stringer <joestrin...@nicira.com> > Acked-by: Ben Pfaff <b...@nicira.com> > Signed-off-by: Andy Zhou <az...@ovn.org> <snip> > @@ -26,4 +38,4 @@ script: ./.travis/build.sh $OPTS > notifications: > email: > recipients: > - - secure: > KnZ6yDXDcC4VoiI04ZYR4sRTln7q16aXA7gVBa/M1jPWxl3BiTi+4idVE5bgrB1AK5iUwUXN6LQpjOdFDw1U/D+sKt+xmVG5MyLaTYIFp1TUOgtSGeiG3IUhpu125PN1i2EhXNqANyWyStCiISDvJkDe4D/tbBehip1AEBuQONk= > + - bu...@openvswith.org This fragment wasn't in the original commit... was it intended? (should it be a separate backport commit?) Other than this, LGTM. Acked-by: Joe Stringer <j...@ovn.org> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
