On Fri, 3 May 2019 at 11:19, Gurucharan Shetty <g...@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 <g...@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 <aatt...@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 > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev