[swift-dev] smoke test not very smoky?

mishal_shah mishal_shah at apple.com
Wed Oct 19 13:07:14 CDT 2016


Yes, I will fix it today by creating new preset for smoke test.

Mishal Shah

> On Oct 19, 2016, at 9:44 AM, Dave Abrahams via swift-dev <swift-dev at swift.org> wrote:
> 
> 
> on Wed Oct 19 2016, Dave Abrahams <swift-dev-AT-swift.org <http://swift-dev-at-swift.org/>> wrote:
> 
>> It still seems like, for a smoke test, we're doing way too much work.
>> This appears to be much more than what I get from build-script -t when
>> I run tests locally.  Maybe I'm misunderstanding the intended role of
>> our smoke tests, but since nobody is correcting me, I'm betting not.
> 
> 
> Someone wrote to me privately:
> 
>  "buildbot_linux_1404" preset used in Linux smoke test contains
>  "--long-test".
> 
> This seems wrong to me.  Can we fix it?
> 
>> 
>> Anyway, regardless of the explanation, what can be done about this?
>> Between a spurious failure in LLDB and the length of the smoke test it
>> took several hours to be able to merge something that could not
>> possibly break the build, which seems absurd.
>> 
>>> On Oct 18, 2016, at 11:44 PM, mishal_shah <mishal_shah at apple.com> wrote:
>>> 
>>> 
>>>> On Oct 18, 2016, at 8:29 PM, Michael Gottesman <mgottesman at apple.com> wrote:
>>>> 
>>>> 
>>>>> On Oct 18, 2016, at 6:40 PM, Dave Abrahams via swift-dev <swift-dev at swift.org> wrote:
>>>>> 
>>>>> 
>>>>> I thought the smoke test was supposed to be a fairly quick test
>>>>> that just covered the basics, but it seems to be doing plenty of *really
>>>>> long* tests, for example:
>>>>> https://ci.swift.org/job/swift-PR-Linux-smoke-test/1916/console
>>>>> which has already been running for over 30 minutes.  Am I missing
>>>>> something?
>>>> 
>>>> I think people have over time been adding more to the Linux tests. But I am not in the know. Mishal, do you know whats happening here?
>>> 
>>> This is most likely due to number cores we have for Linux bots, the new Linux bots are 12 cores vs 48 cores and we are running multiple executors on it. 
>>> 
>>> Old  -  48 cores / 4 executors = 12 cores
>>> New - 12 cores / 2 executors = 6 cores
>>> 
>>> Thanks, 
>>> Mishal Shah
>>>> 
>>>> Michael
>>>> 
>>>>> 
>>>>> -- 
>>>>> -Dave
>>>>> _______________________________________________
>>>>> swift-dev mailing list
>>>>> swift-dev at swift.org
>>>>> https://lists.swift.org/mailman/listinfo/swift-dev
>>>> 
>>> 
>> _______________________________________________
>> swift-dev mailing list
>> swift-dev at swift.org <mailto:swift-dev at swift.org>
>> https://lists.swift.org/mailman/listinfo/swift-dev <https://lists.swift.org/mailman/listinfo/swift-dev>
> 
> -- 
> -Dave
> 
> _______________________________________________
> swift-dev mailing list
> swift-dev at swift.org <mailto:swift-dev at swift.org>
> https://lists.swift.org/mailman/listinfo/swift-dev <https://lists.swift.org/mailman/listinfo/swift-dev>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-dev/attachments/20161019/d21f7f0d/attachment.html>


More information about the swift-dev mailing list