honeycombio / helm-charts

Helm repository and charts for Honeycomb
Apache License 2.0
30 stars 39 forks source link

feat(refinery): Set ShutdownDelay based on terminationGracePeriodSeconds #382

Closed VinozzZ closed 2 months ago

VinozzZ commented 2 months ago

Which problem is this PR solving?

Refinery 2.8 attempts to drain data to remaining peers on shutdown, and has a configurable ShutdownDelay field where it tries to process remaining work while shutting down. In k8s we need this config option to be inline with terminationGracePeriodSeconds to ensure k8s doesn't kill pods early.

it's the same change as #360

Short description of the changes

How to verify that this has the expected result

Test localling using kind