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

[Synthetics] unskip previously flaky test #205573

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

dominiqueclarke
Copy link
Contributor

@dominiqueclarke dominiqueclarke commented Jan 6, 2025

Summary

Resolves #204158
Resolves

Unskips flaky tests.

Passed in stateful and serverless for 50 iterations.

Previous failures related to fleet integration package failures. The logic for installing the synthetics integration via fleet was changed to improve flakiness in #204273. Those changes have also impacted the flakiness of these tests.

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#7645

[✅] x-pack/test/api_integration/deployment_agnostic/configs/serverless/oblt.serverless.config.ts: 50/50 tests passed.
[✅] x-pack/test/api_integration/deployment_agnostic/configs/stateful/oblt.stateful.config.ts: 50/50 tests passed.

see run history

@dominiqueclarke dominiqueclarke added chore backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Team:obs-ux-management Observability Management User Experience Team release_note:skip Skip the PR/issue when compiling release notes labels Jan 6, 2025
@dominiqueclarke dominiqueclarke changed the title unskip test [Synthetics] unskip previously flaky test Jan 6, 2025
@dominiqueclarke dominiqueclarke marked this pull request as ready for review January 6, 2025 14:23
@dominiqueclarke dominiqueclarke requested a review from a team as a code owner January 6, 2025 14:23
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@dominiqueclarke dominiqueclarke merged commit 00a846f into elastic:main Jan 6, 2025
17 checks passed
@dominiqueclarke dominiqueclarke deleted the chore/fix-test branch January 6, 2025 19:00
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12638599407

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Cherrypick failed because the selected commit (00a846f) is empty. Did you already backport this commit?

Manual backport

To create the backport manually run:

node scripts/backport --pr 205573

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205573 locally

kowalczyk-krzysztof pushed a commit to kowalczyk-krzysztof/kibana that referenced this pull request Jan 7, 2025
## Summary

Unskips flaky tests.

Passed in stateful and serverless for 50 iterations.

Previous failures related to fleet integration package failures. The
logic for installing the synthetics integration via fleet was changed to
improve flakiness in elastic#204273. Those changes have also impacted the
flakiness of these tests.
@dominiqueclarke dominiqueclarke added backport:skip This commit does not require backporting and removed backport missing Added to PRs automatically when the are determined to be missing a backport. backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting chore release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-management Observability Management User Experience Team v9.0.0
Projects
None yet
4 participants