On Fri, Feb 12, 2016 at 02:38:45PM +0100, Pavel Březina wrote: > No. Running CI should be a part of reviewer's responsibility and we should not accept patches without a passing CI link -or- a link where the failure is clearly not related to the diff (maybe a buggy test or an ill CI VM) -or- where the patch is clearly not covered by our tests (maybe a docs fix).
I ran the CI tests for you: http://sssd-ci.duckdns.org/logs/job/37/32/summary.html but feel free to ask me for instructions next time. And yes, it's suboptimal that pre-commit CI must be run manually and requires access to RH VPN, but that's the way it is for now.. _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org