Skip to content

[8.1] [Reporting] Switch from EventLog integration to ECS logging (#124762)#125184

Merged
tsullivan merged 1 commit intoelastic:8.1from
tsullivan:backport/8.1/pr-124762
Feb 10, 2022
Merged

[8.1] [Reporting] Switch from EventLog integration to ECS logging (#124762)#125184
tsullivan merged 1 commit intoelastic:8.1from
tsullivan:backport/8.1/pr-124762

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backport

This is an automatic backport to 8.1 of:

Questions ?

Please refer to the Backport tool documentation

…124762)

* [Reporting] Remove EventLog Dependency

* calculate duration

* use LogMeta interface of core logger

* fix ts

* rename the debug log tag

* clean up return types for testing

* remove reporting fields from the event log mappings

* unwrap code from iife

* add class for log adapter

* remove useless factory fn

* remove eventLog

* user field was meant to be ECS field

* duration is nanoseconds

* fix nanoseconds application and test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 87eaa75)
@tsullivan tsullivan added the backport This PR is a backport of another PR label Feb 10, 2022
@tsullivan tsullivan enabled auto-merge (squash) February 10, 2022 03:26
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

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
reporting 30 29 -1
Unknown metric groups

API count

id before after diff
reporting 30 29 -1

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

@tsullivan tsullivan merged commit fa6590b into elastic:8.1 Feb 10, 2022
@tsullivan tsullivan deleted the backport/8.1/pr-124762 branch February 10, 2022 13:28
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.

2 participants