chart icon indicating copy to clipboard operation
chart copied to clipboard

make probes on streaming and web deployments user configurable

Open jstewart612 opened this issue 3 years ago • 4 comments

Implements https://github.com/mastodon/chart/issues/9

jstewart612 avatar Dec 07 '22 01:12 jstewart612

@SISheogorath all requested changes made, please verify/review.

jstewart612 avatar Dec 12 '22 19:12 jstewart612

@SISheogorath @renchap rebased, please review

jstewart612 avatar Mar 11 '23 17:03 jstewart612

(Sorry for the huge delay in getting around to this)

Generally I think this is a great idea. The values look good to me, I just want to ping @renchap to take a look as well before I approve.

timetinytim avatar Apr 18 '24 14:04 timetinytim

So we had a discussion internally about this.

Generally it's a good idea to allow the customization of healthcheck endpoints. But there are two concerns with this particular implementation:

  • Increasing the complexity of the values.yaml file
  • Possibility of future breaking changes

To clarify the second point, consider the possibility that we change the healthcheck endpoint in a future version. The onus to update the values file is then on everyone who uses the chart, and will lead to things breaking if people don't know they have to update it.

I would instead suggest that the probes be constructed out of whatever values the user chooses to define, and fills in the rest of the probes' parameters with default values.

Hopefully that makes sense!

timetinytim avatar Apr 23 '24 10:04 timetinytim