[Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports

[Ian Jackson]

There is a limit in cr-daily-branch, but none in cs-bisection-step.

adhoc-revtuple-generator could usefully have this built in but that's
not so simple, so do it again here.  We already slurp the whole thing
into core so from a resource usage point of view we might as well do
the length check here too.

Reported-by: David Vrabel 
Signed-off-by: Ian Jackson 
---
 cs-bisection-step |5 +
 1 file changed, 5 insertions(+)

diff --git a/cs-bisection-step b/cs-bisection-step
index e51babd..928a147 100755
--- a/cs-bisection-step
+++ b/cs-bisection-step
@@ -898,6 +898,11 @@ END
"./adhoc-revtuple-generator -S @revtuplegenargs $rts";
 my $revinfo= `$revrune`;
 if (!$?) {
+   if (length($revinfo) > 24000) {
+   $revinfo = <

Re: [Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports

[Ian Campbell]

On Mon, 2015-11-30 at 13:41 +, Ian Jackson wrote:
> There is a limit in cr-daily-branch, but none in cs-bisection-step.
> 
> adhoc-revtuple-generator could usefully have this built in but that's
> not so simple, so do it again here.  We already slurp the whole thing
> into core so from a resource usage point of view we might as well do
> the length check here too.
> 
> Reported-by: David Vrabel 
> Signed-off-by: Ian Jackson 

Acked-by: Ian Campbell 

> +(Revision log too long, ommitted.)

My MUA's spell checker thinks it is "omitted".

Ian.

___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports

[Ian Jackson]

Ian Campbell writes ("Re: [OSSTEST PATCH] cs-bisection-step: Limit size of 
revision log included in reports"):
> Acked-by: Ian Campbell 
> 
> > +(Revision log too long, ommitted.)
> 
> My MUA's spell checker thinks it is "omitted".

Thanks.  I have ommittedd the spurrious lettter.

Ian.

___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel