Re: [ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-20 Thread Ben Pfaff
This package has a pretty narrow audience. I think that it's more likely that someone will install it by mistake than by intention, and if one really needs it, it's easy to enable it. On Mon, May 19, 2014 at 3:14 PM, David Yang wrote: > If someone were to install this package, we'd expect that t

[ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread David Yang
If someone were to install this package, we'd expect that they'd want this enabled since the default behavior after package install should be to start the daemons. Right now, the VTEP start script doesn't do anything unless this flag is set to true. Bug #1246486 Signed-off-by: David Yang --- de

Re: [ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread Ben Pfaff
David, thanks. Please add that information to the commit message. Justin and David, I see that you two co-authored the Debian packaging for the VTEP code. What was the original reason not to enable VTEP by default on install? Is the reason no longer valid? On Mon, May 19, 2014 at 03:14:40PM -0

Re: [ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread David Yang
If someone were to install this package, we'd expect that they'd want this enabled since the default behavior after package install should be to start the daemons. Right now, the VTEP start script doesn't do anything unless this flag is set to true. On Mon, May 19, 2014 at 3:00 PM, Ben Pfaff wrot

Re: [ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread Ben Pfaff
On Mon, May 19, 2014 at 02:52:35PM -0700, David Yang wrote: > Signed-off-by: David Yang Why? ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread Gurucharan Shetty
Looks good to me. If no one else have any comments, I will apply this in an hour.. On Mon, May 19, 2014 at 2:52 PM, David Yang wrote: > Signed-off-by: David Yang > --- > debian/openvswitch-vtep.default |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/debian/openvswitc

[ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread David Yang
Signed-off-by: David Yang --- debian/openvswitch-vtep.default |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/openvswitch-vtep.default b/debian/openvswitch-vtep.default index 2e888e7..5049aeb 100644 --- a/debian/openvswitch-vtep.default +++ b/debian/openvswitch-vtep.

[ovs-dev] [PATCH] debian/openvswitch-vtep.default: enable VTEP on install

2014-05-19 Thread David Yang
--- debian/openvswitch-vtep.default |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/openvswitch-vtep.default b/debian/openvswitch-vtep.default index 2e888e7..5049aeb 100644 --- a/debian/openvswitch-vtep.default +++ b/debian/openvswitch-vtep.default @@ -1,4 +1,4 @@ #