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

* [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports
       [not found] <565C3490.2070904@citrix.com>
@ 2015-11-30 13:41 ` Ian Jackson
  2015-11-30 13:57   ` Ian Campbell
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Jackson @ 2015-11-30 13:41 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson, Ian Campbell, David Vrabel

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 <david.vrabel@citrix.com>
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 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 = <<END;
+(Revision log too long, ommitted.)
+END
+		    }
                     summary_report("",$revinfo,-1);
                 }
             }
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread