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..