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.

Sign up for GitHub

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

Jump to bottom

feat(api): Add /health endpoint #28

Merged
merged 4 commits into from
Jun 8, 2024
Merged

feat(api): Add /health endpoint #28

merged 4 commits into from
Jun 8, 2024

Conversation

hayk96
Copy link
Owner

@hayk96 hayk96 commented Jun 8, 2024

1. What this PR does / why we need it:

  • [ENHANCEMENT] Added a new endpoint: /health for retrieving system health.
  • [ENHANCEMENT] Added a new function that continuously checks (600 checks at 3-second intervals) for establishing a connection to Prometheus.
  • [BUGFIX] The Prometheus /runtimeinfo API call check has been moved under the validation function.
  • [BUGFIX] Added proper exception handling while checking the status of the reload API of Prometheus at runtime.

2. Make sure that you've checked the boxes below before you submit PR:

  • DCO signed
  • No conflict with the main branch.

@hayk96
fix(validations): move Prometheus /runtimeinfo API call under validat…
fc74735 
…ions

Signed-off-by: hayk96 <hayko5999@gmail.com>
@hayk96
feat(api): Add /health endpoint
ea36d4a 
Signed-off-by: hayk96 <hayko5999@gmail.com>
@hayk96
docs: Update CHANGELOG.md
b4bbfd4 
Signed-off-by: hayk96 <hayko5999@gmail.com>
@hayk96
chore(api): Bump app version #patch
9adcf37 
Signed-off-by: hayk96 <hayko5999@gmail.com>
@hayk96 hayk96 added bug Something isn't working enhancement New feature or request feature request Suggest new feature labels Jun 8, 2024
@hayk96 hayk96 self-assigned this Jun 8, 2024
@hayk96 hayk96 linked an issue Jun 8, 2024 that may be closed by this pull request
Add health endpoint #27
Closed
@hayk96 hayk96 merged commit 4374aa2 into main Jun 8, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hayk96 hayk96

Labels
bug Something isn't working enhancement New feature or request feature request Suggest new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add health endpoint
1 participant
@hayk96