Skip to content

Conversation

@ridwanmsharif
Copy link
Collaborator

@ridwanmsharif ridwanmsharif commented Nov 17, 2023

This change adds a config option to the scrape manager that is quite useful on serverless environments where we are sensitive to the start up latencies of the scraper. The serverless instance might only last for a few seconds and may not be able to afford the minimum 5s reload interval before the scrape pools are created.

This change also adds an option in the discovery manager that will allow users to skip the initial
wait before sending target sets to the scrape manager.

@pintohutch pintohutch requested a review from bwplotka November 17, 2023 20:58
@ridwanmsharif ridwanmsharif force-pushed the ridwanmsharif/instant-reload branch 3 times, most recently from 44e9516 to 37e0de9 Compare November 20, 2023 20:39
@ridwanmsharif
Copy link
Collaborator Author

Rebased on top of #119 which fixes the flakey test.

This config option is quite useful on serverless environments where
we are sensitive to the start up latencies of the scraper. The
serverless instance might only last for a few seconds and may not be
able to afford the minimum 5s reload interval before the scrape pools
are created.

Signed-off-by: Ridwan Sharif <ridwanmsharif@google.com>
@ridwanmsharif ridwanmsharif force-pushed the ridwanmsharif/instant-reload branch from 37e0de9 to ad19320 Compare November 20, 2023 21:26
This change adds an option that will allow users to skip the initial
wait before sending target sets to the scrape manager. This is
particularly useful in environments where the startup latency is
required to be low just as in serverless deployments.

Signed-off-by: Ridwan Sharif <ridwanmsharif@google.com>
@bwplotka
Copy link
Collaborator

Looks like #122 is already better, and there is even #123 so closing this one 🤗

@bwplotka bwplotka closed this Nov 22, 2023
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