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

Reply via email to