Yaniv Bronhaim has posted comments on this change.
Change subject: Remove vdsm directories on uninstallation ......................................................................
Patch Set 1: -Code-Review
(1 comment)
btw, what about all the hooks directories as %dir %{_libexecdir}/%{vdsm_name}/hooks, don't we need to remove those as well?
.................................................... 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} don't we need also:
%dir %{_datadir}/%{vdsm_name}/storage/protect 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*