Skip to content

[dagster-airlift][jobs 6/n] jobs UI #29278

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: dpeng817/dpeng817/monitoring_job_real
Choose a base branch
from

Conversation

dpeng817
Copy link
Contributor

@dpeng817 dpeng817 commented Apr 15, 2025

Summary & Motivation

I bit most of the implementation from #28997. The changes:

  • genericize the gql layer beyond airlift to "externalJobSource", allowing for multiple potential systems.
  • allow the overview timeline to make use of the icon/external stuff.

How I Tested These Changes

Live against kitchen sink.

@dpeng817 dpeng817 requested review from OwenKephart and prha April 15, 2025 01:39
@dpeng817 dpeng817 marked this pull request as ready for review April 15, 2025 01:39
Copy link

github-actions bot commented Apr 15, 2025

Deploy preview for dagit-storybook ready!

✅ Preview
https://dagit-storybook-20weivvqt-elementl.vercel.app
https://airlift-jobs-ui.components-storybook.dagster-docs.io

Built with commit 7c45b11.
This pull request is being automatically deployed with vercel-action

Copy link

github-actions bot commented Apr 15, 2025

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-ac98gau75-elementl.vercel.app
https://airlift-jobs-ui.core-storybook.dagster-docs.io

Built with commit 7c45b11.
This pull request is being automatically deployed with vercel-action

@dpeng817 dpeng817 force-pushed the dpeng817/dpeng817/monitoring_job_real branch from 431a600 to cdf963d Compare April 15, 2025 16:35
@dpeng817 dpeng817 force-pushed the dpeng817/dpeng817/monitoring_job_real branch from cdf963d to 84e01c1 Compare April 15, 2025 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant