Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Create job after the upload is done #3608

Merged
merged 3 commits into from
Oct 31, 2023
Merged

Create job after the upload is done #3608

merged 3 commits into from
Oct 31, 2023

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Oct 30, 2023

  • Decoupled the job creation from the JobHelper initialization
  • Move the job creation after the all the files have been uploaded

closes #3563

- Decoupled the job creation from the JobHelper initialization
- Move the job creation after the all the files have been uploaded
@codecov-commenter
Copy link

codecov-commenter commented Oct 30, 2023

Codecov Report

Merging #3608 (1fcd31c) into main (d50fd48) will decrease coverage by 0.43%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3608      +/-   ##
==========================================
- Coverage   39.95%   39.52%   -0.43%     
==========================================
  Files         303      275      -28     
  Lines       37724    32708    -5016     
  Branches     1738     1738              
==========================================
- Hits        15073    12929    -2144     
+ Misses      22330    19458    -2872     
  Partials      321      321              

see 59 files with indirect coverage changes

@chkeita chkeita marked this pull request as ready for review October 31, 2023 02:20
@chkeita chkeita merged commit 902557f into microsoft:main Oct 31, 2023
24 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed job template submission should not result in "zombie" jobs
3 participants