Fix build URL in cross reference

Correctly fetch the builder id
MariaDB · Oct 4, 2023 · d15030f · d15030f
1 parent 8bfc651
commit d15030f
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 3 deletions.
diff --git a/cross-reference/crossreference/cr/models.py b/cross-reference/crossreference/cr/models.py
@@ -5,6 +5,16 @@
 from django.db import connection
 from datetime import datetime
 
+class Builder(models.Model):
+    id = models.IntegerField(primary_key=True)
+    name = models.TextField()
+    description = models.TextField(null=True, blank=True)
+    name_hash = models.CharField(max_length=40, unique=True)
+
+    class Meta:
+        managed = False
+        db_table = 'builders'
+
 class TestRun(models.Model):
   branch = models.CharField(max_length=100, blank=True, null=True)
   revision = models.CharField(max_length=256, blank=True, null=True)
@@ -14,6 +24,11 @@ class TestRun(models.Model):
   typ = models.CharField(max_length=32)
   info = models.CharField(max_length=255, blank=True, null=True)
 
+  @property
+  def platform_name(self):
+      builder = Builder.objects.filter(name=self.platform).first()
+      return builder.id if builder else None
+
   class Meta:
     managed = False
     db_table = 'test_run'

diff --git a/cross-reference/crossreference/cr/router.py b/cross-reference/crossreference/cr/router.py
@@ -1,12 +1,12 @@
-from .models import TestRun, TestFailure
+from .models import TestRun, TestFailure, Builder
 
 class MariaDBRouter:
     """
     Database router directing TestRun and TestFailure models to MariaDB.
     """
 
     def db_for_read(self, model, **hints):
-        if model in [TestRun, TestFailure]:
+        if model in [TestRun, TestFailure, Builder]:
             return 'mariadb'
         return 'default'
 

diff --git a/cross-reference/crossreference/cr/templates/cr/index.html b/cross-reference/crossreference/cr/templates/cr/index.html
@@ -91,7 +91,7 @@ <h2 style="color: #4e629a;">Results</h2>
     	</thead>
         {% for test in test_runs %}
             <tr class="expandable">
-                <td> [<a href="https://buildbot.mariadb.org/#/builders/{{ test.test_run_id.revision }}/builds/{{ test.test_run_id.bbnum }}" > Build details </a>] </td>
+                <td> [<a href="https://buildbot.mariadb.org/#/builders/{{ test.test_run_id.platform_name }}/builds/{{ test.test_run_id.bbnum }}" > Build details </a>] </td>
                 <td>{{test.test_run_id.branch}}</td>
                 <td>{{test.test_run_id.revision| slice:9}}</td>
                 <td>{{test.platform}}</td>