aligning cf push health-check default value


Koper, Dies <diesk@...>
 

Hi CLI users,

With apps deployed to DEAs, a health check is performed at application start-up targeting the app's port, unless you specified `--no-route`, in which case the process is monitored.
With Diego, the health check is performed continuously and the type of check was exposed through an option to the `cf push` command.
This option defaults to `port`, which isn't always appropriate for apps pushed without a route, such as worker apps.

We propose fixing the `--health-check-type` option's default value to align with the behaviour seen for DEAs, i.e. to use "none" if option `--no-route` is used:

--health-check-type, -u Application health check type (Default: 'none' if '--no-route' is set, otherwise 'port')`
Would anyone object to such change?

Cheers,
Dies Koper
Cloud Foundry CLI PM

Join cf-dev@lists.cloudfoundry.org to automatically receive all group messages.