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

Add healthcheck endpoint for web #11770

Merged
merged 1 commit into from Sep 7, 2019

Conversation

@ykzts
Copy link
Collaborator

commented Sep 6, 2019

Add a health check to check DB communication. The health check is connected with an internal request, so it does not force connection with HTTPS.

This gem also uses GitLab.

@ykzts ykzts force-pushed the health-check branch from aab6029 to dd3a695 Sep 6, 2019

@nightpool

This comment has been minimized.

Copy link
Collaborator

commented Sep 6, 2019

there's some kind of logical conflict between this PR and the one you opened right before it: https://github.com/tootsuite/mastodon/pull/11769/files

@ykzts

This comment has been minimized.

Copy link
Collaborator Author

commented Sep 6, 2019

Yes, I will resolve the conflict based on what was merged earlier.

@ykzts ykzts force-pushed the health-check branch from dd3a695 to 9dc37d3 Sep 6, 2019

@Gargron
Gargron approved these changes Sep 7, 2019

@Gargron Gargron merged commit d7268be into master Sep 7, 2019

2 checks passed

build-and-test Workflow: build-and-test
Details
codeclimate All good!
Details

@ykzts ykzts deleted the health-check branch Sep 7, 2019

@Shleeble

This comment has been minimized.

Copy link
Contributor

commented Sep 7, 2019

Excellent. I'll use that for my pingdom style external monitoring :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.