Giuseppe Vallarelli has posted comments on this change. Change subject: Added the hook dispatcher ......................................................................
Patch Set 2: (2 inline comments) .................................................... File hooks/hook-dispatcher Line 25: from collections import OrderedDict Line 26: from dulwich.repo import Repo Line 27: Line 28: Line 29: def ignore(ignored_hooks, current_hooks, args): All right. Line 30: """ Line 31: Return the hooks list without the given hooks Line 32: Line 33: :param ignored_hooks; csv lis tof the hooks to ignore Line 32: Line 33: :param ignored_hooks; csv lis tof the hooks to ignore Line 34: :param current_hooks: array with the currently available hooks Line 35: """ Line 36: ignored_hooks = set([hook.strip() for hook in ignored_hooks.split(',')]) I see. Line 37: logging.info(" Ignoring the hooks %s", ignored_hooks) Line 38: for hook in ignored_hooks: Line 39: match = filter(lambda h: re.match(hook, h), current_hooks) Line 40: if match: -- To view, visit http://gerrit.ovirt.org/15386 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iada3d1bef5357366d5f319561efac8ee85a614f0 Gerrit-PatchSet: 2 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Giuseppe Vallarelli <gvall...@redhat.com> Gerrit-Reviewer: Itamar Heim <ih...@redhat.com> Gerrit-Reviewer: Ohad Basan <oba...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches