Restrict pipeline analytics status filter to terminal statuses

Daniele Rossettirequested to merge
pipeline-analytics-status-enum into main

What does this MR do?

Restricts the pipeline analytics status filter to the only statuses that are meaningful for the finishedPipelines Analytics Engine. Since that AE is backed by the ClickHouse ci_finished_pipelines table, it only ever contains finished pipelines, so the valid statuses are:

  • canceled
  • failed
  • skipped
  • success

Previously the analytics status filter reused the shared CI status enum (13 values incl. running, pending, created, …). Those would silently return zero results rather than erroring, which is misleading.

Changes

  • analytics.rs: define an analytics-specific status enum instead of reusing ci_status_field_type() (still used by Jobs and standard pipelines).
  • Update the invalid-value error message test to reflect the four statuses.

Notes

This is the interim fix discussed in ai-assist!5668. Broader coverage (all pipeline states, more filters) is tracked as a follow-up in #148 (moved) — migrating to a new pipelines AE backed by siphon_p_ci_pipelines.

🤖 Generated with Claude Code

Merge request reports