On 20 July 2017 at 16:12, Bohuslav Kabrda <bkab...@redhat.com> wrote:
> On Thu, Jul 20, 2017 at 7:51 AM, Nick Coghlan <ncogh...@gmail.com> wrote:
>> * checking for refleaks means we can't run parallel tests across
>> multiple processes. If we were to move refleak testing out to an
>> integration test in Taskotron, then the tests could be run with the
>> "-j4" option, significantly reducing the time needed to run the %check
>> step
>>
>
> If it's possible to move this testing to taskotron, then it sounds ok to me.

Based on my changes to the sclo-python spec, I can confirm that
passing "-j0" (which autoscales to the number of CPUs +2 to allow for
blocking tests) cuts my local RPM build times from just under 40
minutes to just over 15.

So I'll use that to speed up the sclo-python bootstrapping builds, and
then we can wait for the dist-git move into Pagure and the
introduction of https://fedoraproject.org/wiki/Changes/InvokingTests
before looking at changing the main build.

>> * the `--failfast` option causes the tests to be aborted as soon as
>> one fails. While that doesn't make any difference in the success case,
>> it will mean faster turnaround cycles in the case where one fails. I'm
>> less sure of the value of this one, as the downside is that if you're
>> getting multiple failures, you only find out about them one at a time.
>
> I'd suggest to not use this.

Agreed, especially as it's easy enough to add in to the spec file if
you actually want it in a mock build for some reason.

>> * there's a `--slowest` flag in recent releases that lists the 10
>> slowest tests in a given run. As far as I know, it doesn't add much
>> overhead to the test execution process, and provides an indicator of
>> where to focus efforts if the goal is to make the tests more
>> efficient.
>
> Yeah, I think improving the performance of the slowest tests could be gain
> for everyone, not just for Fedora.

One caveat on this option is that it's a relatively recent addition
that was never backported to the 3.5 branch, so it's only applicable
to builds of 3.6+.

Cheers,
Nick.

-- 
Nick Coghlan   |   ncogh...@gmail.com   |   Brisbane, Australia
_______________________________________________
python-devel mailing list -- python-devel@lists.fedoraproject.org
To unsubscribe send an email to python-devel-le...@lists.fedoraproject.org

Reply via email to