Project

General

Profile

Bug #11987

Bash command used for running provisioning script via SSH is masking scripts's exit code

Added by Jiří Machálek almost 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Compute resources
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Exit codes returned from provisioning templates are ignored because of sending results through pipe to command tee.


Related issues

Related to Foreman - Bug #14004: After_commit failure doesn't handle the rollback for the actions performed from the pre_commit phase and active record objectsNew2016-03-02
Related to Foreman - Tracker #14002: Orchestration build around ActiveRecord hooks trackerNew2016-03-02

Associated revisions

Revision c919008a (diff)
Added by Jiri Machalek almost 4 years ago

Fixes #11987 - Check exit code of deployment scripts

This commit fixes checking exit codes returned from provisioning templates, which were ignored
because of sending results through pipe to command tee. Also adds optional host deleting if deployment
fails because of non zero exit code. This can be configured by setting clean_up_failed_deployment
(default true).

History

#1 Updated by Dominic Cleal almost 4 years ago

  • Category set to Compute resources
  • Status changed from New to Ready For Testing
  • Assignee set to Jiří Machálek

#2 Updated by Anonymous almost 4 years ago

  • Status changed from Ready For Testing to Closed

#3 Updated by Dominic Cleal over 3 years ago

  • Legacy Backlogs Release (now unused) set to 104

#4 Updated by Dominic Cleal over 3 years ago

  • Legacy Backlogs Release (now unused) changed from 104 to 71

#5 Updated by Ivan Necas over 3 years ago

  • Related to Bug #14004: After_commit failure doesn't handle the rollback for the actions performed from the pre_commit phase and active record objects added

#6 Updated by Ivan Necas over 3 years ago

  • Related to Tracker #14002: Orchestration build around ActiveRecord hooks tracker added

Also available in: Atom PDF