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] Describe how to use Elastic Agent to monitor Logstash #14959

Merged
merged 5 commits into from Mar 23, 2023

Conversation

dedemorton
Copy link
Contributor

@dedemorton dedemorton commented Mar 11, 2023

Release notes

[rn:skip]

What does this PR do?

Adds Elastic Agent as another way to collect monitoring data.

This work is tracked by elastic/observability-docs#2602.

There are additional PRs to address changes required to monitoring docs for other stack components. These are tactical fixes to get the agent info into all the docs.

Please respond to questions addressed to reviewers.

Why is it important/What is the impact to the user?

Users need to know that Elastic Agent integrations are now available and how to use them.

Checklist

  • My code follows the style guidelines of this project

Related issues

To Do before merging

  • Remove questions to reviewers.

@github-actions
Copy link
Contributor

📃 DOCS PREVIEWhttps://logstash_14959.docs-preview.app.elstc.co/diff

docs/static/monitoring/monitoring-ea.asciidoc Outdated Show resolved Hide resolved
docs/static/monitoring/monitoring-ea.asciidoc Outdated Show resolved Hide resolved
docs/static/monitoring/monitoring-ea.asciidoc Outdated Show resolved Hide resolved
Co-authored-by: Kevin Lacabane <klacabane@gmail.com>
@github-actions
Copy link
Contributor

📃 DOCS PREVIEWhttps://logstash_14959.docs-preview.app.elstc.co/diff

@dedemorton
Copy link
Contributor Author

@klacabane When the doc build finishes, this PR will be ready for final review. Thanks!

@github-actions
Copy link
Contributor

📃 DOCS PREVIEWhttps://logstash_14959.docs-preview.app.elstc.co/diff

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Thank you for your work on this. Looks great!
I left some comments inline for your consideration. Otherwise, LGTM.

docs/static/monitoring/monitoring-overview.asciidoc Outdated Show resolved Hide resolved
docs/static/monitoring/monitoring-ea.asciidoc Outdated Show resolved Hide resolved
dedemorton and others added 2 commits March 23, 2023 09:34
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
@github-actions
Copy link
Contributor

📃 DOCS PREVIEWhttps://logstash_14959.docs-preview.app.elstc.co/diff

Copy link
Contributor

@klacabane klacabane left a comment

Choose a reason for hiding this comment

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

🚀

@dedemorton dedemorton merged commit 58abffc into elastic:main Mar 23, 2023
4 checks passed
@dedemorton dedemorton deleted the obsdocs-issue#2602 branch March 23, 2023 18:23
@dedemorton
Copy link
Contributor Author

@logstashmachine backport 8.7

@karenzone
Copy link
Contributor

@logstashmachine backport 8.7

github-actions bot pushed a commit that referenced this pull request Mar 23, 2023
* [DOCS] Describe how to use Elastic Agent to monitor Logstash

* Apply suggestions from code review

Co-authored-by: Kevin Lacabane <klacabane@gmail.com>

* Remove reviewer questions

* Apply suggestions from code review

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

* Fix security statement

---------

Co-authored-by: Kevin Lacabane <klacabane@gmail.com>
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
(cherry picked from commit 58abffc)
dedemorton added a commit that referenced this pull request Mar 24, 2023
…#14971)

* [DOCS] Describe how to use Elastic Agent to monitor Logstash

* Apply suggestions from code review

Co-authored-by: Kevin Lacabane <klacabane@gmail.com>

* Remove reviewer questions

* Apply suggestions from code review

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

* Fix security statement

---------

Co-authored-by: Kevin Lacabane <klacabane@gmail.com>
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
(cherry picked from commit 58abffc)

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
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.

None yet

4 participants