From b78d6bbf0b9593ee204acca9ebaa2f81130297ea Mon Sep 17 00:00:00 2001 From: Mauro Doglio Date: Mon, 7 Sep 2015 13:25:21 +0200 Subject: [PATCH] Bug 1182464 - Make the failure_line signature a TextField. I also removed a limit on the length of `message` because it didn't make sense. --- treeherder/model/migrations/0001_initial.py | 4 ++-- treeherder/model/models.py | 4 ++-- .../model/sql/template_schema/treeherder_reference_1.sql.tmpl | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/treeherder/model/migrations/0001_initial.py b/treeherder/model/migrations/0001_initial.py index e22dce723..abd735339 100644 --- a/treeherder/model/migrations/0001_initial.py +++ b/treeherder/model/migrations/0001_initial.py @@ -104,8 +104,8 @@ class Migration(migrations.Migration): ('subtest', models.TextField(null=True, blank=True)), ('status', models.CharField(max_length=7, choices=[('PASS', 'PASS'), ('FAIL', 'FAIL'), ('OK', 'OK'), ('ERROR', 'ERROR'), ('TIMEOUT', 'TIMEOUT'), ('CRASH', 'CRASH'), ('ASSERT', 'ASSERT'), ('SKIP', 'SKIP'), ('NOTRUN', 'NOTRUN')])), ('expected', models.CharField(blank=True, max_length=7, null=True, choices=[('PASS', 'PASS'), ('FAIL', 'FAIL'), ('OK', 'OK'), ('ERROR', 'ERROR'), ('TIMEOUT', 'TIMEOUT'), ('CRASH', 'CRASH'), ('ASSERT', 'ASSERT'), ('SKIP', 'SKIP'), ('NOTRUN', 'NOTRUN')])), - ('message', models.TextField(max_length=255, null=True, blank=True)), - ('signature', models.CharField(max_length=255, null=True, blank=True)), + ('message', models.TextField(null=True, blank=True)), + ('signature', models.TextField(null=True, blank=True)), ('level', models.CharField(blank=True, max_length=8, null=True, choices=[('PASS', 'PASS'), ('FAIL', 'FAIL'), ('OK', 'OK'), ('ERROR', 'ERROR'), ('TIMEOUT', 'TIMEOUT'), ('CRASH', 'CRASH'), ('ASSERT', 'ASSERT'), ('SKIP', 'SKIP'), ('NOTRUN', 'NOTRUN')])), ('stack', models.TextField(null=True, blank=True)), ('stackwalk_stdout', models.TextField(null=True, blank=True)), diff --git a/treeherder/model/models.py b/treeherder/model/models.py index 35c909fb0..2b66fb587 100644 --- a/treeherder/model/models.py +++ b/treeherder/model/models.py @@ -549,8 +549,8 @@ class FailureLine(models.Model): subtest = models.TextField(blank=True, null=True) status = models.CharField(max_length=7, choices=STATUS_CHOICES) expected = models.CharField(max_length=7, choices=STATUS_CHOICES, blank=True, null=True) - message = models.TextField(max_length=255, blank=True, null=True) - signature = models.CharField(max_length=255, blank=True, null=True) + message = models.TextField(blank=True, null=True) + signature = models.TextField(blank=True, null=True) level = models.CharField(max_length=8, choices=STATUS_CHOICES, blank=True, null=True) stack = models.TextField(blank=True, null=True) stackwalk_stdout = models.TextField(blank=True, null=True) diff --git a/treeherder/model/sql/template_schema/treeherder_reference_1.sql.tmpl b/treeherder/model/sql/template_schema/treeherder_reference_1.sql.tmpl index 043a5ae74..06bbdc0a4 100644 --- a/treeherder/model/sql/template_schema/treeherder_reference_1.sql.tmpl +++ b/treeherder/model/sql/template_schema/treeherder_reference_1.sql.tmpl @@ -688,7 +688,7 @@ DROP TABLE IF EXISTS `failure_line`; `status` varchar(7) NOT NULL, `expected` varchar(7) NULL, `message` longtext NULL, - `signature` varchar(255) NULL, + `signature` longtext NULL, `level` varchar(8) NULL, `stack` longtext NULL, `stackwalk_stdout` longtext NULL,