Merge pull request #214 from DalgoT4D/bug/schedule-active-inactive

pause or resume the deployment while activating or deactivating the schedules

Author: Ishankoradia

proxy/service.py

@@ -1164,7 +1164,11 @@ def set_deployment_schedule(deployment_id: str, status: str) -> None:
         del schedule["updated"]
         del schedule["deployment_id"]
         schedule["active"] = status == "active"
-        prefect_patch(f"deployments/{deployment_id}", {"schedules": [schedule]})
+
+        prefect_patch(
+            f"deployments/{deployment_id}",
+            {"schedules": [schedule], "paused": status == "inactive"},
+        )
 
     return None
 

tests/test_service.py

@@ -1518,7 +1518,7 @@ def test_set_deployment_schedule_result_1():
         set_deployment_schedule(deployment_id, "active")
         mock_patch.assert_called_once_with(
             f"deployments/{deployment_id}",
-            {"schedules": [{"schedule": {"cron": "0 0 * * *"}, "active": True}]},
+            {"schedules": [{"schedule": {"cron": "0 0 * * *"}, "active": True}], "paused": False},
         )
 
 
@@ -1542,7 +1542,7 @@ def test_set_deployment_schedule_result_2():
         set_deployment_schedule(deployment_id, "inactive")
         mock_patch.assert_called_once_with(
             f"deployments/{deployment_id}",
-            {"schedules": [{"schedule": {"cron": "0 0 * * *"}, "active": False}]},
+            {"schedules": [{"schedule": {"cron": "0 0 * * *"}, "active": False}], "paused": True},
         )