Pool API improve slots validation

[pierrejeambrun]

Add backend validation to pool API on slots attribute. It is not possible anymore to create or update pools slots to have null or negative values.

[jason810496]

It seems -1 is still supported for pools, as it is treated as infinite. So maybe we should allow positive integers and -1 for the slots field.

airflow/airflow-core/src/airflow/models/pool.py

Lines 90 to 91 in bb0ff04

	# -1 for infinite
	slots: Mapped[int] = mapped_column(Integer, default=0)

airflow/airflow-core/src/airflow/models/pool.py

Lines 216 to 222 in bb0ff04

	pool_rows = session.execute(query)
	for pool_name, total_slots_in, include_deferred in pool_rows:
	total_slots = float("inf") if total_slots_in == -1 else total_slots_in
	pools[pool_name] = PoolStats(
	total=total_slots, running=0, queued=0, open=0, deferred=0, scheduled=0
	)
	pool_includes_deferred[pool_name] = include_deferred

[pierrejeambrun]

Arf, yes there is that

[pierrejeambrun]

Ok, after more thoughts, lets merge this for now, which is better than no validation at all.

Currently -1 is not possible to use via the API, because it will be returned as infinity to the server which will 500. (and the front does not currently handle infinity as available pool slots).

I'll create a separate issue for that.

[github-actions]

Backport successfully created: v3-1-test

Status	Branch	Result
✅	v3-1-test

[pierrejeambrun]

Issue here #61115