fix 415 bg

Author: CamDavidsonPilon

CHANGELOG.md

@@ -7,6 +7,7 @@
 #### Breaking changes
 
   - `pio job-status` has been replaced by `pio jobs status`.
+  - changed `/unit_api/jobs/stop` from using query params to json body.
 
 #### Bug fixes
 
@@ -15,7 +16,7 @@
  - Images ship with DAC43608 library (missed the previous OS jump!)
  - Improvements to "Updating your model" on first boot.
  - Fix /pioreactor page crashing if a worker is inactive in the experiment.
-
+ - Fixed clock syncing on the cluster
 
 ### 25.11.19
 

core/pioreactor/web/api.py

@@ -402,7 +402,9 @@ def get_clocktime(pioreactor_unit: str) -> DelayedResponseReturnValue:
 @api_bp.route("/system/utc_clock", methods=["POST"])
 def set_clocktime() -> DelayedResponseReturnValue:
     # first update the leader:
-    task1 = tasks.multicast_post("/unit_api/system/utc_clock", [get_leader_hostname()], request.get_json())
+    task1 = tasks.multicast_post(
+        "/unit_api/system/utc_clock", [get_leader_hostname()], json=request.get_json()
+    )
     task1.get(blocking=True, timeout=20)
 
     # then tell the workers to update to leader's value (via chrony)

core/pioreactor/web/unit_api.py

@@ -177,7 +177,7 @@ def get_clock_time():
 @unit_api_bp.route("/system/utc_clock", methods=["PATCH", "POST"])
 def set_clock_time() -> DelayedResponseReturnValue:  # type: ignore[return]
     if HOSTNAME == get_leader_hostname():
-        if request.json:
+        if request.get_json(silent=True):  # don't throw 415
             data = request.json
             new_time = data.get("utc_clock_time")
             if not new_time: