diff mbox

[ovs-dev,5/5] debian: Add a package for OVN docker drivers.

Message ID 1450201926-29276-5-git-send-email-guru@ovn.org
State Not Applicable
Headers show

Commit Message

Gurucharan Shetty Dec. 15, 2015, 5:52 p.m. UTC
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
---
 debian/.gitignore                     |    1 +
 debian/automake.mk                    |    1 +
 debian/control                        |   18 ++++++++++++++++++
 debian/openvswitch-ovn-docker.install |    2 ++
 4 files changed, 22 insertions(+)
 create mode 100644 debian/openvswitch-ovn-docker.install

Comments

Ben Pfaff Dec. 16, 2015, 9:55 a.m. UTC | #1
On Tue, Dec 15, 2015 at 09:52:06AM -0800, Gurucharan Shetty wrote:
> Signed-off-by: Gurucharan Shetty <guru@ovn.org>

I think that this is architecture-independent (there's no actual
binaries in it, just Python source), so that it should be Architecture:
"all" instead of "linux-any".

Is there a reason someone wouldn't want to install this?  That is, is
there a reason not to fold it into the -common package?
Gurucharan Shetty Dec. 16, 2015, 3:41 p.m. UTC | #2
On 16 December 2015 at 01:55, Ben Pfaff <blp@ovn.org> wrote:

> On Tue, Dec 15, 2015 at 09:52:06AM -0800, Gurucharan Shetty wrote:
> > Signed-off-by: Gurucharan Shetty <guru@ovn.org>
>
> I think that this is architecture-independent (there's no actual
> binaries in it, just Python source), so that it should be Architecture:
> "all" instead of "linux-any".
>
I will correct it.


>
> Is there a reason someone wouldn't want to install this?  That is, is
> there a reason not to fold it into the -common package?
>
One reason I separated it out:
Eventually, a startup script to automatically start these drivers during
system startup.
If you feel that we are adding too many packages, I will reconsider the
approach.
Ben Pfaff Dec. 23, 2015, 5:30 p.m. UTC | #3
On Wed, Dec 16, 2015 at 07:41:01AM -0800, Guru Shetty wrote:
> On 16 December 2015 at 01:55, Ben Pfaff <blp@ovn.org> wrote:
> > Is there a reason someone wouldn't want to install this?  That is, is
> > there a reason not to fold it into the -common package?
> >
> One reason I separated it out:
> Eventually, a startup script to automatically start these drivers during
> system startup.

That's an OK reason.
diff mbox

Patch

diff --git a/debian/.gitignore b/debian/.gitignore
index d600dbc..781e3ed 100644
--- a/debian/.gitignore
+++ b/debian/.gitignore
@@ -15,6 +15,7 @@ 
 /openvswitch-ovn-central
 /openvswitch-ovn-common
 /openvswitch-ovn-controller
+/openvswitch-ovn-docker
 /openvswitch-pki
 /openvswitch-switch
 /openvswitch-switch.copyright
diff --git a/debian/automake.mk b/debian/automake.mk
index bdfacc9..d0a35ad 100644
--- a/debian/automake.mk
+++ b/debian/automake.mk
@@ -37,6 +37,7 @@  EXTRA_DIST += \
 	debian/openvswitch-ovn-controller.postinst \
 	debian/openvswitch-ovn-controller.postrm \
 	debian/openvswitch-ovn-controller.template \
+	debian/openvswitch-ovn-docker.install \
 	debian/openvswitch-pki.dirs \
 	debian/openvswitch-pki.postinst \
 	debian/openvswitch-pki.postrm \
diff --git a/debian/control b/debian/control
index d640bac..48e5c0f 100644
--- a/debian/control
+++ b/debian/control
@@ -144,6 +144,24 @@  Description: OVN central components
  openvswitch-ovn-central provides the userspace daemons, utilities and
  databases for OVN that is run at a central location.
 
+Package: openvswitch-ovn-docker
+Architecture: linux-any
+Depends: openvswitch-switch (= ${binary:Version}),
+         openvswitch-common (= ${binary:Version}),
+         python (>= 2.7),
+         python-openvswitch (= ${source:Version}),
+         openvswitch-ovn-common (= ${binary:Version}),
+         ${misc:Depends},
+         ${python:Depends},
+         ${shlibs:Depends}
+Description: OVN Docker drivers
+ OVN, the Open Virtual Network, is a system to support virtual network
+ abstraction.  OVN complements the existing capabilities of OVS to add
+ native support for virtual network abstractions, such as virtual L2 and L3
+ overlays and security groups.
+ .
+ openvswitch-ovn-docker provides the docker drivers for OVN.
+
 Package: openvswitch-ipsec
 Architecture: linux-any
 Depends: ipsec-tools (>=0.8~alpha20101208),
diff --git a/debian/openvswitch-ovn-docker.install b/debian/openvswitch-ovn-docker.install
new file mode 100644
index 0000000..5833067
--- /dev/null
+++ b/debian/openvswitch-ovn-docker.install
@@ -0,0 +1,2 @@ 
+usr/bin/ovn-docker-overlay-driver
+usr/bin/ovn-docker-underlay-driver