Skip to content

Conversation

adk-swisstopo
Copy link
Member

No description provided.

We want to add a readiness probe and update the liveness probe to be more
tolerant of slow responses. In order to verify this works as expected, we need
to be able to inject arbitrary delays in checker responses.

This change adds a `CHECKER_DELAY` setting that allows delaying the `/checker`
endpoint by a commensurate number of seconds. This also adds previously missing
unit test against the checker endpoint.
Github is still able to render the table correctly without the terminating pipe
on each line so it does not really matter. But that's a little nicer still.
The default is 2 and not really suitable when we have a reverses proxy in front
with a higher idle connection timeout.
PB-1943: make keepalive timeout configurable.
@github-actions github-actions bot changed the title #minor New Release v2.2.0 - #minor Sep 18, 2025
Copy link
Contributor

@msom msom left a comment

Choose a reason for hiding this comment

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

Didn't @ltshb mentioned something about filling out the PR description because of changelogs?

@adk-swisstopo
Copy link
Member Author

Didn't @ltshb mentioned something about filling out the PR description because of changelogs?

I don't see anything in https://github.com/geoadmin/doc-guidelines/blob/master/VERSIONING_RELEASE.md#new-semver-release
What kind of description is expected that would not be redundant with the list of commits?

@msom
Copy link
Contributor

msom commented Sep 18, 2025

Didn't @ltshb mentioned something about filling out the PR description because of changelogs?

I don't see anything in https://github.com/geoadmin/doc-guidelines/blob/master/VERSIONING_RELEASE.md#new-semver-release What kind of description is expected that would not be redundant with the list of commits?

Not sure, maybe I confuse it with the PR description of the feature/bug PR's...

@adk-swisstopo adk-swisstopo merged commit acf8f3f into master Sep 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants