Skip to content

Showing approximate time until next dag_run in Airflow - #20273

Merged
bbovenzi merged 6 commits into
apache:mainfrom
pateash:#19098-Adding-Time-remaining-for-next-dag-run-in-Airflow-UI
Dec 16, 2021
Merged

Showing approximate time until next dag_run in Airflow #20273
bbovenzi merged 6 commits into
apache:mainfrom
pateash:#19098-Adding-Time-remaining-for-next-dag-run-in-Airflow-UI

Conversation

@pateash

@pateashpateash commented Dec 13, 2021

Copy link
Copy Markdown
Contributor

closes: #19098

Description

It will be really helpful if we can add a dynamic message/tooltip which shows time remaining for next dagrun in UI.

Use case/motivation

Although we have next_run available in UI, user has to look into schedule and find the timedifference between schedule and current time, It will be really convenient to have that information available on fingertips.

@boring-cyborgboring-cyborgBot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Dec 13, 2021
@pateash

pateash commented Dec 13, 2021

Copy link
Copy Markdown
ContributorAuthor

image

image

image

@pateashpateash changed the title Showing approximate time remaining for next dag_run in Airflow UI Showing approximate time remaining for next dag_run in Airflow Dec 13, 2021
@pateash

pateash commented Dec 13, 2021

Copy link
Copy Markdown
ContributorAuthor

image

image

bbovenzi
bbovenzi requested changes Dec 14, 2021
View reviewed changes

@bbovenzibbovenzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. I just think we can improve the copy. Maybe Next run in? and then it doesn't need to be bold

@pateash

pateash commented Dec 14, 2021•
edited
Loading

Copy link
Copy Markdown
ContributorAuthor

Looks great. I just think we can improve the copy. Maybe Next run in? and then it doesn't need to be bold

@bbovenzi Sure, Is this looks good?
image

@ashbashb changed the title Showing approximate time remaining for next dag_run in Airflow Showing approximate time until for next dag_run in Airflow Dec 14, 2021
bbovenzi
bbovenzi approved these changes Dec 16, 2021
View reviewed changes

@bbovenzibbovenzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work!

@github-actionsgithub-actionsBot added the okay to merge It's ok to merge this PR as it does not require more tests label Dec 16, 2021
@github-actions

github-actionsBot commented Dec 16, 2021

Copy link
Copy Markdown
Contributor

The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

@pateashpateash changed the title Showing approximate time until for next dag_run in Airflow Showing approximate time until next dag_run in Airflow Dec 16, 2021
@bbovenzi
bbovenzi merged commit e148bf6 into apache:mainDec 16, 2021
@pateash
pateash deleted the #19098-Adding-Time-remaining-for-next-dag-run-in-Airflow-UI branch December 16, 2021 17:34
@pateash
pateash restored the #19098-Adding-Time-remaining-for-next-dag-run-in-Airflow-UI branch February 15, 2022 13:46
@pateash
pateash deleted the #19098-Adding-Time-remaining-for-next-dag-run-in-Airflow-UI branch February 15, 2022 13:49
@ephraimbuddyephraimbuddy added the type:new-feature Changelog: New Features label Apr 14, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbovenzibbovenzibbovenzi approved these changes
@ashbashbAwaiting requested review from ashb
@ryanahamiltonryanahamiltonAwaiting requested review from ryanahamilton

Assignees

No one assigned

Labels

area:UIRelated to UI/UX. For Frontend Developers.area:webserverWebserver related Issuesokay to mergeIt's ok to merge this PR as it does not require more teststype:new-featureChangelog: New Features

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Showing approximate Time until next dag run in Airflow UI

3 participants

@pateash@bbovenzi@ephraimbuddy