serokell/deploy-rs
1
1
Fork 0
mirror of https://github.com/serokell/deploy-rs.git synced 2024-09-11 02:35:39 +03:00
Code Issues Projects Releases Wiki Activity

Merge pull request #242 from serokell/phil/fix-rollback-error-message

Browse Source 
[Chore] fix error messages claiming to have rolled back when not actually doing so
This commit is contained in:
Roman Melnikov 2023-11-02 11:21:55 +01:00 committed by GitHub
commit d507370803
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/cli.rs
View File

@ -582,8 +582,8 @@ async fn run_deploy(
if dry_activate {
info!("dry run, not rolling back");
}
info!("Revoking previous deploys");
if rollback_succeeded && cmd_overrides.auto_rollback.unwrap_or(true) {
info!("Revoking previous deploys");
// revoking all previous deploys
// (adheres to profile configuration if not set explicitely by
// the command line)
@ -592,8 +592,9 @@ async fn run_deploy(
deploy::deploy::revoke(*deploy_data, *deploy_defs).await?;
}
}
return Err(RunDeployError::Rollback);
}
return Err(RunDeployError::Rollback);
return Err(RunDeployError::DeployProfile(e))
}
succeeded.push((deploy_data, deploy_defs))
}