Dan Kenigsberg has posted comments on this change.
Change subject: Remove vdsm directories on uninstallation ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
yes, we should explicitly own
%{_libexecdir}/%{vdsm_name}/hooks
.................................................... File vdsm.spec.in Line 794: %dir %attr(-, %{vdsm_user}, %{vdsm_group}) @vdsmrepo@ Line 795: %ghost %dir %attr(-, %{vdsm_user}, %{vdsm_group}) @vdsmrepo@/hsm-tasks Line 796: %ghost %dir %attr(-, %{vdsm_user}, %{vdsm_group}) @vdsmrepo@/mnt Line 797: %dir %{_libexecdir}/%{vdsm_name} Line 798: %dir %{_datadir}/%{vdsm_name} no. it exists only in git, not on runtime. Line 799: %dir %{_datadir}/%{vdsm_name}/netconf Line 800: %dir %{_datadir}/%{vdsm_name}/storage Line 801: %dir %{_datadir}/%{vdsm_name}/storage/imageRepository Line 802: %{_datadir}/%{vdsm_name}/alignmentScan.py*