Skip to content

Fix Dag list run badges and counters not refreshing after completion - #74734

Merged
vatsrahul1001 merged 3 commits into
apache:mainfrom
anzinmhd:fix/74703-dag-run-state-refresh
Oct 18, 2026
Merged

vatsrahul1001 merged 3 commits into
apache:mainfrom
anzinmhd:fix/74703-dag-run-state-refresh

Conversation

@anzinmhd

@anzinmhd anzinmhd commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Dag run badges and counters on the Dag list could remain stale after a run completed, requiring a page refresh.

Remove the separate pending-run polling gate from the Dag-list and run-count queries. Both queries retain their existing fast/slow polling intervals, allowing them to fetch the final run state even when no active runs remain.

Closes #74703

Validation

  • Reproduced on Airflow 3.4.0b1.
  • Added two regression tests covering stale run state and run counts. Both fail with the original code and pass with the fix.
  • Verified manually that the badge and run counters update without refreshing.
  • Formatting and targeted ESLint checks passed.

Tradeoff: The Dag-list and run-count queries continue polling at the slower interval when idle, adding occasional API requests to prevent stale state.


Was generative AI tooling used to co-author this PR?
  • Yes — Codex (GPT-6)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Oct 14, 2026
@bbovenzi bbovenzi added this to the Airflow 3.4.0 milestone Oct 14, 2026
@bbovenzi bbovenzi added the backport-to-v3-4-test Backport to v3-4-test label Oct 14, 2026
@bbovenzi

Copy link
Copy Markdown
Contributor

Some static check failures

@anzinmhd

Copy link
Copy Markdown
Contributor Author

Fixed the TypeScript error by replacing MSW’s unsupported onUnhandledRequest option with onUnhandledFrame. Local TypeScript checking and both regression tests passed. Pushed the correction; awaiting CI results.

@vatsrahul1001
vatsrahul1001 merged commit 912bbe5 into apache:main Oct 18, 2026
91 checks passed
@boring-cyborg

boring-cyborg Bot commented Oct 18, 2026

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-4-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
✅ v3-4-test PR Link

@anzinmhd
anzinmhd deleted the fix/74703-dag-run-state-refresh branch October 18, 2026 09:21
vatsrahul1001 pushed a commit that referenced this pull request Oct 18, 2026
… completion (#74734) (#74954)

* Fix Dag run state refresh

* Fix MSW option in Dag refresh regression tests
(cherry picked from commit 912bbe5)

Co-authored-by: Anzin Mohammed <anzinmhd765@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request Oct 20, 2026
… completion (#74734) (#74954)

* Fix Dag run state refresh

* Fix MSW option in Dag refresh regression tests
(cherry picked from commit 912bbe5)

Co-authored-by: Anzin Mohammed <anzinmhd765@gmail.com>
SameerMesiah97 pushed a commit to SameerMesiah97/airflow-oss-sm97 that referenced this pull request Oct 27, 2026
…pache#74734)

* Fix Dag run state refresh

* Fix MSW option in Dag refresh regression tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-4-test Backport to v3-4-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dagrun state badge needs a page refresh to reflect correct state

3 participants