Skip to content

[Job Launcher] Update job creation to use manifest JSON#3855

Open
flopez7 wants to merge 10 commits into3806from
flopez/3806-2
Open

[Job Launcher] Update job creation to use manifest JSON#3855
flopez7 wants to merge 10 commits into3806from
flopez/3806-2

Conversation

@flopez7
Copy link
Copy Markdown
Contributor

@flopez7 flopez7 commented Apr 1, 2026

Issue tracking

#3806 #3864

Context behind the change

  • Replaced the fortune job creation endpoint with a generic job creation endpoint that accepts a manifest JSON body.
  • Introduced JobManifestDto to encapsulate job details including manifest.
  • Updated JobService to handle different job types based on the manifest provided.
  • Modified tests to reflect changes in job creation logic and added validation for new manifest types.
  • Enhanced ManifestService to validate and upload manifests for various job types including CVAT and HCaptcha.

How has this been tested?

Ran and created fortune job locally

Release plan

None

Potential risks; What to monitor; Rollback plan

None

@flopez7 flopez7 requested a review from Copilot April 1, 2026 15:05
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

5 Skipped Deployments
Project Deployment Actions Updated (UTC)
faucet-frontend Ignored Ignored Preview Apr 7, 2026 0:19am
faucet-server Ignored Ignored Preview Apr 7, 2026 0:19am
human-app Skipped Skipped Apr 7, 2026 0:19am
human-dashboard-frontend Skipped Skipped Apr 7, 2026 0:19am
staking-dashboard Skipped Skipped Apr 7, 2026 0:19am

Request Review

@flopez7 flopez7 requested review from Dzeranov and portuu3 and removed request for Copilot April 1, 2026 15:05
@flopez7 flopez7 self-assigned this Apr 1, 2026
@vercel vercel bot temporarily deployed to Preview – human-app April 6, 2026 11:44 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard April 6, 2026 11:44 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend April 6, 2026 11:44 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app April 6, 2026 14:42 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend April 6, 2026 14:42 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard April 6, 2026 14:42 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend April 6, 2026 16:35 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard April 6, 2026 16:35 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app April 6, 2026 16:35 Inactive
@flopez7 flopez7 requested a review from portuu3 April 7, 2026 08:34
@flopez7 flopez7 added the do-not-merge PR shouldn't be merged until this label is removed label Apr 7, 2026
Base automatically changed from flopez/3806-1 to 3806 April 7, 2026 09:48
@vercel vercel bot temporarily deployed to Preview – staking-dashboard April 7, 2026 12:19 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend April 7, 2026 12:19 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app April 7, 2026 12:19 Inactive
@flopez7 flopez7 removed do-not-merge PR shouldn't be merged until this label is removed labels Apr 7, 2026
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.

2 participants