Skip to content

Introduce `ci_needs_size_limit` to fine control DAG needs

Kamil Trzciński requested to merge control-dag-needs-limit-with-plan into master

What does this MR do?

Related to: #197886 (closed)

I think we are ready to raise DAG limit. We need to test it better and have much better control than we have today. This improves that.

This does:

  • Remove old ci_dag_limit_needs which had a weird state (enabled by default), instead of disabled by default
  • Add plan_limits.ci_plan_needs_size_limit defaulting 50 a presumably a value that we would like to bump limits too
  • Indicate that 50 once enabled by default is ~performance validated option
  • Allow systems administrator to overwrite the needs limit using plan_limits

Does this MR meet the acceptance criteria?

Conformity

Edited by Bob Van Landuyt

Merge request reports