<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 22, 2014 at 11:59 AM, Vojtech Juranek <span dir="ltr"><<a href="mailto:vjuranek@redhat.com" target="_blank">vjuranek@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> Does anyone volunteer for ISPN-4813?<br>
<br>
</span>me not:-) as this one is IMHO little bit tricky - it's (at least partially)<br>
related to the question "which size is the correct size?". Was there any clear<br>
conclusion in recent " About size()" thread?<br>
<span class=""><br>
<br>
> > actually, if there's a failing test and it's clear which commit has caused<br>
> > it, we can stop merging PRs from the developer<br>
> > who has introduced the regression until he fix it. This should be clear<br>
> > message, that the first thing he should<br>
> > work on is to fix the test. IMHO we can start with it now and don't need<br>
> > green testsuite for it<br>
><br>
> In theory that sounds good, but who do we stop merging for on account the<br>
> current failures? :)<br>
<br>
</span>probably nobody, we need to find volunteers in this case. I meant it only for<br>
issues where it's clear who introduced the regression (e.g. me for recently<br>
failing NodeAuth*IT tests). As above, not a complete solution, but rather a<br>
small step to move to the state where we would like to be<br></blockquote><div><br></div><div>Yeah, that's what I meant, most of the time it's hard to track down who caused a specific failure.</div><div><br></div><div>BTW, I forgot to mention something in the previous email: when you investigate a test failure please mark the test as investigated in TeamCity (preferably with a link to an issue in JIRA). And if it's an intermittent failure, make sure it's not marked as fixed automatically (TC is a little greedy here, it thinks the test is fixed the moment it doesn't fail in a build).</div><div><br></div><div>Cheers</div><div>Dan</div><div><br></div></div></div></div>