feat: Convert waiting and starting status to 202

.changeset/lucky-cats-teach.md

@@ -0,0 +1,5 @@
+---
+"@core/sync-service": patch
+---
+
+Return `202` for `waiting` and `starting` health status - accepts requests but will fail to service them.

packages/sync-service/Dockerfile

@@ -60,5 +60,8 @@ ARG RELEASE_NAME=electric
 COPY --from=builder /app/_build/prod/rel/${RELEASE_NAME} ./
 RUN mv /app/bin/${RELEASE_NAME} /app/bin/entrypoint
 
+
+HEALTHCHECK --start-interval=2s --start-period=10s CMD curl --fail http://localhost:${ELECTRIC_PORT-3000}/v1/health || exit 1
+
 ENTRYPOINT ["/app/bin/entrypoint"]
 CMD ["start"]

packages/sync-service/lib/electric/plug/health_check_plug.ex

@@ -16,8 +16,8 @@ defmodule Electric.Plug.HealthCheckPlug do
 
     {status_code, status_text} =
       case get_service_status.() do
-        :waiting -> {503, "waiting"}
-        :starting -> {503, "starting"}
+        :waiting -> {202, "waiting"}
+        :starting -> {202, "starting"}
         :active -> {200, "active"}
         :stopping -> {503, "stopping"}
       end

packages/sync-service/test/electric/plug/health_check_plug_test.exs

@@ -30,22 +30,22 @@ defmodule Electric.Plug.HealthCheckPlugTest do
     end
 
     @tag connection_status: :waiting
-    test "returns 503 when in waiting mode", ctx do
+    test "returns 202 when in waiting mode", ctx do
       conn =
         conn(ctx)
         |> HealthCheckPlug.call([])
 
-      assert conn.status == 503
+      assert conn.status == 202
       assert Jason.decode!(conn.resp_body) == %{"status" => "waiting"}
     end
 
     @tag connection_status: :starting
-    test "returns 503 when in starting mode", ctx do
+    test "returns 202 when in starting mode", ctx do
       conn =
         conn(ctx)
         |> HealthCheckPlug.call([])
 
-      assert conn.status == 503
+      assert conn.status == 202
       assert Jason.decode!(conn.resp_body) == %{"status" => "starting"}
     end