Skip to content

[8.16] Fix generation of dynamic mapping for object with specific subfield (#204104)#204491

Closed
kibanamachine wants to merge 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-204104
Closed

[8.16] Fix generation of dynamic mapping for object with specific subfield (#204104)#204491
kibanamachine wants to merge 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-204104

Conversation

@kibanamachine
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

…lastic#204104)

Fix generation of dynamic mapping for objects that have more specific
subfields in separate definitions.

This can be reproduced for example with:
```
- name: labels
  type: object
  object_type: keyword
  object_type_mapping_type: '*'
- name: labels.count
  type: long
```

Fleet expands and deduplicates field definitions before generating the
mappings, so the definitions above are converted to something like the
following:
```
- name: labels
  type: group
  object_type: keyword
  object_type_mapping_type: '*'
  fields:
  - name: count
    type: long
```

Usually fields of type `group` don't have an `object_type`, so this was
being ignored, the dynamic mapping was not being generated.

This issue was not reproduced if the object field name includes a
wildcard, like in `labels.*`, because then the expansion and
deduplication resolves to something like this:
```
- name: labels
  type: group
  fields:
  - name: '*'
    type: object
    object_type: keyword
    object_type_mapping_type: '*'
  - name: count
    type: long
```

(cherry picked from commit e3877e0)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Dec 16, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) December 16, 2024 21:03
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Dec 16, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Dec 16, 2024

💔 Build Failed

Failed CI Steps

History

cc @jsoriano

@jsoriano jsoriano disabled auto-merge December 17, 2024 12:50
@jsoriano jsoriano closed this Dec 17, 2024
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 Team:Fleet Team label for Observability Data Collection Fleet team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants