Skip to content

[eem] _search sort_by and display_name#202361

Merged
klacabane merged 12 commits intoelastic:mainfrom
klacabane:205-sorting-display_name
Dec 2, 2024
Merged

[eem] _search sort_by and display_name#202361
klacabane merged 12 commits intoelastic:mainfrom
klacabane:205-sorting-display_name

Conversation

@klacabane
Copy link
Contributor

@klacabane klacabane commented Nov 29, 2024

  • allow optional entity_source.display_name setting. entity.display_name will always be set on the entities, falling back to entity.id if provided field is not set
  • allow sort_by parameter to _search API
  • removed the metadata. prefix in the query aggregation. metadata will now be set at the root of the document (eg for metadata host.name entity = { entity.id: 'foo', host.name: 'bar' }
  • timestamp_field is now optional

@klacabane klacabane marked this pull request as ready for review November 29, 2024 18:11
@klacabane klacabane requested a review from a team as a code owner November 29, 2024 18:11
@klacabane klacabane self-assigned this Nov 29, 2024
@klacabane klacabane added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Team:obs-entities DEPRECATED - Observability Entities Team labels Nov 29, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-entities (Team:obs-entities)

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Nov 29, 2024
@github-actions
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@elasticmachine
Copy link
Contributor

⏳ Build in-progress

  • Buildkite Build
  • Commit: 7a224a5
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-202361-7a224a579825

History

cc @klacabane

@klacabane klacabane merged commit 697af57 into elastic:main Dec 2, 2024
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Dec 2, 2024
- allow optional `entity_source.display_name` setting.
`entity.display_name` will always be set on the entities, falling back
to `entity.id` if provided field is not set
- allow `sort_by` parameter to `_search` API
- removed the `metadata.` prefix in the query aggregation. metadata will
now be set at the root of the document (eg for metadata `host.name`
entity = `{ entity.id: 'foo', host.name: 'bar' }`
- timestamp_field is now optional

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 697af57)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 2, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[eem] _search sort_by and display_name
(#202361)](#202361)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Kevin
Lacabane","email":"kevin.lacabane@elastic.co"},"sourceCommit":{"committedDate":"2024-12-02T20:15:57Z","message":"[eem]
_search sort_by and display_name (#202361)\n\n- allow optional
`entity_source.display_name` setting.\r\n`entity.display_name` will
always be set on the entities, falling back\r\nto `entity.id` if
provided field is not set\r\n- allow `sort_by` parameter to `_search`
API\r\n- removed the `metadata.` prefix in the query aggregation.
metadata will\r\nnow be set at the root of the document (eg for metadata
`host.name`\r\nentity = `{ entity.id: 'foo', host.name: 'bar' }`\r\n-
timestamp_field is now optional\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"697af576d8aae87c53a2f60646b6657be9b914ad","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","Team:obs-entities"],"title":"[eem]
_search sort_by and
display_name","number":202361,"url":"https://github.com/elastic/kibana/pull/202361","mergeCommit":{"message":"[eem]
_search sort_by and display_name (#202361)\n\n- allow optional
`entity_source.display_name` setting.\r\n`entity.display_name` will
always be set on the entities, falling back\r\nto `entity.id` if
provided field is not set\r\n- allow `sort_by` parameter to `_search`
API\r\n- removed the `metadata.` prefix in the query aggregation.
metadata will\r\nnow be set at the root of the document (eg for metadata
`host.name`\r\nentity = `{ entity.id: 'foo', host.name: 'bar' }`\r\n-
timestamp_field is now optional\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"697af576d8aae87c53a2f60646b6657be9b914ad"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/202361","number":202361,"mergeCommit":{"message":"[eem]
_search sort_by and display_name (#202361)\n\n- allow optional
`entity_source.display_name` setting.\r\n`entity.display_name` will
always be set on the entities, falling back\r\nto `entity.id` if
provided field is not set\r\n- allow `sort_by` parameter to `_search`
API\r\n- removed the `metadata.` prefix in the query aggregation.
metadata will\r\nnow be set at the root of the document (eg for metadata
`host.name`\r\nentity = `{ entity.id: 'foo', host.name: 'bar' }`\r\n-
timestamp_field is now optional\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"697af576d8aae87c53a2f60646b6657be9b914ad"}}]}]
BACKPORT-->

Co-authored-by: Kevin Lacabane <kevin.lacabane@elastic.co>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
- allow optional `entity_source.display_name` setting.
`entity.display_name` will always be set on the entities, falling back
to `entity.id` if provided field is not set
- allow `sort_by` parameter to `_search` API
- removed the `metadata.` prefix in the query aggregation. metadata will
now be set at the root of the document (eg for metadata `host.name`
entity = `{ entity.id: 'foo', host.name: 'bar' }`
- timestamp_field is now optional

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-entities DEPRECATED - Observability Entities Team v8.18.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants