Dan Kenigsberg has posted comments on this change.

Change subject: hooks: Include sudoers file in ethtool_options rpm
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

Toni, Ido: any idea how https://bugzilla.redhat.com/show_bug.cgi?id=1126478 
could be verified?

This seems to need backporting

http://gerrit.ovirt.org/#/c/35725/1/vdsm_hooks/ethtool_options/Makefile.am
File vdsm_hooks/ethtool_options/Makefile.am:

Line 19: #
Line 20: 
Line 21: EXTRA_DIST = \
Line 22:        ethtool_options.py      \
Line 23:        sudoers                 \
trailing tabs not standard in our project.
Line 24:        $(NULL)
Line 25: 
Line 26: install-data-local: install-data-sudoers
Line 27:        $(MKDIR_P) $(DESTDIR)$(vdsmhooksdir)/after_network_setup


-- 
To view, visit http://gerrit.ovirt.org/35725
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3f1529452050370c82eb2a2675b74c2e5136d427
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuzn...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to