From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: hooks: Add localdisk hook
......................................................................
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/71272/5/vdsm_hooks/localdisk/Makefile.am
File vdsm_hooks/localdisk/Makefile.am:
Line 43: $(RM) /usr/lib/udev/rules.d/12-vdsm-localdisk.rules
Line 44:
Line 45: install-data-sudoers:
Line 46: $(MKDIR_P) $(DESTDIR)$(sysconfdir)/sudoers.d
Line 47: $(INSTALL_DATA) $(srcdir)/50_vdsm_localdisk \
From CI:
07:54:37 /usr/bin/install -c -m 644 ./50_vdsm_localdisk \
07:54:37
/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/rpmbuild/BUILDROOT/vdsm-4.20.0-279.git7e4a773.el7.centos.x86_64/etc/sudoers.d/50_vdsm_localdisk
07:54:37 /usr/bin/install: cannot stat './50_vdsm_localdisk': No such file or
directory
Line 48: $(DESTDIR)$(sysconfdir)/sudoers.d/50_vdsm_localdisk
Line 49:
Line 50: uninstall-data-sudoers:
Line 51: $(RM) $(DESTDIR)$(sysconfdir)/sudoers.d/50_vdsm_localdisk
--
To view, visit
https://gerrit.ovirt.org/71272
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b6ad8e8ebb4cd71bff02157f28cd2be5c31ac35
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com>
Gerrit-HasComments: Yes