This repository has been archived by the owner on Sep 12, 2023. It is now read-only.

introduce timeouts for puppet-done-test [addresses #97] #100

Open

pcfe wants to merge 1 commit into master from Issue97

Collaborator

pcfe commented Nov 2, 2017

should fix Issue puppet done test should stop after some time #97
working at $current_customer
does not make the build fail on exit 1 though, I tried testing with bash -x all the way down, maybe I just missed a spot.

Reviewers

the main aim is to not spend too long waitig on puppet
failing the build when the script does exit 1 may or may not be a user's expectation, unsure.


          introduce timeouts for puppet-done-test

9bcc45f

- should fix Issue #97
- working fine when tested at $current_customer

pcfe requested review from evgeni, opuk, nstrug, chr15p and ture-karlsson

November 2, 2017 15:30

pcfe changed the title ~~introduce timeouts for puppet-done-test~~ introduce timeouts for puppet-done-test [addresses #97]

pcfe assigned ericzolf and nstrug

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

evgeni Awaiting requested review from evgeni

opuk Awaiting requested review from opuk

nstrug Awaiting requested review from nstrug

chr15p Awaiting requested review from chr15p

ture-karlsson Awaiting requested review from ture-karlsson

At least 1 approving review is required to merge this pull request.

Labels

None yet