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

[DOCS] Add ECS fields to Logs monitoring guide #1203

Merged
merged 4 commits into from
Jun 29, 2020

Conversation

EamonnTP
Copy link
Contributor

@EamonnTP EamonnTP commented Jun 18, 2020

This PR adds the ECS fields required for populating the Logs UI in Kibana. The PR also addresses minor issues in the Logs installation instructions that were called out in this issue.

Preview

Installation: https://stack-docs_1203.docs-preview.app.elstc.co/guide/en/logs/guide/master/install-logs-monitoring.html

ECS fields: https://stack-docs_1203.docs-preview.app.elstc.co/guide/en/logs/guide/master/logs-fields-reference.html

Related issues

Closes #646
https://github.com/elastic/observability-dev/issues/223

@EamonnTP EamonnTP requested a review from Kerry350 June 23, 2020 08:30
Copy link

@Kerry350 Kerry350 left a comment

Choose a reason for hiding this comment

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

LGTM bar one comment.

@EamonnTP EamonnTP merged commit 87c115e into elastic:master Jun 29, 2020
@EamonnTP EamonnTP deleted the add-logs-fields branch June 29, 2020 15:55
EamonnTP pushed a commit to EamonnTP/stack-docs that referenced this pull request Jun 29, 2020
* Add ECS fields

* Fix indentation

* Update fields list

* Update following review
EamonnTP pushed a commit to EamonnTP/stack-docs that referenced this pull request Jun 29, 2020
* Add ECS fields

* Fix indentation

* Update fields list

* Update following review
EamonnTP pushed a commit that referenced this pull request Jun 29, 2020
* Add ECS fields

* Fix indentation

* Update fields list

* Update following review
EamonnTP pushed a commit that referenced this pull request Jun 29, 2020
* Add ECS fields

* Fix indentation

* Update fields list

* Update following review
@webmat
Copy link

webmat commented Jul 2, 2020

Love this new page, thanks for getting this done!

Quick adjustment: log.file.path is part of ECS, so you can change its entry around to "ECS Field: True" :-)

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

Successfully merging this pull request may close these issues.

[DOCS] Minor issues in Logs installation instructions
4 participants