Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing background info and k8s configs for examples #54

Closed
peterkuiper opened this issue Jun 8, 2018 · 7 comments
Closed

Missing background info and k8s configs for examples #54

peterkuiper opened this issue Jun 8, 2018 · 7 comments
Assignees

Comments

@peterkuiper
Copy link
Contributor

peterkuiper commented Jun 8, 2018

I think a bit of background info about what actually happens on SIGTERM and maybe some example k8s yaml files to get people going would be great (readinessProbe, livenessProbe and terminationGracePeriodSeconds). Your blog post explains things a bit more in depth: things get confusing :)

Maybe even an example of it working in action with minikube would be great. How can you confirm that everything is working correctly?

Either way, great module. I replaced our custom code (in the end did the same thing but with more clutter). We added a const serverListenAsync = promisify(server.listen.bind(server)) so server startup steps can be nicely chained (ie, db connection, redis etc).

Does godaddy use terminus in production? Maybe there are some tips and tricks you can share.

@gergelyke gergelyke self-assigned this Jun 8, 2018
@gergelyke
Copy link
Collaborator

Hi Peter,

yeah, that makes sense, I'll work on that!

Yes, we use it in production.

@peterkuiper
Copy link
Contributor Author

If you need any help, please let me know!

@gergelyke
Copy link
Collaborator

Thanks @peterkuiper - once I have a PR ready, I will send it your way!

@gergelyke
Copy link
Collaborator

@peterkuiper managed to put together a first draft: #58

please let me know what you'd change!

@gergelyke
Copy link
Collaborator

closing as #58 is merged

@peterkuiper
Copy link
Contributor Author

@gergelyke Sorry for the late response: LGTM!

@gergelyke
Copy link
Collaborator

cool, thanks @peterkuiper!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants