Skip to content
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

[DOP-23921] Improve Job, Run & Operation lineage UX #25

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

dolfinus
Copy link
Member

@dolfinus dolfinus commented Feb 25, 2025

Job with runs.

Depends on MobileTeleSystems/data-rentgen#171.

Before:
Снимок экрана_20250225_175923-1

After:
Снимок экрана_20250225_180304-1

Runs are grouped within JobNode component (like in #23), sorted by created_at desc, paginated. Added optional filter by external_id.

Runs & jobs have similar edges (based on #24). This allows to collapse list of runs to their parent job node. But job edges are thinner than run edges, to show that this is a derivative.
Снимок экрана_20250226_111400-1

Run with operations

Before:
Снимок экрана_20250225_175959-1

After:
Снимок экрана_20250225_180409-1

Operations are grouped within RunNide component (like in #23), sorted by created_at desc and paginated. Added optional filter by combination of name, group and description.

Operations, runs & jobs have similar edges (based on #24). This allows to collapse list of operations into their parent run node or even job node. Run edges are thinner than operations, and job edges are even more thin, to show that they are derivatives.
Снимок экрана_20250226_111651-1

Job with runs & operations

Before - complete mess:
Снимок экрана_20250225_180103-1

After - still a mess:
Снимок экрана_20250225_180602-1

So I've decided to completely remove OPERATION granularity from Dataset show page.

@dolfinus dolfinus self-assigned this Feb 25, 2025
@dolfinus dolfinus requested a review from Zabilsya February 25, 2025 15:12
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