You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "bobu-putheeckal (via GitHub)" <gi...@apache.org> on 2026/05/24 20:44:24 UTC
[PR] Add first-run asset counter regression coverage [airflow]
bobu-putheeckal opened a new pull request, #67441:
URL: https://github.com/apache/airflow/pull/67441
Related: #64616
## What
- Add API route regression coverage for an asset-scheduled DAG that has never run before.
- Verify queued updates for two of three name-only assets are reported by `/ui/next_run_assets/{dag_id}` before the first consumer DagRun exists.
## Why
The reported counter issue depends on the first-ever consumer DAG run path. Current `main` exposes the queued asset state correctly, and this test locks down that behavior.
## Tests
- `uv run pytest tests/unit/api_fastapi/core_api/routes/ui/test_assets.py::TestNextRunAssets -q`
- `uv run ruff format tests/unit/api_fastapi/core_api/routes/ui/test_assets.py`
- `uv run ruff check tests/unit/api_fastapi/core_api/routes/ui/test_assets.py`
---
##### Was generative AI tooling used to co-author this PR?
- [x] Yes (please specify the tool below)
GPT-5 Codex
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add first-run asset counter regression coverage [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #67441:
URL: https://github.com/apache/airflow/pull/67441#issuecomment-4597715097
This pull request has been automatically marked as stale because the author has not responded to a request for more information. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add first-run asset counter regression coverage [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #67441:
URL: https://github.com/apache/airflow/pull/67441#issuecomment-4665453451
This pull request has been closed because the author has not responded to a request for more information.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add first-run asset counter regression coverage [airflow]
Posted by "bobu-putheeckal (via GitHub)" <gi...@apache.org>.
bobu-putheeckal commented on PR #67441:
URL: https://github.com/apache/airflow/pull/67441#issuecomment-4530757530
I am also moving this to draft while waiting for your preference on whether this coverage-only PR is useful.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add first-run asset counter regression coverage [airflow]
Posted by "bobu-putheeckal (via GitHub)" <gi...@apache.org>.
bobu-putheeckal commented on PR #67441:
URL: https://github.com/apache/airflow/pull/67441#issuecomment-4530754774
Thanks for checking. My read is that current \ already handles the reported first-run asset state correctly, so this PR is regression coverage only, not a runtime fix. If you would prefer to close #64616 as already resolved and skip coverage-only PRs for this case, I am happy to close this PR.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add first-run asset counter regression coverage [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on PR #67441:
URL: https://github.com/apache/airflow/pull/67441#issuecomment-4529949507
Should https://github.com/apache/airflow/issues/64616 be open or is it resolved and are you just adding coverage for it?
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add first-run asset counter regression coverage [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #67441: Add first-run asset counter regression coverage
URL: https://github.com/apache/airflow/pull/67441
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org