[wildfly-dev] New invocation merge

Kabir Khan kabir.khan at jboss.com
Tue Jan 31 10:20:15 EST 2017


> On 31 Jan 2017, at 15:16, Jason T. Greene <jason.greene at redhat.com> wrote:
> 
> Right exactly. 
> 
> What does everyone think about a daily or weekly email of the test output to this list?
Daily would narrow down the range of suspicious commits. Although that information should still be available on TeamCity itself, I think this makes it more visible.

(I take it this is the elaboration about the grouping of the failures? :))

> 
> On Jan 31, 2017, at 9:14 AM, Kabir Khan <kabir.khan at jboss.com> wrote:
> 
>>> 
>>> On 31 Jan 2017, at 15:09, Jason T. Greene <jason.greene at redhat.com> wrote:
>>> 
>>> How about the tests are changed to require a sys prop. That way you don't need a branch.
>> 
>> So rather than @Ignore we add something like 
>> 
>> @BeforeClass
>> public static checkRunInvocationTests() {
>>   //WFLY-XXXX
>>   Assume.assumeTrue(System.getProperty("wildfly.temp.disabled-invocation"))
>> }
>> 
>>> 
>>>> On Jan 31, 2017, at 9:07 AM, Jason T. Greene <jason.greene at redhat.com> wrote:
>>>> 
>>>> I agree with this approach but we need a way to group these failures in such a way that we know whether or not a change has regressions. 
>> Can you elaborate a little bit?
>>>> 
>>>>> On Jan 31, 2017, at 9:03 AM, Kabir Khan <kabir.khan at jboss.com> wrote:
>>>>> 
>>>>> Hi,
>>>>> 
>>>>> The new invocation library, which is the basis of for the ejb client etc. is currently developed in a branch. There are still about 130-140 test failures, but the team feels it is time to merge to wildfly master at some stage later this week. This will get more visibility of the failures and also lower the barrier of entry for whoever can jump in and help fix the failures.
>>>>> 
>>>>> Are there any objections to merging this? 
>>>>> 
>>>>>> From my point of view we would need to @Ignore the failing tests since there are enough transient failures in our testsuite to make it hard to find usual suspects if the number of failed tests is large. We could set up another CI job against a branch which is master with the @Ignores reverted, and I could keep that up to date as I merge to master, so that we get a good picture of the current testsuite failures.
>>>>> 
>>>>> Thanks,
>>>>> 
>>>>> Kabir
>>>>> _______________________________________________
>>>>> wildfly-dev mailing list
>>>>> wildfly-dev at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/wildfly-dev
>>>> 
>>>> _______________________________________________
>>>> wildfly-dev mailing list
>>>> wildfly-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/wildfly-dev
>> 




More information about the wildfly-dev mailing list