[JENKINS-30564] Failsafe timeout duration for ElasticTimeoutStrategy - Githubissues

jenkinsci / build-timeout-plugin

Jenkins build-timeout plugin

https://plugins.jenkins.io/build-timeout/

31 stars 80 forks source link

[JENKINS-30564] Failsafe timeout duration for ElasticTimeoutStrategy #49

Closed ikedam closed 8 years ago

ikedam commented 8 years ago

An update of #46.

Added some tests I commented in https://github.com/jenkinsci/build-timeout-plugin/pull/46#issuecomment-151439114
Squashed some commits in #46 . I decided to do this as:
- Commits are not bound to the JIRA issue.
- There were commits completely overridden by following commits.
- There were commits with useless comments.

Commits in #46	New commits
ea2bc002e9c1895cd68bbed8128d33ea49f8714d, 1555384f5914359a6352d0aaf8cc5ecbedcc8b1b, 650d8a5782dab979b65fded0650ca8b237645e4d, 2152caeb2aa1604917b9efe39c98ed2e78a98b77, a07b3f27bd36b0319c43b019aa5235e4edb4a3a6, cdcc95cc3b408d917f4edc34e0a9f6a093c90940	697eafc31c63c015fd5720fee762bba35fb47800
b972c308cb3c3170e104c14b2cea1a0d4ad02d82	264a9a73bbf9958d9dab48cae93145185a50ebd0

My additional commits:

2c055aa4b244287f6a80e134654867e70c550f09

fbelzunc commented 8 years ago

It looks good to me.

:+1:

ikedam commented 8 years ago

Thanks for the review.