-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Synthetics] Add push monitor api #131270
[Synthetics] Add push monitor api #131270
Conversation
1f2be8b
to
dead4ff
Compare
ac4bc07
to
9071b40
Compare
…-ref HEAD~1..HEAD --fix'
…/dominiqueclarke/kibana into feature/synthetics-push-monitors
a2e6516
to
5cf6ffc
Compare
@elasticmachine merge upstream |
Hey Dominique, I've E2E tested this in the following way and have some input to provide. How I tested this PR.
Possible Gotchas
Open Questions/Input |
34ced55
to
143bb63
Compare
@lucasfcosta We've discussed enabled/disabled offline. |
Test it e2e and it's working now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !!
Working e2e
💛 Build succeeded, but was flakyFailed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
PostFF LGTM, tested following @lucasfcosta 's instructions. |
Summary
Creates a new route for syncing push monitors with Monitor Management
This route will handle creating new push monitors, updating existing push monitors, and, when requested, deleting stale push monitors.
This PR does not handle UI changes necessary to support push monitors with the UI Monitor Management table, including disabling editing of project monitors. That will be handled in a follow up PR.
Testing
Adding new monitors
Expected output
Testing that monitors were created successfully
Run this command to get the monitor
Grab the monitor id from the
id
property, then run the following commandExpected output
Updating monitors
Run this command twice
Expected output 1
Expected output 2
Testing reporting of stale monitors
Run this command which changes the id of one of the monitors
Expected output
Testing monitor deletion
Run command with
keep_stale: false
Expected output
Readonly flow
Project monitors should be read only. The enabled toggle should display in a disabled state with a short tooltip explanation.
When clicking the edit button, it will take you to a readonly version of the browser monitor configuration page.
Note: Script is not able to be visuaized at this time, as it is first zipped then base64 encoded. Furher refinement is needed for displaying the file structure within the synthetics app.