Skip to content

[8.15] [ AutoImport] Introduce automatic log type detection graph (#190407)#192403

Merged
bhapas merged 3 commits intoelastic:8.15from
bhapas:backport/8.15/pr-190407
Sep 9, 2024
Merged

[8.15] [ AutoImport] Introduce automatic log type detection graph (#190407)#192403
bhapas merged 3 commits intoelastic:8.15from
bhapas:backport/8.15/pr-190407

Conversation

@bhapas
Copy link
Copy Markdown
Contributor

@bhapas bhapas commented Sep 9, 2024

Backport

This will backport the following commits from main to 8.15:

Questions ?

Please refer to the Backport tool documentation

…90407)

## Summary

This PR introduces a new graph in `Auto Import` called -
`LogTypeDetection`

Currently, only JSON/NDJSON formats are supported to be uploaded for
building custom integrations. With this feature the capabilities to
upload different log types is allowed.

Although parsing of the new log types will be handled separately with a
separate [issue.](elastic/security-team#9845)

- The logs are initially parsed for JSON/NDJSON types in the UI side.
- If it is not JSON/NDJSON format , then a new API `AnalyzeLogs` is
triggered.
- UI allows any type of logs to be uploaded.
- Currently there is a server level content length restriction of `1MB`
which needs to be extended.
- For any log types other than JSON/NDJSON the handling graphs are not
yet implemented , hence a `501 Not implemented` message appears.
- The idea is to support `structured` , `csv` , `unstructured` syslog
handling graphs.

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Hanna Tamoudi <hanna.tamoudi@elastic.co>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 9f01f73)

# Conflicts:
#	x-pack/plugins/integration_assistant/server/types.ts
@bhapas bhapas added the backport This PR is a backport of another PR label Sep 9, 2024
@bhapas bhapas enabled auto-merge (squash) September 9, 2024 18:56
@bhapas bhapas self-assigned this Sep 9, 2024
@bhapas bhapas requested a review from kgeller September 9, 2024 19:00
@bhapas bhapas added release_note:feature Makes this part of the condensed release notes Team:Security-Scalability Security Integrations Scalability Team labels Sep 9, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-scalability (Team:Security-Scalability)

@bhapas bhapas merged commit b8fa90d into elastic:8.15 Sep 9, 2024
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
integrationAssistant 544 547 +3

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
integrationAssistant 41 46 +5

Async chunks

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

id before after diff
integrationAssistant 937.6KB 938.7KB +1.0KB
Unknown metric groups

API count

id before after diff
integrationAssistant 49 54 +5

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @bhapas

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 release_note:feature Makes this part of the condensed release notes Team:Security-Scalability Security Integrations Scalability Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants