From: Ian Campbell <ian.campbell@citrix.com>
To: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: xen-devel@lists.xenproject.org
Subject: Re: [OSSTEST PATCH 4/6] Revert "cs-bisection-step: Abandon repro attempts after a bit"
Date: Wed, 20 May 2015 14:04:43 +0100 [thread overview]
Message-ID: <1432127083.12989.244.camel@citrix.com> (raw)
In-Reply-To: <1432126453-22068-5-git-send-email-ian.jackson@eu.citrix.com>
On Wed, 2015-05-20 at 13:54 +0100, Ian Jackson wrote:
> This safety catch is unnecessary and unhelpful.
>
> It is unnecessary because 489773b4 "Detect flailing" will detect
> attempts by the bisector to repeatedly run the same flight and hope
> for different results.
>
> It is unhelpful because it can happen for good reasons that a
> particular revision has been tested many times. In particular:
>
> - The osstest push gate input tree may have not been advanced for a
> long time and been failing its push gate.
>
> - The bisector may have (for some reason[1]) restarted with a new
> baseline, and the temporarally-stripy pass/fail requirement would
> then require the basis fail to be repro'd, again.
>
> [1] Currently this happens much more often than is desirable. This
> will be fixed in a moment.
>
> This reverts commit 2676277181599a889657354028b992379aa6142b.
Acked-by: Ian Campbell <ian.campbell@citrix.com>
next prev parent reply other threads:[~2015-05-20 13:05 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-20 12:54 [OSSTEST PATCH 0/6] Bisector fixes Ian Jackson
2015-05-20 12:54 ` [OSSTEST PATCH 1/6] cs-bisection-step: Refer to jobs we create just by job name Ian Jackson
2015-05-20 13:02 ` Ian Campbell
2015-05-20 12:54 ` [OSSTEST PATCH 2/6] cs-bisection-step: Clarify and correct flailing message Ian Jackson
2015-05-20 13:03 ` Ian Campbell
2015-05-20 12:54 ` [OSSTEST PATCH 3/6] cs-bisection-step: Flail detection: look only at our blessing Ian Jackson
2015-05-20 13:04 ` Ian Campbell
2015-05-20 12:54 ` [OSSTEST PATCH 4/6] Revert "cs-bisection-step: Abandon repro attempts after a bit" Ian Jackson
2015-05-20 13:04 ` Ian Campbell [this message]
2015-05-20 12:54 ` [OSSTEST PATCH 5/6] cs-bisection-step: Do not treat repro attempts as flail Ian Jackson
2015-05-20 13:05 ` Ian Campbell
2015-05-20 12:54 ` [OSSTEST PATCH 6/6] Revert "cs-bisection-step: allow -bisect blessed flights for basis pass" Ian Jackson
2015-05-20 13:06 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1432127083.12989.244.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.