add foreign key to AnsibleRepository

fixes: pulp#1506

CHANGES/1506.misc

@@ -0,0 +1 @@
+Switched to foreign key for ``last_sync_task`` in AnsibleRepository model due to performance issues.

pulp_ansible/app/migrations/0056_alter_ansiblerepository_last_sync_task.py

@@ -0,0 +1,21 @@
+# Generated by Django 4.2.3 on 2023-07-07 11:57
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("core", "0107_distribution_hidden"),
+        ("ansible", "0055_ansiblerepository_last_sync_task"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="ansiblerepository",
+            name="last_sync_task",
+            field=models.ForeignKey(
+                null=True, on_delete=django.db.models.deletion.CASCADE, to="core.task"
+            ),
+        ),
+    ]

pulp_ansible/app/models.py

@@ -527,10 +527,11 @@ class AnsibleRepository(Repository, AutoAddObjPermsMixin):
     last_synced_metadata_time = models.DateTimeField(null=True)
     gpgkey = models.TextField(null=True)
     private = models.BooleanField(default=False)
-
-    @property
-    def last_sync_task(self):
-        return _get_last_sync_task(self.pk)
+    last_sync_task = models.ForeignKey(
+        Task,
+        on_delete=models.CASCADE,
+        null=True,
+    )
 
     class Meta:
         default_related_name = "%(app_label)s_%(model_name)s"

pulp_ansible/app/tasks/collections.py

@@ -168,6 +168,9 @@ def sync(remote_pk, repository_pk, mirror, optimize):
     remote = CollectionRemote.objects.get(pk=remote_pk)
     repository = AnsibleRepository.objects.get(pk=repository_pk)
 
+    repository.last_sync_task = Task.current()
+    repository.save(update_fields=["last_sync_task"])
+
     is_repo_remote = False
     if repository.remote:
         is_repo_remote = remote.pk == repository.remote.pk