Skip to content

Commit

Permalink
Do general setup before reporting any results
Browse files Browse the repository at this point in the history
We cannot report results before the general setup is completed because
the necessary folders might be missing.
  • Loading branch information
jherbel committed Nov 14, 2023
1 parent 446f6f4 commit 3f92019
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion v2/robotmk/src/bin/scheduler/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,9 @@ fn run() -> Result<()> {
bail!("Terminated")
}

write_state(&results::SchedulerState::Setup, &global_config)?;
setup::general::setup(&global_config, &suites).context("General setup failed")?;
debug!("General setup completed");
write_state(&results::SchedulerState::RCCSetup, &global_config)?;
let suites = setup::rcc::setup(&global_config, suites).context("RCC-specific setup failed")?;
debug!("RCC-specific setup completed");

Expand Down
2 changes: 1 addition & 1 deletion v2/robotmk/src/bin/scheduler/results.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ pub fn suite_results_directory(results_directory: &Utf8Path) -> Utf8PathBuf {

#[derive(Serialize)]
pub enum SchedulerState {
Setup,
RCCSetup,
EnvironmentBuilding,
Scheduling,
}
Expand Down

0 comments on commit 3f92019

Please sign in to comment.