Page MenuHomeDevCentral

only touch .initialized on success of service restart
Needs RevisionPublic

Authored by xcombelle on Aug 8 2016, 23:32.
Referenced Files
Unknown Object (File)
Sat, Dec 7, 07:51
Unknown Object (File)
Tue, Dec 3, 00:16
Unknown Object (File)
Wed, Nov 27, 17:18
Unknown Object (File)
Tue, Nov 26, 11:35
Unknown Object (File)
Fri, Nov 22, 14:09
Unknown Object (File)
Fri, Nov 22, 08:39
Unknown Object (File)
Tue, Nov 19, 07:44
Unknown Object (File)
Fri, Nov 15, 19:39

Details

Reviewers
dereckson
Test Plan

make the service fail somehow.
On original the .initialized file should be created anyway
On new version there is no .initialized file created

Diff Detail

Repository
rDPHAB Docker image for Phabricator
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
good-restart
Build Status
Buildable 892
Build 1033: arc lint + arc unit

Event Timeline

xcombelle retitled this revision from to only touch .initialized on succes of service restart.
xcombelle updated this object.
xcombelle edited the test plan for this revision. (Show Details)
xcombelle added a reviewer: dereckson.

To fail on error is probably a good idea to avoid to say .initialized when an error occurs

So a good approach could be http://www.gnu.org/software/bash/manual/bashref.html#The-Set-Builtin

dereckson retitled this revision from only touch .initialized on succes of service restart to only touch .initialized on success of service restart.Aug 16 2016, 19:20
dereckson edited edge metadata.

That would be set -e

dereckson requested changes to this revision.Aug 18 2016, 22:56
dereckson edited edge metadata.
This revision now requires changes to proceed.Aug 18 2016, 22:56