Skip to content

Rework runner job submission flow#3743

Open
un-def wants to merge 1 commit intomasterfrom
issue_3740_make_runner_submission_process_more_lenient
Open

Rework runner job submission flow#3743
un-def wants to merge 1 commit intomasterfrom
issue_3740_make_runner_submission_process_more_lenient

Conversation

@un-def
Copy link
Copy Markdown
Collaborator

@un-def un-def commented Apr 3, 2026

The overall process is more lenient now:

  • It's safe to call /api/submit more than once -- even after /api/upload_code
  • It's safe to call /api/upload_code more than once or don't call it at all -- code upload is now optional -- if we have nothing to upload, we can skip this step

Since /api/upload_code is now optional, the server no longer calls this method if there is no code (RemoteRepo diff or VirtualRepo file archive) to upload. In addition, Python API (used by CLI internally) has been optimized -- it does not upload a code blob in there is no diff (RemoteRepo) or no files (empty VirtualRepo, incl. default virtual repo used when no repo specified in the run configuration).

Fixes: #3740

The overall process is more lenient now:

* It's safe to call `/api/submit` more than once -- even after
  `/api/upload_code`
* It's safe to call `/api/upload_code` more than once or don't call it
  at all -- code upload is now optional -- if we have nothing to upload,
  we can skip this step

Since `/api/upload_code` is now optional, the server no longer calls
this method if there is no code (`RemoteRepo` diff or `VirtualRepo` file
archive) to upload. In addition, Python API (used by CLI internally) has
been optimized -- it does not upload a code blob in there is no diff
(`RemoteRepo`) or no files (empty `VirtualRepo`, incl. default virtual
repo used when no repo specified in the run configuration).

Fixes: #3740
@un-def un-def requested a review from r4victor April 3, 2026 15:55
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.

[Bug]: _submit_job_to_runner is unrecoverably broken once any runner API call after /api/submit failed

1 participant