Add admin failures panel

Author: vtm9

apps/codebattle/lib/codebattle/code_check/checker.ex

@@ -8,6 +8,8 @@ defmodule Codebattle.CodeCheck.Checker do
 
   require Logger
 
+  @failure_results ~w(error service_failure service_timeout timeout)
+
   @spec call(Codebattle.Task.t(), String.t(), String.t(), CodeCheck.check_meta()) :: CodeCheck.check_result()
   def call(task, solution_text, lang_slug, meta \\ %{}) do
     lang_meta = Languages.meta(lang_slug)
@@ -53,7 +55,8 @@ defmodule Codebattle.CodeCheck.Checker do
       lang: token.lang_meta.slug,
       started_at: started_at,
       duration_ms: token.execution_time_msec,
-      result: token.result.status
+      result: token.result.status,
+      error_description: run_error_description(token.result)
     })
   end
 
@@ -71,6 +74,38 @@ defmodule Codebattle.CodeCheck.Checker do
 
   defp maybe_emit_telemetry(_token, _meta), do: :ok
 
+  defp run_error_description(%{status: status} = result) when status in @failure_results do
+    description =
+      cond do
+        is_binary(result[:output_error]) and String.trim(result[:output_error]) != "" ->
+          result[:output_error]
+
+        is_binary(result[:output]) and String.trim(result[:output]) != "" ->
+          result[:output]
+
+        status in ["service_timeout", "timeout"] ->
+          "Code check execution timed out"
+
+        true ->
+          nil
+      end
+
+    normalize_error_description(description)
+  end
+
+  defp run_error_description(_result), do: nil
+
+  defp normalize_error_description(nil), do: nil
+
+  defp normalize_error_description(description) when is_binary(description) do
+    description
+    |> String.trim()
+    |> case do
+      "" -> nil
+      trimmed -> String.slice(trimmed, 0, 4_000)
+    end
+  end
+
   defp get_executor do
     if FunWithFlags.enabled?(:use_remote_zig_executor) do
       Codebattle.CodeCheck.Executor.RemoteZig

apps/codebattle/lib/codebattle/code_check/run.ex

@@ -17,11 +17,21 @@ defmodule Codebattle.CodeCheck.Run do
     field(:started_at, :utc_datetime_usec)
     field(:duration_ms, :integer)
     field(:result, :string)
+    field(:error_description, :string)
   end
 
   def changeset(%Run{} = run, attrs) do
     run
-    |> cast(attrs, [:user_id, :game_id, :tournament_id, :lang, :started_at, :duration_ms, :result])
+    |> cast(attrs, [
+      :user_id,
+      :game_id,
+      :tournament_id,
+      :lang,
+      :started_at,
+      :duration_ms,
+      :result,
+      :error_description
+    ])
     |> validate_required([:game_id, :lang, :started_at, :duration_ms, :result])
     |> validate_inclusion(:result, @results)
     |> validate_number(:duration_ms, greater_than_or_equal_to: 0)