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 synchronous jobs #131

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbanck-ntap
Copy link

We needed a possibility to execute some jobs / queries synchronously every time /metrics is requested. So I added the infrastructure to execute jobs with an interval <= 0 synchronously when the http-handler is called.

interval: '0s' # an interval <= 0 will make the queries synchronous

This is a rebase and re-submission from #10 which erroneously got marked as merged a while ago.

@mbanck-ntap mbanck-ntap mentioned this pull request Jul 17, 2024
We needed a possibility to execute some jobs / queries synchronously every time
/metrics is requested. So I added the infrastructure to execute jobs with an
interval <= 0 synchronously when the http-handler is called.

interval: '0s' # an interval <= 0 will make the queries synchronous

Some typos fixed by Felix Hillingshaeuser, thanks!
@mbanck-ntap
Copy link
Author

Did somebody take a look at this?

@dewey
Copy link
Member

dewey commented Feb 14, 2025

Hey @mbanck-ntap, I don't really have time to test it, but it looks good to me and as everything is limited to the 0 case it shouldn't cause any issues for people not using it. Could you rebase on master then I could merge it and create a release.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants