Skip to content

Document /health thread dump timeout system property#8537

Merged
kmartens27 merged 4 commits intojenkins-infra:masterfrom
gbhat618:document-healthcheck-thread-dump-timeout-property
Nov 11, 2025
Merged

Document /health thread dump timeout system property#8537
kmartens27 merged 4 commits intojenkins-infra:masterfrom
gbhat618:document-healthcheck-thread-dump-timeout-property

Conversation

@gbhat618
Copy link
Contributor

@gbhat618 gbhat618 commented Nov 11, 2025 •
edited
Loading

documenting the property added in jenkinsci/jenkins#11266

Testing

verified by make run locally,

image

@welcome
Copy link

welcome bot commented Nov 11, 2025

Yay, your first pull request towards jenkins.io was created successfully! Thank you so much!

A contributor will provide feedback soon. Meanwhile, you can join the chats and community forums to connect with other Jenkins users, developers, and maintainers.

@probot-autolabeler probot-autolabeler bot added the documentation Jenkins documentation, including user and developer docs, solution pages, etc. label Nov 11, 2025
gbhat618
gbhat618 commented Nov 11, 2025
View reviewed changes
gbhat618
gbhat618 commented Nov 11, 2025
View reviewed changes
@gbhat618 gbhat618 mentioned this pull request Nov 11, 2025
Merged
14 tasks
@gbhat618 gbhat618 marked this pull request as ready for review November 11, 2025 08:43
@gbhat618 gbhat618 requested a review from a team as a code owner November 11, 2025 08:43
@gbhat618 gbhat618 requested a review from MarkEWaite November 11, 2025 08:43
gbhat618
gbhat618 commented Nov 11, 2025
View reviewed changes
content/doc/book/managing/system-properties.adoc
description: |
Log a thread dump when a `/health` check exceeds this timeout to help diagnose stuck requests.
Useful for debugging Kubernetes liveness probe failures caused by stuck health checks.
The value can be specified in ISO-8601 duration format (e.g., `PT20S`) or simple format with 1-character suffix (e.g., `20s`).
Copy link
Contributor Author

@gbhat618 gbhat618 Nov 11, 2025

Choose a reason for hiding this comment

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

as per https://github.com/jenkinsci/remoting/blob/master/src/main/java/org/jenkinsci/remoting/util/DurationStyle.java

MarkEWaite
MarkEWaite approved these changes Nov 11, 2025
View reviewed changes
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Looks great to me. Thanks very much.

@kmartens27 this pull request should be merged after the 2.536 release is complete and the 2.536 changelog has merged from pull request:

@kmartens27 kmartens27 enabled auto-merge (squash) November 11, 2025 16:49
@kmartens27 kmartens27 merged commit 2e176e5 into jenkins-infra:master Nov 11, 2025
6 checks passed
@welcome
Copy link

welcome bot commented Nov 11, 2025

Congratulations on getting your very first jenkins.io pull request merged 🎉🥳

This is a fantastic achievement, and we're thrilled to have you as part of our community! Thank you for your valuable input, and we look forward to seeing more of your contributions in the future!

We would like to invite you to join the community chats and forums to meet other Jenkins contributors 😊
Don't forget to check out the participation page to learn more about how to contribute to Jenkins.

@gbhat618 gbhat618 deleted the document-healthcheck-thread-dump-timeout-property branch November 11, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MarkEWaite MarkEWaite MarkEWaite approved these changes

Assignees

No one assigned

Labels

documentation Jenkins documentation, including user and developer docs, solution pages, etc.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gbhat618 @MarkEWaite @kmartens27