[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