Use default failureThreshold value#2328
Use default failureThreshold value#2328jarifibrahim wants to merge 3 commits intofabric8-services:masterfrom
Conversation
Ike Plugins (test-keeper)Thank you @jarifibrahim for this contribution! It appears that no tests have been added or updated in this PR. Automated tests give us confidence in shipping reliable software. Please add some as part of this change. If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command For more information please head over to official documentation. You can find there how to configure the plugin. |
Codecov Report
@@ Coverage Diff @@
## master #2328 +/- ##
==========================================
- Coverage 70.17% 70.15% -0.02%
==========================================
Files 171 171
Lines 16625 16625
==========================================
- Hits 11666 11664 -2
- Misses 3829 3830 +1
- Partials 1130 1131 +1
Continue to review full report at Codecov.
|

The fabric8-wit deployment to prod-preview was successful with #2327. We don't need
failureThreshold:120which isfrom https://docs.openshift.com/container-platform/3.5/rest_api/openshift_v1.html#v1-probe
This PR changes the failureThreshold value back to its default value which was changed in #2298 and #2295
Also see #2291