Make benchmark-results-collector trigger manual

Dan Notesteinrequested to merge
fix/disable-auto-benchmark-collector into develop

Summary

  • Set the Trigger benchmark-results-collector job to manual instead of automatic on develop/default branch
  • The api-benchmark-script is currently commented out in e2e_benchmark_on_postgrest, so request_process_times.log is not generated
  • This causes the downstream benchmark-results-collector job to fail with FileNotFoundError

Root Cause

The benchmark collector expects artifacts/request_process_times.log but this file is only generated when benchmarks run. Since line 663 has # - *api-benchmark-script commented out, the file is never created.

Fix

Make the trigger job manual until benchmarks are re-enabled. Added a comment explaining why.

Test plan

  • Verify the pipeline passes on this branch (the trigger job should show as manual, not automatic)
  • Merge to develop and verify develop pipeline no longer auto-fails on the collector stage

🤖 Generated with Claude Code

Edited by Dan Notestein

Merge request reports