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

Reactive Health Service #1

Open
ojcarfr opened this issue Nov 5, 2018 · 0 comments
Open

Reactive Health Service #1

ojcarfr opened this issue Nov 5, 2018 · 0 comments
Assignees
Labels
core Affects the core framework good first issue Good for newcomers help wanted Extra attention is needed

Comments

@ojcarfr
Copy link
Contributor

ojcarfr commented Nov 5, 2018

A health endpoint can contain a lot of health check policies, so we need a service in order to run all of such checks, reacting to every health check status change. This strategy will allow avoid denial of service throttling among check operations.

Every health check can be declared either to be executed on its own polling process (buffering the result) or to be executed on demand by the runner. Therefore, seems like the runner should be declared in the same way, buffering all the results whether is declared to poll checks.

The expected health check result is defined as:

  • Health check name.
  • A boolean value indicating health success.
  • Health check status (from a defined status set).
  • Result message.
  • Execution cost (in timestamp).
  • Tags? (could be useful on monitoring pulling).
  • Additional info?

The runner result will group every health check result, defined as:

  • Is health (boolean).
  • Service status (useful on warm up).
  • Execution cost (in timestamp).
  • Check result collection.
@ojcarfr ojcarfr added enhancement New feature or request help wanted Extra attention is needed labels Nov 6, 2018
@ojcarfr ojcarfr added good first issue Good for newcomers core Affects the core framework and removed enhancement New feature or request labels Nov 6, 2018
@ojcarfr ojcarfr self-assigned this Nov 6, 2018
@ojcarfr ojcarfr mentioned this issue Nov 22, 2018
@ojcarfr ojcarfr mentioned this issue Jun 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Affects the core framework good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant