-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Spike] [Heartbeat] Explore implementing timeout
for browser monitors
#29454
Comments
Pinging @elastic/uptime (Team:Uptime) |
Building this correctly for inline monitors is pretty straightforward, it's a bit more complex for suites. I propose the following:
|
All of this states we need ahead of time compilation for the HB especially for suites, haven't thought about how we would do this. |
@dominiqueclarke @andrewvc with zip url and suites being deprecated, is this still a requirement? |
Hi! We're labeling this issue as |
Captured in elastic/synthetics#133, closing |
Heartbeat exposes a
timeout
field for monitor types. However,browser
monitors do not currently supporttimeout
and the value is ignoredWe should consider implementing
timeout
for browser monitors, and the side-effects of doing so. In particular, we will need to determine how this logic ought to behave for both inline and zip URL monitors.The text was updated successfully, but these errors were encountered: