On 12/16/22 16:29, Timothy Redaelli wrote:
> In 59e8cb8a053d ("rhel: Move conf.db to /var/lib/openvswitch, using
> symlinks.")
> conf.db is created as empty file in /var/lib/openvswitch, if it doesn't
> exists, but this prevent ovsdb-server to start.
>
> This commit changes the previous behaviour to set
> /var/lib/openvswitch owner to openvswitch:hugetlbfs, if built with
> dpdk, or openvswitch:openvswitch.
>
> Fixes: 59e8cb8a053d ("rhel: Move conf.db to /var/lib/openvswitch, using
> symlinks.")
> Reported-at:
> https://mail.openvswitch.org/pipermail/ovs-dev/2022-December/400045.html
> Reported-by: Roi Dayan <[email protected]>
> Signed-off-by: Timothy Redaelli <[email protected]>
> ---
> rhel/openvswitch-fedora.spec.in | 12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
> index 8d692b36c..6c8813793 100644
> --- a/rhel/openvswitch-fedora.spec.in
> +++ b/rhel/openvswitch-fedora.spec.in
> @@ -340,12 +340,6 @@ for base in conf.db .conf.db.~lock~; do
> if test ! -e $old && test ! -h $old; then
> ln -s $new $old
> fi
> - touch $new
> -%if %{with dpdk}
> - chown openvswitch:hugetlbfs $new
> -%else
> - chown openvswitch:openvswitch $new
> -%endif
> done
>
> %if 0%{?systemd_post:1}
> @@ -506,7 +500,11 @@ fi
> %{_prefix}/lib/udev/rules.d/91-vfio.rules
> %endif
> %doc NOTICE README.rst NEWS rhel/README.RHEL.rst
> -/var/lib/openvswitch
> +%if %{with dpdk}
> +%attr(750,openvswitch,hugetlbfs) /var/lib/openvswitch
> +%else
> +%attr(750,openvswitch,openvswitch) /var/lib/openvswitch
> +%endif
> %attr(750,root,root) /var/log/openvswitch
> %ghost %attr(755,root,root) %{_rundir}/openvswitch
> %ghost %attr(644,root,root) %{_rundir}/openvswitch.useropts
Thanks, Timothy. This change seems to work.
Roi, could you, please, check if it solves the problem in your setup?
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
