This reverts commit 59e8cb8a053d50f49629be8b6fd614562d066404.

Commit broke the package install on a clean system and also doesn't
seem to manage access rights for created symlinks correctly.

Revert it until a proper solution is proposed.

Reported-at: 
https://mail.openvswitch.org/pipermail/ovs-dev/2022-December/400045.html
Reported-by: Roi Dayan <r...@nvidia.com>
Signed-off-by: Ilya Maximets <i.maxim...@ovn.org>
---
 rhel/openvswitch-fedora.spec.in | 27 ++++-----------------------
 1 file changed, 4 insertions(+), 23 deletions(-)

diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index 4a3e6294b..de6cb30ea 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -238,6 +238,8 @@ rm -rf $RPM_BUILD_ROOT/%{_datadir}/openvswitch/python/
 
 install -d -m 0755 $RPM_BUILD_ROOT/%{_sharedstatedir}/openvswitch
 
+touch $RPM_BUILD_ROOT%{_sysconfdir}/openvswitch/conf.db
+touch $RPM_BUILD_ROOT%{_sysconfdir}/openvswitch/.conf.db.~lock~
 touch $RPM_BUILD_ROOT%{_sysconfdir}/openvswitch/system-id.conf
 
 install -p -m 644 -D selinux/openvswitch-custom.pp \
@@ -326,27 +328,6 @@ if [ $1 -eq 1 ]; then
 fi
 %endif
 
-# Ensure that /etc/openvswitch/conf.db links to /var/lib/openvswitch,
-# moving an existing file if there is one.
-#
-# Ditto for .conf.db.~lock~.
-for base in conf.db .conf.db.~lock~; do
-    new=/var/lib/openvswitch/$base
-    old=/etc/openvswitch/$base
-    if test -f $old && test ! -e $new; then
-        mv $old $new
-    fi
-    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}
     # This may not enable openvswitch service or do daemon-reload.
     %systemd_post %{name}.service
@@ -432,8 +413,8 @@ fi
 %endif
 %dir %{_sysconfdir}/openvswitch
 %{_sysconfdir}/openvswitch/default.conf
-%config %ghost %{_sharedstatedir}/openvswitch/conf.db
-%ghost %{_sharedstatedir}/openvswitch/.conf.db.~lock~
+%config %ghost %{_sysconfdir}/openvswitch/conf.db
+%ghost %{_sysconfdir}/openvswitch/.conf.db.~lock~
 %config %ghost %{_sysconfdir}/openvswitch/system-id.conf
 %config(noreplace) %{_sysconfdir}/sysconfig/openvswitch
 %defattr(-,root,root)
-- 
2.38.1

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to