search

stardog-union / helm-charts

Stardog Helm Charts
Apache License 2.0
9 stars 12 forks source link

Add startupProbe #97

Closed ck-netlution closed 10 months ago

ck-netlution commented 11 months ago

It is currently necessary to change the livenessProbe.initialDelaySeconds if stardog does not start fast enough. This can be due to slow cloud storage or other reasons like a index rebuild that takes longer than a normal startup. This means that the database cannot start fast, if there

This adds startupProbe. The startupProbe is like the livenessProbe but only until it succeeds once. Then it gets replaced by the livenessProbe. This makes it possible to have no initialDelaySeconds and low periodSeconds for the livenessProbe to have fast failure detection while still allowing for long starup periods with the startupProbe having a high failureThreshold.

ck-netlution commented 11 months ago

Hey @ctffarley, I was wondering if you could take a look at this PR and give me feedback on how we can get this merged.

ck-netlution commented 10 months ago

Hey @ctffarley, It has been a while since I opened this PR. Could you take a look and give me some feedback on how we can get this merged?

scardena commented 10 months ago

@ck-netlution FYI, this feature has been added and released today: https://artifacthub.io/packages/helm/stardog/stardog