mattermost / mattermost-helm

Mattermost Helm charts for Kubernetes
Apache License 2.0
162 stars 145 forks source link

Added startupProbe #407

Closed mustdiechik closed 12 months ago

mustdiechik commented 1 year ago

Added startupProbe

mattermost-build commented 1 year ago

Hello @mustdiechik,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement? Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

mustdiechik commented 1 year ago

/check-cla

stafot commented 1 year ago

@mustdiechik In my view these values (failureThreshold,periodSeconds,initialDelaySeconds,timeoutSeconds) shouldn't be hardcoded in the template. You could follow the approach we have on this PR.

spirosoik commented 1 year ago

/check-cla

spirosoik commented 1 year ago

+1 on what @stafot mentioned

spirosoik commented 1 year ago

@mustdiechik looks good. Did you sign the CLA? Seems that there's still an issue

mustdiechik commented 1 year ago

/check-cla

stafot commented 1 year ago

@mustdiechik CLA looks good to me unsure why it fails. Although you need to resolve conflicts on this PR

mustdiechik commented 1 year ago

@stafot check please

mustdiechik commented 12 months ago

@spirosoik ^^

spirosoik commented 12 months ago

@mustdiechik let's have a similar PR for the enterprise edition too. It will be valuable there too.