Skip to content

Commit 448d323

Browse files
authored
Revert JRuby manifest timeout to 180s (CF platform maximum) (#1142)
PR #1140 changed the timeout from 180s to 300s, but CloudFoundry has a platform-wide maximum of 180s (cc.maximum_health_check_timeout: 180). Deployment fails with: 'health_check_timeout Maximum exceeded: max 180s' The test timeout increase to 5 minutes (from PR #1140) remains in place and is working correctly. This change only reverts the manifest timeout to comply with the CF platform constraint. With timeout: 180s and test polling: 5min, tests should pass in most cases. Some flakiness may remain on extremely slow CI workers, which is an acceptable tradeoff given the platform constraint. Fixes deployment failures introduced by PR #1140.
1 parent c857aeb commit 448d323

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

fixtures/default/sinatra_jruby/manifest.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22
applications:
33
- name: sinatra_jruby_web_app
44
health-check-type: process
5-
timeout: 300
5+
timeout: 180

0 commit comments

Comments
 (0)