Skip to content

[8.16] [Security Solution] Fix PATCH rule API test failure in Serverless (#202885)#202942

Merged
nikitaindik merged 1 commit intoelastic:8.16from
nikitaindik:backport/8.16/pr-202885
Dec 4, 2024
Merged

[8.16] [Security Solution] Fix PATCH rule API test failure in Serverless (#202885)#202942
nikitaindik merged 1 commit intoelastic:8.16from
nikitaindik:backport/8.16/pr-202885

Conversation

@nikitaindik
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

…astic#202885)

**Resolves: elastic#202812

## Summary
This PR fixes a problem in the MKI Serverless periodic pipeline that was
introduced in elastic#201825. The issue happened because the test tried to
delete prebuilt rule assets stored in .kibana_security_solution, but you
can’t access this index in Serverless MKI.

The fix makes sure this call only runs in non-Serverless MKI
environments.

(cherry picked from commit 0e10dbf)

# Conflicts:
#	x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/rule_patch/basic_license_essentials_tier/patch_rules.ts
@nikitaindik nikitaindik added the backport This PR is a backport of another PR label Dec 4, 2024
@nikitaindik nikitaindik enabled auto-merge (squash) December 4, 2024 14:34
@nikitaindik nikitaindik merged commit 8dc8fd4 into elastic:8.16 Dec 4, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #3 / EQL Tab rendering should render the timeline table
  • [job] [logs] FTR Configs #80 / Observability Rules Synthetics SyntheticsCustomStatusRule "after all" hook in "SyntheticsCustomStatusRule"

Metrics [docs]

✅ unchanged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants