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 RasterSource Healthcheck #5017

Merged
merged 4 commits into from
Jun 6, 2019
Merged

Conversation

notthatbreezy
Copy link

@notthatbreezy notthatbreezy commented Jun 6, 2019

Overview

Adds a healthcheck that loads some metadata about a tiff. If the tiff location has moved, the healthcheck throws a warning, but does not error out (in case the image is accidentally moved).

Checklist

  • Description of PR is in an appropriate section of the changelog and grouped with similar changes if possible

Demo

This screenshot is a demo of what the logs look like when the image has moved:

image

Notes

It is unclear if this is going to work as far as making sure that servers that pass healthchecks but can't serve tiles no longer do that. It is not possible to reproduce this in development so the only way to test I think is to push it out and test.

Testing Instructions

  • Rebuild servers, start server, go to localhost:8080/healthcheck

Closes #4945

.through(mods.recoverToSick)
healthcheck.check flatMap { check =>
if (check.value.reduce.isHealthy) Ok("A-ok")
else {
logger.warn("Healthcheck Failed (postres, memcached, or gdal)")
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added logging so that we could search for this in our logs if we needed to at a later date

@notthatbreezy notthatbreezy force-pushed the feature/cmb/add-healthcheck branch from 792d91d to 86f7608 Compare June 6, 2019 12:11
@notthatbreezy notthatbreezy merged commit d850be1 into develop Jun 6, 2019
@notthatbreezy notthatbreezy deleted the feature/cmb/add-healthcheck branch June 6, 2019 14:11
notthatbreezy pushed a commit that referenced this pull request Jun 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Update Backsplash Healthcheck to Include Reading Tiles
2 participants