deepujain opened a new pull request, #64306:
URL: https://github.com/apache/airflow/pull/64306

   ## Summary
   `airflowctl dagrun list` was using the shared list paginator, which kept 
fetching and appending additional pages until it returned every matching run. 
That made `--limit` behave like a page size instead of a hard cap. This change 
makes DAG run listing return exactly the requested page and preserves the limit 
the user asked for.
   
   ## Changes
   - **`airflow-ctl/src/airflowctl/api/operations.py`** -- changed 
`DagRunOperations.list()` to make a single API request and return the server 
response directly, so `--limit` is honored as the final result cap.
   - **`airflow-ctl/tests/airflow_ctl/api/test_operations.py`** -- added a 
regression test that verifies DAG run listing does not paginate past the first 
page when `limit=1`.
   
   ## Test plan
   - [x] `uv run --project /Users/dejain/nvidia/oss/airflow-64066/airflow-ctl 
ruff check 
/Users/dejain/nvidia/oss/airflow-64066/airflow-ctl/src/airflowctl/api/operations.py
 
/Users/dejain/nvidia/oss/airflow-64066/airflow-ctl/tests/airflow_ctl/api/test_operations.py`
   - [x] `uv run --project /Users/dejain/nvidia/oss/airflow-64066/airflow-ctl 
ruff format --check 
/Users/dejain/nvidia/oss/airflow-64066/airflow-ctl/src/airflowctl/api/operations.py
 
/Users/dejain/nvidia/oss/airflow-64066/airflow-ctl/tests/airflow_ctl/api/test_operations.py`
   - [x] `uv run --project /Users/dejain/nvidia/oss/airflow-64066/airflow-ctl 
pytest 
/Users/dejain/nvidia/oss/airflow-64066/airflow-ctl/tests/airflow_ctl/api/test_operations.py::TestDagRunOperations
 -xvs`
   
   Fixes #64066
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to