Stephan Erb commented on AURORA-1599:

For reference: This is now possible since 

> HealthCheckerConfig should validate it has only 1 set field
> -----------------------------------------------------------
>                 Key: AURORA-1599
>                 URL: https://issues.apache.org/jira/browse/AURORA-1599
>             Project: Aurora
>          Issue Type: Story
>          Components: Client
>            Reporter: John Sirois
>            Priority: Minor
> As discussed in https://reviews.apache.org/r/42816/ - the 
> {{HealthCheckerConfig}} struct has 2 fields, but these 2 fields are treated 
> like a union today behind the scenes.  This union-style treatment is silent - 
> we check for a {{shell}} 1st and use it and ignore {{http}} if also 
> populated.  This could lead to user confusion.  It would be better to fail 
> fast if both types of health check are configured.

This message was sent by Atlassian JIRA

Reply via email to