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

Better healthcheck geoserver #88

Merged
merged 1 commit into from
Jun 6, 2024
Merged

Better healthcheck geoserver #88

merged 1 commit into from
Jun 6, 2024

Conversation

edevosc2c
Copy link
Member

@edevosc2c edevosc2c commented Jun 5, 2024

This change the healthcheck of geoserver because:

  • The healthcheck fail when some external datastores doesn't work.
  • This is compute expensive to load this request on large datastores.

Half downside:

  • layer geor:public_layer needs to exist

@edevosc2c edevosc2c added the enhancement New feature or request label Jun 5, 2024
Copy link
Member

@jeanpommier jeanpommier left a comment

Choose a reason for hiding this comment

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

Thanks Emilien. I believe it was the healthcheck that was ised, historically, although I can't find it back.

It should be more efficient indeed, thanks for the change

@edevosc2c edevosc2c merged commit 1a98999 into main Jun 6, 2024
1 check passed
@edevosc2c edevosc2c deleted the healthcheck-geoserver branch June 6, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants