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.

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

[8.x] Handle Json in Set and Append Value field (#204336) #205470

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

Closes [elastic#193186](elastic#193186)

## Sumary

This PR introduces changes in two processors: Set y Append. The aim of
this changes is to allow introducing the Value field in both processors
in Json format. Now, Set accepts Text format or Json and Append a list
of values or Json. The toggle between the format is done by the new
Label Append button.
<img width="705" alt="Screenshot 2024-12-16 at 07 30 11"
src="https://github.com/user-attachments/assets/0f6283ed-6117-48d8-bdd8-e0685ae1b42d"
/>

Also, since the Set processor already had a Label Append button for
toggling between the `Value` and the `Copy_from` field, this now has
been replaced by a toggle button that enables a field for the
`Copy_from` field (see the last comments in the related Issue for see
the most updated mocks).
<img width="708" alt="Screenshot 2024-12-16 at 07 30 49"
src="https://github.com/user-attachments/assets/4a632bda-4b4b-49b1-b3d4-d11579d7c27c"
/>

Note: the Set copies still pending review so probably will be changed.

### Demo

https://github.com/user-attachments/assets/f9d1da6a-782d-4197-836b-fab46b4476b7
(cherry picked from commit 59b2292)
@kibanamachine kibanamachine force-pushed the backport/8.x/pr-204336 branch from 9493afb to e22de77 Compare January 3, 2025 06:58
@kibanamachine kibanamachine merged commit 4152249 into elastic:8.x Jan 3, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Integration Tests #4 / capacity based claiming should claim tasks to full capacity

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
ingestPipelines 332 333 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ingestPipelines 403.8KB 406.4KB +2.6KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
ingestPipelines 15.1KB 15.1KB +47.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
ingestPipelines 29 32 +3

Total ESLint disabled count

id before after diff
ingestPipelines 29 32 +3

cc @SoniaSanzV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants