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

Added getPoolNames method to ThreadPoolManager #2065

Merged
merged 1 commit into from
Jan 5, 2021

Conversation

pravussum
Copy link
Contributor

This PR adds a new getPoolNames method to ThreadPoolManager in order to iterate over all thread pools. The intention is to extract metrics for a new Prometheus exporter binding I've got in the works.

Signed-off-by: Robert Bach openhab@mortalsilence.net (github: pravussum)

@pravussum pravussum requested a review from a team as a code owner January 3, 2021 20:55
Signed-off-by: Robert Bach <openhab@mortalsilence.net>
@wborn wborn added rebuild Triggers the Jenkins PR build and removed rebuild Triggers the Jenkins PR build labels Jan 4, 2021
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

lgtm, thanks.

Do you already have anything to share, like a WIP PR for your prometheus exporter? I would be pretty interested in it!

@kaikreuzer kaikreuzer merged commit 75f52ac into openhab:master Jan 5, 2021
@kaikreuzer kaikreuzer added this to the 3.1 milestone Jan 5, 2021
@kaikreuzer kaikreuzer added the enhancement An enhancement or new feature of the Core label Jan 5, 2021
@kaikreuzer kaikreuzer changed the title add getPoolNames method to ThreadPoolManager Added getPoolNames method to ThreadPoolManager Jan 5, 2021
@pravussum
Copy link
Contributor Author

lgtm, thanks.

Do you already have anything to share, like a WIP PR for your prometheus exporter? I would be pretty interested in it!

I've got a working prototype, right now refactoring it to be published as PR. I'll announce it in the forum. Thanks for the merge 👍

@kaikreuzer
Copy link
Member

Prometheus exporter binding

Please note that it must not be done as a binding, though. Exposing openHAB for Prometheus probably best qualifies as an I/O add-on. Please also note #774, which would be the preferred way of implementation (i.e. by using the MicroProfile API).

splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this pull request Jul 11, 2023
Signed-off-by: Robert Bach <openhab@mortalsilence.net>
GitOrigin-RevId: 75f52ac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants