Skip to content

Cherry-pick #19211 to 7.x: Use libbeat logger #19553

Merged
michalpristas merged 1 commit intoelastic:7.xfrom
michalpristas:backport_19211_7.x
Jul 2, 2020
Merged

Cherry-pick #19211 to 7.x: Use libbeat logger #19553
michalpristas merged 1 commit intoelastic:7.xfrom
michalpristas:backport_19211_7.x

Conversation

@michalpristas
Copy link
Copy Markdown
Contributor

Cherry-pick of PR #19211 to 7.x branch. Original message:

What does this PR do?

Changes our dependency on ecs logger and uses libbeat logger by default.
By default we log to file now, so users can share logs easier.

Why is it important?

ECS compliant logging.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Fixes: #15129

@michalpristas michalpristas self-assigned this Jul 1, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jul 1, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ingest-management (Team:Ingest Management)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jul 1, 2020
Copy link
Copy Markdown
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Backport looks good.

@michalpristas michalpristas merged commit 1f88f81 into elastic:7.x Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Ingest Management:beta1 Group issues for ingest management beta1 review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants