[SSSD] [PATCH v2] CI: Allow disabling distro-(in)dependent tests
Jakub Hrozek
jhrozek at redhat.com
Thu Sep 18 07:07:39 UTC 2014
On 18 Sep 2014, at 07:18, Lukas Slebodnik <lslebodn at redhat.com> wrote:
> On (15/09/14 17:35), Nikolai Kondrashov wrote:
>> On 09/12/2014 10:21 PM, Lukas Slebodnik wrote:
>>> On (12/09/14 15:58), Nikolai Kondrashov wrote:
>>>> On 09/12/2014 02:36 PM, Lukas Slebodnik wrote:
>>>>> On (12/09/14 14:22), Nikolai Kondrashov wrote:
>>>>>> On 09/12/2014 01:47 PM, Michal Židek wrote:
>>>>>>> On 09/12/2014 12:05 PM, Nikolai Kondrashov wrote:
>>>>>>>> I understand, but distro-dependent/independent test running is not a target
>>>>>>>> in itself. The target is to have the CI servers available for routine use.
>>>>>>>>
>>>>>>>> Currently rigorous test runs typically take around an hour. Once Jakub goes
>>>>>>>> through pending and acknowledged patches and merges them in quick
>>>>>>>> succession, the CI servers become unavailable for most of the day, running
>>>>>>>> rigorous post-commit tests. Having people submit jobs for at least moderate
>>>>>>>> tests will also slow it down.
>>>>>>>>
>>>>>>>
>>>>>>> Maybe we could do something like Lukas proposed. Increasing the
>>>>>>> virtual machines RAM and use tmpfs. Maybe the speed problem
>>>>>>> will disappear.
>>>>>>
>>>>>> I can try increasing RAM on all five CI hosts and enabling mock's tmpfs plugin
>>>>>> on them. How much physical memory would you like me to configure them with?
>>>>>>
>>>>> I would use say 2 GiB should be enough (4x more than current amount).
>>>>> If there is problem we can increase to 3GiB.
>>>>
>>>> I ran the rigorous tests with 2GiB of RAM and default tmpfs plugin configuration.
>>>>
>>>> The run time on each machine has reduced to about 24 minutes, which is quite
>>>> close to the maximum of 20 minutes I got with the patch being discussed, and
>>>> could be acceptable. It also indicates that the bottleneck was disk I/O, IMO.
>>>>
>>>> However, it turned out that RHEL6 cannot install our dependencies on tmpfs:
>>>> http://sssd-ci.duckdns.org/logs/job/0/77/rhel6/ci-build-debug/ci-mock-fedora20.log
>>>>
>>> There is problem with installation package systemd, because it uses capbilities
>>> in spec file.
>>> %caps(cap_dac_override,cap_sys_ptrace=pe) %{_bindir}/systemd-detect-virt
>>>
>>> Solution is easy: We should not try to rebuild srpm for fedora on rhel.
>>> I proposed it in another mail thread that on rhel6 we should try to rebuild
>>> srpm only in epel6 mock buid root.
>>
>> Sure, I can make a patch adding skipping Fedora mock build on RHEL6.
>>
>> However, what about this one? Can we get it merged? If not, then why?
>>
> 1) Don't wonder that you did not get reply for log time.
> You had replied to the wrong mail. I didn't review you patches.
>
> 2) I read the whole thread and I found answers to your question in the two
> Michal's mails:
> https://lists.fedorahosted.org/pipermail/sssd-devel/2014-September/021560.html
> https://lists.fedorahosted.org/pipermail/sssd-devel/2014-September/021576.html
> Try to read therm one more time. If it does not help, I am pretty sure, Michal
> will be glad to exaplain it at today meeting.
>
Let’s. Stay. Technical.
And yeah, let’s discuss the issue over the phone because this thread is clearly going nowhere with 20+ e-mails.
More information about the sssd-devel
mailing list