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

Updater healthcheck #237

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Updater healthcheck #237

wants to merge 8 commits into from

Conversation

cherman23
Copy link

Purpose

Currently there is no way to know if updater runs properly. With healthchecks a ping will be sent to the searchneu-support channel.

Tickets

https://github.com/orgs/sandboxnu/projects/20?pane=issue&itemId=82334730

Contributors

@cherman23

Feature List

  • The updater is currently set to run every 5 minutes, a ping is sent out to healthchecks.io when the the updater runs properly. If the updater doesn't run the service will wait 10 minutes prior to sending a message to the searchneu-support channel in the sandbox slack.
  • The service sends out a get request to link provided in the healthchecks.io account.
  • Any modifications made to how we get a notification if updater doesn't process must be done through healthchecks.io portal.

Notes

I @cherman23 have the login and am the owner for the healthchecks.io account and @ananyaspatil is a manager of the account. Contact either for passing off login information and/or changing ownership.

Reviewers:

Primary reviewer:

@ananyaspatil

Secondary reviewers:

@mehallhm
@nickpfeiffer05
@olivial215
@serena-boyu

@coveralls
Copy link
Collaborator

coveralls commented Oct 31, 2024

Coverage Status

coverage: 80.374% (-0.2%) from 80.533%
when pulling 9e844a8 on updater-healthcheck
into 6cd914e on master.

services/healthchecks.ts Outdated Show resolved Hide resolved
services/updater.ts Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants