[ovs-dev] [PATCH 1/2] fedora: Ability to auto enable openvswitch service.

Ansis Atteka ansisatteka at gmail.com
Thu May 9 22:27:26 UTC 2019


On Fri, 3 May 2019 at 11:19, Gurucharan Shetty <guru at ovn.org> wrote:
>
> We currently have rhel/openvswitch.spec.in that automatically
> enables openvswitch service when the package is installed using
> chkconfig.
>
> But fedora rpm may not enable openvswitch service automatically.
> The macro currently being used in fedora rpm (systemd_post) will
> look for preset files in /etc/systemd/system-preset/ to figure
> out whether openvswitch service needs to be automatically enabled.
> But, the fedora package does not provide such a file. The argument
> is that people may want to install the package for binaries and
> not necessarily to run OVS.
>
> If someone now wants to install the fedora package and automatically
> enable openvswitch, he will have to create a new package that OVS
> package depends on to install the preset file. This is unwieldy.

Is this the preset list you are referring to -
https://src.fedoraproject.org/rpms/fedora-release/blob/master/f/90-default.preset
? Maybe Aaron can comment if openvswitch should get in that list?
>
> This commit, provides a rpm build time option to enable the openvswitch
> service automatically. If you now run the below command, openvswitch
> service will be automatically enabled during package installation.
>
> make rpm-fedora RPMBUILD_OPT="--with autoenable"
>
> Signed-off-by: Gurucharan Shetty <guru at ovn.org>

Since you have #ifdef'ed out your newly introduced code and not
changed the default behavior, then this patch should not affect the
packages distributed by Fedora. So:

Acked-by: Ansis Atteka <aatteka at ovn.org>



> ---
>  rhel/openvswitch-fedora.spec.in | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
> index ce728b4..e8165f9 100644
> --- a/rhel/openvswitch-fedora.spec.in
> +++ b/rhel/openvswitch-fedora.spec.in
> @@ -32,6 +32,9 @@
>  # This is enabled by default for versions of the distribution that
>  # have Python 3 by default (Fedora > 22).
>  %bcond_with build_python3
> +# If there is a need to automatically enable the package after installation,
> +# specify the "--with autoenable"
> +%bcond_with autoenable
>
>  # Enable PIE, bz#955181
>  %global _hardened_build 1
> @@ -382,6 +385,7 @@ fi
>  %endif
>
>  %if 0%{?systemd_post:1}
> +    # This may not enable openvswitch service or do daemon-reload.
>      %systemd_post %{name}.service
>  %else
>      # Package install, not upgrade
> @@ -390,6 +394,11 @@ fi
>      fi
>  %endif
>
> +%if %{with autoenable}
> +    systemctl daemon-reload
> +    systemctl enable openvswitch
> +%endif
> +
>  %post selinux-policy
>  %selinux_modules_install -s targeted %{_datadir}/selinux/packages/%{name}/openvswitch-custom.pp
>
> --
> 1.9.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev


More information about the dev mailing list