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

Elastic Agent Integration - we can add more clarity to the overview notes section on what this is and how it relates to Elastic Agent overall #723

Closed
EricDavisX opened this issue Feb 17, 2021 · 8 comments

Comments

@EricDavisX
Copy link
Contributor

we got feedback from a CSM that it wasn't clear. So I'm putting a ticket in so we can review and improve it.

The note that caught their attention was that... the Integration says it is "only to monitor itself" and the implication was taken to apply that to all of the Elastic Agent functionality. It is confusing because we have Elastic Agent and then an Elastic Agent integration.

However it is best to impart this, we probably want to inform that this specific integration is in fact for monitoring the Elastic Agent, but that note is not meant to imply that monitoring the Agent process is all that Elastic Agent can do.

elastic-agent-integration

@EricDavisX EricDavisX added the Team:Fleet Label for the Fleet team [elastic/fleet] label Feb 17, 2021
@elasticmachine
Copy link

Pinging @elastic/fleet (Team:Fleet)

@ph
Copy link
Contributor

ph commented Feb 17, 2021

@mostlyjason Can you work with @nchaulet to improve the description of this integration?

@mostlyjason
Copy link
Contributor

@nchaulet did you follow the guidelines for creating this package and dashboards? https://github.com/elastic/obs-dc-team/blob/master/development/generic_guidelines.md #269. For example, I don't see a screenshot of the dashboards. The process table also looks like it has a markdown error.

CC @sorantis in case there are other guidelines we should follow

Also, does this integration add any inputs or is it just assets? Is it a default package?

@ph
Copy link
Contributor

ph commented Feb 18, 2021

It currently just add assets.

@mostlyjason in 7.13 we want to make it a default package like system.

@jen-huang jen-huang added Integration:elastic_agent Elastic Agent Team:Elastic-Agent Label for the Agent team and removed Team:Fleet Label for the Fleet team [elastic/fleet] labels May 3, 2021
@elasticmachine
Copy link

Pinging @elastic/agent (Team:Agent)

@ph
Copy link
Contributor

ph commented May 4, 2021

@mostlyjason Currently It only contains dashboard, Maybe we could write something like this.

The Elastic Agent integration is a default integration that contains the dashboards required to visualize the monitoring information of a running Elastic Agent and the running processes. The integration will be automatically updated on every new release of the stack.

@mostlyjason
Copy link
Contributor

mostlyjason commented May 10, 2021

How about this?

The Elastic Agent integration is a default integration that contains the dashboards required to visualize the monitoring information of a running Elastic Agent and the running processes. It does not need to be added to agent policies. Enable agent monitoring in the agent policy settings instead. The integration will be automatically updated on every new release of the stack.

This is another good example for this use case elastic/kibana#96525

@ph ph added the v7.14.0 label May 11, 2021
@botelastic
Copy link

botelastic bot commented Sep 19, 2022

Hi! We just realized that we haven't looked into this issue in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Sep 19, 2022
@botelastic botelastic bot closed this as completed Mar 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants