On 12/19/22 13:23, Ilya Maximets wrote:
> 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 <r...@nvidia.com>
>> Signed-off-by: Timothy Redaelli <tredae...@redhat.com>
>> ---
>> 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?
Meanwhile, I applied this patch as it is definitely an improvement.
Thanks!
>
> Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
