[ovs-dev] [PATCH 1/1] OVN: Firewall service files for OVN

[Ben Pfaff]

On Wed, Feb 01, 2017 at 09:59:22AM +0100, Marcin Mirecki wrote:
> From adeba4e31c616f2212a0d8f5bac173ed1d42552a Mon Sep 17 00:00:00 2001
> From: Marcin Mirecki <mmirecki at redhat.com>
> Date: Tue, 31 Jan 2017 16:33:36 +0100
> Subject: [PATCH] Firewall service files for OVN
> 
> Firewall service files allowing to open firewalld
> ports required for running OVN
> 
> Signed-off-by: Marcin Mirecki<mmirecki at redhat.com>
> ---
>  ovn/ovn-central-firewall-service.xml | 7 +++++++
>  ovn/ovn-host-firewall-service.xml    | 6 ++++++
>  2 files changed, 13 insertions(+)
>  create mode 100644 ovn/ovn-central-firewall-service.xml
>  create mode 100644 ovn/ovn-host-firewall-service.xml
> 
> diff --git a/ovn/ovn-central-firewall-service.xml
> b/ovn/ovn-central-firewall-service.xml
> new file mode 100644
> index 0000000..e7f871d
> --- /dev/null
> +++ b/ovn/ovn-central-firewall-service.xml
> @@ -0,0 +1,7 @@
> +<?xml version="1.0" encoding="utf-8"?>
> +<service>
> +  <short>ovn-central-firewall-service</short>
> +  <description>Firewall service for ovn central</description>
> +  <port protocol="tcp" port="6641"/>
> +  <port protocol="tcp" port="6442"/>
> +</service>
> diff --git a/ovn/ovn-host-firewall-service.xml
> b/ovn/ovn-host-firewall-service.xml
> new file mode 100644
> index 0000000..f606890
> --- /dev/null
> +++ b/ovn/ovn-host-firewall-service.xml
> @@ -0,0 +1,6 @@
> +<?xml version="1.0" encoding="utf-8"?>
> +<service>
> +  <short>ovn-host-firewall-service</short>
> +  <description>Firewall service for ovn host</description>
> +  <port protocol="udp" port="6081"/>
> +</service>

Thanks for the patch!

The patch doesn't document how to use these files, nor does it does add
anything to "make install" to install them, nor does it add anything to
distribution packaging to install them.  I think that it should do at
least one of those.