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.

Sign up for GitHub

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

Jump to bottom

[Detection Engine][Tests] Skip flakey ML tests - need to follow up #196990

Merged
merged 1 commit into from
Oct 21, 2024
Merged

[Detection Engine][Tests] Skip flakey ML tests - need to follow up #196990

merged 1 commit into from
Oct 21, 2024

Conversation

yctercero
Copy link
Contributor

Summary

The periodic pipeline ML tests have been flaking. Confirmed flake, not regression. Skipping tests to stabilize pipeline. We will need to do a quick follow up to re-enable tests.

@yctercero yctercero added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Detection Engine Security Solution Detection Engine Area backport:version Backport to applied version labels labels Oct 21, 2024
@yctercero yctercero self-assigned this Oct 21, 2024
@yctercero yctercero requested a review from a team as a code owner October 21, 2024 02:27
@yctercero yctercero requested a review from vitaliidm October 21, 2024 02:27
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detection-engine (Team:Detection Engine)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @yctercero

@nkhristinin nkhristinin merged commit c7c5d82 into elastic:main Oct 21, 2024
29 checks passed
@nkhristinin
Copy link
Contributor

Merged this PR by myself, to not block quality gate

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 23, 2024
@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 196990 locally

@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 196990 locally

5 similar comments
@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 196990 locally

@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 196990 locally

@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 196990 locally

@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 196990 locally

@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 196990 locally

@yctercero yctercero added the backport:skip This commit does not require backporting label Oct 31, 2024
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nkhristinin nkhristinin nkhristinin approved these changes

@vitaliidm vitaliidm Awaiting requested review from vitaliidm vitaliidm is a code owner automatically assigned from elastic/security-detection-engine

Assignees

@yctercero yctercero

Labels
backport:skip This commit does not require backporting backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Detection Engine Security Solution Detection Engine Area v9.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yctercero @elasticmachine @nkhristinin @kibanamachine