cloudfoundry/go-cfclient

Make health-check-http-endpoint optional

Gourab1998 opened this issue · 5 comments

Why can health-check-http-endpoint not be optiional?
https://github.com/cloudfoundry-community/go-cfclient/blob/main/operation/manifest.go#L50

image

This is a problem for non "web" type processes since they should not have health-check-http-endpoint.
Please let me know if I am missing something

@Gourab1998 I believe this won't pose any problems for other types. The invocation of configuration by CF will depend on the type.

@ckganesan When I try to push a manifest of health check type "process" this is what cf controller returns
image

Also, ideally speaking we should not keep optional field as mandatory 😅

The example provided for push app here fails due to same error.

#383 : Made the HTTP endpoint optional.

Fixed by #383