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

[8.1] [Uptime] Unskip flaky tests (#130059) #130719

Closed

Conversation

dominiqueclarke
Copy link
Contributor

Backport

This will backport the following commits from main to 8.1:

Questions ?

Please refer to the Backport tool documentation

* unskip flaky test

* move Observer location test to Synthetics e2e

* uptime - update monitor states endpoint api integration tests

* uptime - reskip flaky tests

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 250e3f9)

# Conflicts:
#	x-pack/plugins/uptime/server/rest_api/uptime_route_wrapper.ts
@dominiqueclarke dominiqueclarke enabled auto-merge (squash) April 20, 2022 18:07
Comment on lines +11 to +14
"error": Object {
"message": "400 Bad Request",
"type": "validate",
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
"error": Object {
"message": "400 Bad Request",
"type": "validate",
},

@dominiqueclarke
Copy link
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented Apr 27, 2022

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Default CI Group #18 / apis uptime uptime REST endpoints with real-world data monitor states endpoint will fetch monitor state data for the given down filters
  • [job] [logs] Default CI Group #18 / apis uptime uptime REST endpoints with real-world data monitor states endpoint will fetch monitor state data for the given down filters
  • [job] [logs] Default CI Group #6 / Uptime app with real-world data uptime alerts tls alert can save alert
  • [job] [logs] Default CI Group #6 / Uptime app with real-world data uptime alerts tls alert can save alert

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

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

Successfully merging this pull request may close these issues.

3 participants