8000 Add endpoints for managing sub-DAG runs by yottahmd · Pull Request #915 · dagu-org/dagu · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add endpoints for managing sub-DAG runs #915

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 50 commits into from
May 6, 2025
Merged

Add endpoints for managing sub-DAG runs #915

merged 50 commits into from
May 6, 2025
+5,863 −3,681

Conversation

yottahmd
Copy link
Collaborator

Changes

  • Added endpoint /runs/{dagName}/{requestId}/sub-runs/{subRequestId}.
  • Updated the web UI to add pages to display sub-DAG run details.

yottahmd added 30 commits April 30, 2025 00:54
@yottahmd
Update endpoint path for getDAGStepLog and updateDAGStepStatus
123b5e7
@yottahmd
Migrate .eslintrc.json to eslint.config.mjs
c28c2b9
@yottahmd
editor.insertSpaces: true
7f7a08a
@yottahmd
Fix build issues in frontend
d95fab8
@yottahmd
Fix GetLatestStatus not to return 'Not started' wrongly
ed615a9
@yottahmd
Fix error level of precondition unmatch
75526ab
@yottahmd
Make subRns to be array
620749f
@yottahmd
Add version display on the sidebar
52d439a
@yottahmd
Add favicon
f945096
@yottahmd
Rename fileId to fileName in API schema
b336860
@yottahmd
Add client/DAGClient interface
2415ec6
@yottahmd
Rename sub-runs directory in persistence
3bf5d8f
@yottahmd
Fix wrong variable name in routes
2a2aaf7
@yottahmd
Fix issues flagged by linter
7900afa
@yottahmd
Rename subruns directory names
d57e6dd
@yottahmd
Merge branch 'next' into api-subdag-runs
da2610c
@yottahmd
Comment
f829dc9
@yottahmd
Define constant for dag.json
74bf934
@yottahmd
Update json field tags to be camelCase
67343c0
@yottahmd
Implement unmarshal/marshal for schedule type
2818201
@yottahmd
Fix webpack config to copy the favicon to the correct location
b0bbe48
@yottahmd
Add favicon link to the base go template
8990d97
@yottahmd
Remove StatusText field from status and node types
91212f9
@yottahmd
Rename statusText to statusLabel in v2 API
c172e8f
@yottahmd
replace logo
96b7c39
@yottahmd
Update primary colors
86d9c05
@yottahmd
minor bug fix: do not close menu when link is clicked
3a5ff76
@yottahmd
Add ReadDAG method to the Record interface
43d46d1
@yottahmd
Support loading json files to load DAG object
0ff1ca8
@yottahmd
Fix retry command to use the DAG file from the run history
871c71e
@yottahmd yottahmd changed the title WIP: Add new endpoint for managing sub-DAG runs Add new endpoint for managing sub-DAG runs May 6, 2025
@yottahmd yottahmd changed the title Add new endpoint for managing sub-DAG runs Add endpoints for managing sub-DAG runs May 6, 2025
@yottahmd yottahmd merged commit 5ec999e into next May 6, 2025
4 checks passed
@yottahmd yottahmd deleted the api-subdag-runs branch May 6, 2025 05:47
@codecov Codecov
Copy link
codecov bot commented May 6, 2025

Codecov Report

Attention: Patch coverage is 58.63636% with 364 lines in your changes missing coverage. Please review.

Please upload report for BASE (next@50eaa37). Learn more about missing BASE report.

Files with missing lines Patch % Lines
internal/runstore/client.go 55.65% 84 Missing and 14 partials ⚠️
internal/dagstore/client.go 59.83% 40 Missing and 9 partials ⚠️
internal/dagstore/filestore/filestore.go 41.66% 47 Missing and 2 partials ⚠️
internal/cmd/restart.go 47.27% 20 Missing and 9 partials ⚠️
internal/runstore/filestore/record.go 51.35% 14 Missing and 4 partials ⚠️
internal/runstore/status.go 56.41% 13 Missing and 4 partials ⚠️
internal/cmd/status.go 44.82% 13 Missing and 3 partials ⚠️
internal/cmd/stop.go 54.54% 11 Missing and 4 partials ⚠️
internal/runstore/node.go 25.00% 10 Missing and 2 partials ⚠️
internal/digraph/scheduler/data.go 23.07% 10 Missing ⚠️
... and 16 more
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             next     #915   +/-   ##
=======================================
  Coverage        ?   58.63%           
=======================================
  Files           ?       71           
  Lines           ?     8587           
  Branches        ?        0           
=======================================
  Hits            ?     5035           
  Misses          ?     3064           
  Partials        ?      488
Files with missing lines Coverage Δ
internal/cmd/dry.go 73.75% <100.00%> (ø)
internal/cmd/flags.go 100.00% <ø> (ø)
internal/dagstore/grep/grep.go 95.45% <100.00%> (ø)
internal/dagstore/paginator.go 83.75% <ø> (ø)
internal/digraph/condition.go 59.42% <ø> (ø)
internal/digraph/errors.go 48.57% <ø> (ø)
internal/digraph/step.go 91.66% <ø> (ø)
internal/fileutil/fileutil.go 23.52% <ø> (ø)
internal/runstore/filestore/time.go 100.00% <ø> (ø)
internal/runstore/filestore/writer.go 53.77% <100.00%> (ø)
... and 27 more

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 50eaa37...03a0a0d. Read the comment docs 6C58 .

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yottahmd
0