Skip to content

Commit

Permalink
fix(ci): allowing bors try when submodule PR is not merged yet
Browse files Browse the repository at this point in the history
Signed-off-by: Dmitry Savitskiy <[email protected]>
  • Loading branch information
dsavitskiy committed Jul 20, 2023
1 parent b4c13c9 commit 040a557
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 9 deletions.
1 change: 1 addition & 0 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,7 @@ pipeline {
anyOf {
branch 'master'
branch 'release/*'
branch 'trying'
expression { run_linter == false }
}
}
Expand Down
27 changes: 18 additions & 9 deletions io-engine/src/rebuild/rebuild_task.rs
Original file line number Diff line number Diff line change
Expand Up @@ -220,17 +220,26 @@ impl RebuildTask {
return Ok(());
}

let msg = format!(
"Rebuild job '{src}' -> '{dst}': verification failed",
src = desc.src_uri,
dst = desc.dst_uri
);

match desc.options.verify_mode {
RebuildVerifyMode::None => Ok(()),
RebuildVerifyMode::Fail => Err(RebuildError::VerifyCompareFailed {
bdev: desc.dst_uri.clone(),
}),
RebuildVerifyMode::None => {
error!("{msg}: ignoring");
Ok(())
}
RebuildVerifyMode::Fail => {
error!("{msg}: failing rebuild");
Err(RebuildError::VerifyCompareFailed {
bdev: desc.dst_uri.clone(),
})
}
RebuildVerifyMode::Panic => {
panic!(
"Rebuild job '{src}' -> '{dst}': verification failed",
src = desc.src_uri,
dst = desc.dst_uri
);
error!("{msg}: will panic");
panic!("{}", msg);
}
}
}
Expand Down

0 comments on commit 040a557

Please sign in to comment.