Skip to content

[8.16] [ES Query] Fix saving ECS group by fields for query DSL rule (#203769)#204345

Merged
maryam-saeidi merged 3 commits intoelastic:8.16from
maryam-saeidi:backport/8.16/pr-203769
Dec 17, 2024
Merged

[8.16] [ES Query] Fix saving ECS group by fields for query DSL rule (#203769)#204345
maryam-saeidi merged 3 commits intoelastic:8.16from
maryam-saeidi:backport/8.16/pr-203769

Conversation

@maryam-saeidi
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

…#203769)

Fixes elastic#203472

## Summary

|Rule|Group info|
|---|---|

|![image](https://github.com/user-attachments/assets/fc17c630-d7c2-4615-8056-5e04209b71e6)|![image](https://github.com/user-attachments/assets/55328973-d585-4148-a74f-d2c275b9989d)|

@elastic/response-ops What sort of test do you suggest to add for this
case?

### 🧪 How to run test

#### Deployment agnostic
- [x] Test on MKI
```
// Server
node scripts/functional_tests_server --config x-pack/test/api_integration/deployment_agnostic/configs/serverless/oblt.serverless.config.ts

// Test
node scripts/functional_test_runner --config=x-pack/test/api_integration/deployment_agnostic/configs/serverless/oblt.serverless.config.ts --grep="ElasticSearch query rule"
```

(cherry picked from commit a0fe4e6)

# Conflicts:
#	x-pack/test/api_integration/deployment_agnostic/apis/observability/alerting/es_query/query_dsl.ts
#	x-pack/test/api_integration/deployment_agnostic/apis/observability/alerting/es_query_rule.ts
#	x-pack/test/api_integration/deployment_agnostic/apis/observability/alerting/index.ts
#	x-pack/test_serverless/api_integration/test_suites/observability/es_query_rule/es_query_rule.ts
@maryam-saeidi maryam-saeidi added the backport This PR is a backport of another PR label Dec 16, 2024
@maryam-saeidi maryam-saeidi enabled auto-merge (squash) December 16, 2024 09:06
@maryam-saeidi maryam-saeidi merged commit c4961a4 into elastic:8.16 Dec 17, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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