fixup! feat: Change queries to use lab column information

Rename Builds/Tests FK field from lab to lab_id so lab is free for
annotated lab names without conflicting with the ORM relation.

Signed-off-by: Alan Peixinho <alan.peixinho@profusion.mobi>

Author: alanpeixinho

backend/kernelCI_app/migrations/0020_rename_builds_tests_lab_to_lab_id.py

@@ -0,0 +1,43 @@
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("kernelCI_app", "0019_labs_builds_lab_tests_lab"),
+    ]
+
+    operations = [
+        migrations.RenameField(
+            model_name="builds",
+            old_name="lab",
+            new_name="lab_id",
+        ),
+        migrations.RenameField(
+            model_name="tests",
+            old_name="lab",
+            new_name="lab_id",
+        ),
+        migrations.AlterField(
+            model_name="builds",
+            name="lab_id",
+            field=models.ForeignKey(
+                db_column="lab_id",
+                db_constraint=False,
+                null=True,
+                on_delete=django.db.models.deletion.DO_NOTHING,
+                to="kernelCI_app.labs",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="tests",
+            name="lab_id",
+            field=models.ForeignKey(
+                db_column="lab_id",
+                db_constraint=False,
+                null=True,
+                on_delete=django.db.models.deletion.DO_NOTHING,
+                to="kernelCI_app.labs",
+            ),
+        ),
+    ]

backend/kernelCI_app/models.py

@@ -132,8 +132,12 @@ class Builds(models.Model):
     log_url = models.TextField(blank=True, null=True)
     log_excerpt = models.CharField(max_length=16384, blank=True, null=True)
     misc = models.JSONField(blank=True, null=True)
-    lab = models.ForeignKey(
-        Labs, db_constraint=False, null=True, on_delete=models.DO_NOTHING
+    lab_id = models.ForeignKey(
+        Labs,
+        db_column="lab_id",
+        db_constraint=False,
+        null=True,
+        on_delete=models.DO_NOTHING,
     )
     status = models.CharField(
         max_length=10, choices=StatusChoices.choices, blank=True, null=True
@@ -193,8 +197,12 @@ class UnitPrefix(models.TextChoices):
     input_files = models.JSONField(blank=True, null=True)
     output_files = models.JSONField(blank=True, null=True)
     misc = models.JSONField(blank=True, null=True)
-    lab = models.ForeignKey(
-        Labs, db_constraint=False, null=True, on_delete=models.DO_NOTHING
+    lab_id = models.ForeignKey(
+        Labs,
+        db_column="lab_id",
+        db_constraint=False,
+        null=True,
+        on_delete=models.DO_NOTHING,
     )
     number_value = models.FloatField(blank=True, null=True)
     environment_compatible = ArrayField(models.TextField(), blank=True, null=True)

backend/kernelCI_app/queries/build.py

@@ -56,7 +56,7 @@ def get_build_tests(build_id: str) -> Optional[list[dict]]:
         # TODO remove misc__runtime fallback after lab backfill
         .annotate(
             lab=Coalesce(
-                F("lab__name"),
+                F("lab_id__name"),
                 Cast(F("misc__runtime"), output_field=TextField()),
             )
         )