Align endpoint status choices with workstation session

Author: koopmant

app/grandchallenge/algorithms/migrations/0099_remove_endpoint_endpoint_status_valid_and_more.py

@@ -1,4 +1,4 @@
-# Generated by Django 5.2.14 on 2026-05-29 11:04
+# Generated by Django 5.2.14 on 2026-05-29 15:23
 
 from django.conf import settings
 from django.db import migrations, models
@@ -26,10 +26,10 @@ class Migration(migrations.Migration):
             field=models.CharField(
                 choices=[
                     ("QUEUED", "Queued"),
-                    ("STARTING", "Starting"),
+                    ("STARTED", "Started"),
                     ("RUNNING", "Running"),
-                    ("STOPPED", "Stopped"),
                     ("FAILED", "Failed"),
+                    ("STOPPED", "Stopped"),
                 ],
                 default="QUEUED",
                 max_length=17,
@@ -41,7 +41,7 @@ class Migration(migrations.Migration):
                 condition=models.Q(
                     (
                         "status__in",
-                        ["QUEUED", "STARTING", "RUNNING", "STOPPED", "FAILED"],
+                        ["QUEUED", "STARTED", "RUNNING", "FAILED", "STOPPED"],
                     )
                 ),
                 name="endpoint_status_valid",

app/grandchallenge/algorithms/models.py

@@ -1544,16 +1544,16 @@ class Meta:
 
 class EndpointStatusChoices(TextChoices):
     QUEUED = "QUEUED", _("Queued")
-    STARTING = "STARTING", _("Starting")
+    STARTED = "STARTED", _("Started")
     RUNNING = "RUNNING", _("Running")
-    STOPPED = "STOPPED", _("Stopped")
     FAILED = "FAILED", _("Failed")
+    STOPPED = "STOPPED", _("Stopped")
 
     @classmethod
     def get_active_choices(cls):
         return {
             cls.QUEUED,
-            cls.STARTING,
+            cls.STARTED,
             cls.RUNNING,
         }
 

app/grandchallenge/components/tasks.py

@@ -1682,7 +1682,7 @@ def start_endpoint(*, pk: uuid.UUID, app_label: str, model_name: str):
         )
 
     else:
-        endpoint.update_status(status=endpoint.StatusChoices.STARTING)
+        endpoint.update_status(status=endpoint.StatusChoices.STARTED)
 
 
 @lambda_task(retry_on=(LockNotAcquiredException,))
@@ -1706,7 +1706,7 @@ def handle_endpoint_status_event(*, event: dict):
             pk=params.pk
         )
 
-    if endpoint.status != endpoint.StatusChoices.STARTING:
+    if endpoint.status != endpoint.StatusChoices.STARTED:
         # Nothing to do
         return
 

app/tests/components_tests/test_tasks.py

@@ -1665,7 +1665,7 @@ def test_stop_expired_endpoints(
 @pytest.mark.django_db
 def test_handle_endpoint_status_in_service_event(settings):
     endpoint = EndpointFactory(
-        status=EndpointStatusChoices.STARTING,
+        status=EndpointStatusChoices.STARTED,
     )
     event = {
         "EndpointName": f"{settings.COMPONENTS_REGISTRY_PREFIX}-AE-{endpoint.pk}",
@@ -1681,7 +1681,7 @@ def test_handle_endpoint_status_in_service_event(settings):
 @pytest.mark.django_db
 def test_handle_endpoint_status_failed_events(settings, mocker):
     endpoint = EndpointFactory(
-        status=EndpointStatusChoices.STARTING,
+        status=EndpointStatusChoices.STARTED,
     )
     event = {
         "EndpointName": f"{settings.COMPONENTS_REGISTRY_PREFIX}-AE-{endpoint.pk}",
@@ -1703,7 +1703,7 @@ def test_handle_endpoint_status_failed_events(settings, mocker):
 @pytest.mark.django_db
 def test_handle_endpoint_status_invalid_events(settings, mocker):
     endpoint = EndpointFactory(
-        status=EndpointStatusChoices.STARTING,
+        status=EndpointStatusChoices.STARTED,
     )
     event = {
         "EndpointName": f"{settings.COMPONENTS_REGISTRY_PREFIX}-AE-{endpoint.pk}",
@@ -1724,7 +1724,7 @@ def test_handle_endpoint_status_invalid_events(settings, mocker):
 
 @pytest.mark.parametrize(
     "status",
-    set(EndpointStatusChoices).difference([EndpointStatusChoices.STARTING]),
+    set(EndpointStatusChoices).difference([EndpointStatusChoices.STARTED]),
 )
 @pytest.mark.django_db
 def test_handle_endpoint_status_wrong_state_ignored(mocker, settings, status):