| Message ID | 1469288314-27387-1-git-send-email-rmoats@us.ibm.com |
|---|---|
| State | Changes Requested |
| Headers | show |
On Sat, Jul 23, 2016 at 10:38:34AM -0500, Ryan Moats wrote: > Having a separate debian package for deploying > the ovn-controller-vtep binary enables the ability > to assign specific nodes the role of communicating > with VTEP enabled TORs. > > Change-Id: Ia36aea7d89bd011a57918820b2a9f6e3469b3e04 > Signed-off-by: Ryan Moats <rmoats@us.ibm.com> Should there be an initscript to start and stop it? There's inconsistent use of tabs in the automake.mk change.
Ben Pfaff <blp@ovn.org> wrote on 07/27/2016 03:27:22 PM: > From: Ben Pfaff <blp@ovn.org> > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/27/2016 03:27 PM > Subject: Re: [ovs-dev] [PATCH] ovn: Add ovn-controller-vtep debian package > > On Sat, Jul 23, 2016 at 10:38:34AM -0500, Ryan Moats wrote: > > Having a separate debian package for deploying > > the ovn-controller-vtep binary enables the ability > > to assign specific nodes the role of communicating > > with VTEP enabled TORs. > > > > Change-Id: Ia36aea7d89bd011a57918820b2a9f6e3469b3e04 > > Signed-off-by: Ryan Moats <rmoats@us.ibm.com> > > Should there be an initscript to start and stop it? > > There's inconsistent use of tabs in the automake.mk change. > Indeed there should be an initscript and I see the tab problems. I'll spin a V2 at some point over the next couple of days... Ryan
diff --git a/debian/automake.mk b/debian/automake.mk index 7b5b3aa..ea99407 100644 --- a/debian/automake.mk +++ b/debian/automake.mk @@ -58,6 +58,8 @@ EXTRA_DIST += \ debian/ovn-central.postinst \ debian/ovn-central.postrm \ debian/ovn-central.template \ + debian/ovn-controller-vtep.install \ + debian/ovn-controller-vtep.manpages \ debian/ovn-common.install \ debian/ovn-common.manpages \ debian/ovn-docker.install \ diff --git a/debian/control b/debian/control index 37aff8d..39884bc 100644 --- a/debian/control +++ b/debian/control @@ -113,6 +113,21 @@ Description: OVN common components . ovn-common provides components required by other OVN packages. +Package: ovn-controller-vtep +Architecture: linux-any +Depends: ovn-common (= ${binary:Version}), + ${misc:Depends}, + ${shlibs:Depends} +Description: OVN vtep controller + ovn-controller-vtep is the local controller daemon in + OVN, the Open Virtual Network, for VTEP enabled physical switches. + It connects up to the OVN Southbound database over the OVSDB protocol, + and down to the VTEP database over the OVSDB protocol. + . + ovn-controller-vtep provides the ovn-controller-vtep binary for controlling + vtep gateways. + + Package: ovn-host Architecture: linux-any Depends: openvswitch-switch (= ${binary:Version}), diff --git a/debian/ovn-controller-vtep.install b/debian/ovn-controller-vtep.install new file mode 100644 index 0000000..1d208f3 --- /dev/null +++ b/debian/ovn-controller-vtep.install @@ -0,0 +1 @@ +usr/bin/ovn-controller-vtep diff --git a/debian/ovn-controller-vtep.manpages b/debian/ovn-controller-vtep.manpages new file mode 100644 index 0000000..7873017 --- /dev/null +++ b/debian/ovn-controller-vtep.manpages @@ -0,0 +1 @@ +ovn/controller-vtep/ovn-controller-vtep.8
Having a separate debian package for deploying the ovn-controller-vtep binary enables the ability to assign specific nodes the role of communicating with VTEP enabled TORs. Change-Id: Ia36aea7d89bd011a57918820b2a9f6e3469b3e04 Signed-off-by: Ryan Moats <rmoats@us.ibm.com> --- debian/automake.mk | 2 ++ debian/control | 15 +++++++++++++++ debian/ovn-controller-vtep.install | 1 + debian/ovn-controller-vtep.manpages | 1 + 4 files changed, 19 insertions(+) create mode 100644 debian/ovn-controller-vtep.install create mode 100644 debian/ovn-controller-vtep.manpages