Branch: refs/heads/master
Home: https://github.com/jenkinsci/sauce-ondemand-plugin
Commit: 175dade48bdf0d77b8e085892eec1638bce02c23
https://github.com/jenkinsci/sauce-ondemand-plugin/commit/175dade48bdf0d77b8e085892eec1638bce02c23
Author: Diego Molina <[email protected]>
Date: 2016-06-07 (Tue, 07 Jun 2016)
Changed paths:
M src/main/java/hudson/plugins/sauce_ondemand/PluginImpl.java
M src/main/java/hudson/plugins/sauce_ondemand/SauceOnDemandBuildWrapper.java
M src/main/resources/hudson/plugins/sauce_ondemand/PluginImpl/config.jelly
A
src/main/resources/hudson/plugins/sauce_ondemand/PluginImpl/help-sauceConnectMaxRetries.html
A
src/main/resources/hudson/plugins/sauce_ondemand/PluginImpl/help-sauceConnectRetryWaitTime.html
M
src/test/java/hudson/plugins/sauce_ondemand/ParameterizedSauceBuildWrapperTest.java
Log Message:
-----------
Configurable retry feature when Sauce Connect fails to start (#18)
* First retry version when SauceConnect fails to start
- Using hard coded values for now
- Using a temp variable to simulate failure and then correct behaviour
* Moving retry variables to class level, still hard coded in the class
constructor.
* Adding fields in the general plugin configuration to:
- set up max retries
- set up waiting time per retry
- retrieving values through the PlugImpl class
* Using configured values for retrying in the actual method.
Also avoiding the retrial code when no retries were configured.
* Moving the wait inside the try/catch above:
- Also throwing the exception instead of logging it
- This was keeping the build running when it was cancelled
* Improved field name to make clear that the value should be in seconds.
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
