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

[Inventory] Create new Inventory docs #4341

Merged
merged 19 commits into from
Oct 6, 2024
Merged

Conversation

alaudazzi
Copy link
Contributor

@alaudazzi alaudazzi commented Oct 3, 2024

Description

This PR adds a new page on Inventory explaining what this feature is and how to use it.

Documentation sets edited in this PR

Check all that apply.

  • Stateful (docs/en/observability/*)
  • Serverless (docs/en/serverless/*)
  • Integrations Developer Guide (docs/en/integrations/*)
  • None of the above

Related issues

Checklist

  • Initial draft updates
  • Product/Engineering Review
  • Writer Review

Follow-up tasks

Select one.

  • This PR does not need to be ported to another doc set because:
    • The concepts in this PR only apply to one doc set (serverless or stateful)
    • The PR contains edits to both doc sets (serverless and stateful)
  • This PR needs to be ported to another doc set:

Copy link
Contributor

github-actions bot commented Oct 3, 2024

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

Copy link
Contributor

mergify bot commented Oct 3, 2024

This pull request does not have a backport label. Could you fix it @alaudazzi? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-/d./d is the label to automatically backport to the /d./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

Copy link
Contributor

@roshan-elastic roshan-elastic left a comment

Choose a reason for hiding this comment

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

Hey @alaudazzi - looking good!

I have a few suggestions to tweak for now (mainly based on what is available right now):

From there, you can monitor and understand the health of your entities, check what needs attention and start your investigations.

I'm wondering if it's best practice to talk about upcoming functionality that isn't available? The above will be true but just for an iteration. I'll take your lead on this.

but it can scale to support all of your entities.

Same as above. It doesn't yet but it will (we'll start by incrementally adding new entities ourselves, like K8s, but it won't scale to support all entities for a few months).

Is it OK to say that it 'will' scale to support all of your entities?

Hosts: Anything where host.name is set in a metrics-* or logs-* index
Containers: Anything where container.id is set in a metrics-* or logs-* index
Services: Anything where service.name is set in a apm-* or logs-* index

I'm wondering whether it makes sense to put these in a separate entity section (which would be a catalogue of all OOTB entities)? I'm guessing you thought of this but it makes more sense to keep it simple and specify that here for now?

Understand what needs attention by easily seeing what entities are alerting

Same as above. This will come in the next iteration but not yet. Given not many people will read this - I'm OK with keeping it like this if you think it's best practice from a docs POV?

Drill down into any host, container or services with curated entity views to help you understand performance

I'm thinking it might be better to remove 'with curated entity views'? You'll be able to drill down into any entity in the list (all currently have curated views but new entities will likely use dashboards for a while until we can make the curated view a template).

Explore the entities catalog

Do you think we could rename this as 'Explore your entities'? I'm worried users might think the entities catalog is another feature.

@alaudazzi
Copy link
Contributor Author

alaudazzi commented Oct 3, 2024

Thank you for your feedback, @roshan-elastic!
I'll have a second pass and ping you as soon as the next draft is available. And I'm going to get a working doc preview for you :-)

@alaudazzi
Copy link
Contributor Author

@roshan-elastic do we need to include these concepts?

@roshan-elastic
Copy link
Contributor

Hey @alaudazzi - I would only include what we've shipped for now to keep it really light.

Copy link
Contributor

@colleenmcginnis colleenmcginnis left a comment

Choose a reason for hiding this comment

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

@alaudazzi can you try these changes? I think that should fix the build.

docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
@elasticdocs
Copy link

elasticdocs commented Oct 3, 2024

🚀 Built elastic-dot-co-docs-preview-docs successfully!

Issues? Visit #next-docs in Slack

@alaudazzi
Copy link
Contributor Author

alaudazzi commented Oct 3, 2024

@roshan-elastic
I integrated your comments, and elaborated a bit more on the part related to how the definitions for each entity are detected. This is a doc preview. Other than some minor formatting, please let me know if more changes are required.

Copy link
Contributor

@roshan-elastic roshan-elastic left a comment

Choose a reason for hiding this comment

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

Hey @alaudazzi,

Looking good - just a few tweaks. Hopefully these are straight copy changes if you're OK with that?

In your Observability project, go to Inventory to open the entities catalog.

In your Observability project, go to Inventory to view all of your entities.

If you have Admin privileges, the EEM will be turned on automatically. Otherwise, you need to ask someone with Admin privileges to turn on this feature under Stack Management.

When you open the Inventory for the first time, you will need to enable the Elastic Entity Model in order to use it. Once enabled, the Inventory will be useable for everyone with the correct privileges to access Inventory.

Note that the Inventory feature can be completely disabled using the 'observability:entityCentricExperience' flag in Stack Management.

For each entity, you can either view your data in Discover, or click the entity name and get a detailed view. For example, for an entity of type service, you get the following details:

For each entity, you can either view click the entity name and get a detailed view. For example, for an entity of type service, you get the following details:

Entities are added to the inventory through one of the following approaches:

Add data

Auto-detect logs and metrics Detects hosts (with metrics and logs)
Kubernetes Detects hosts, containers and services
Elastic APM / OpenTelemetry / Synthetic Monitor Detects services

Entities are added to the inventory through one of the following approaches:

Add data
You can add entities using the following onboarding journeys via the left-hand navigation item 'Add data':

  • Auto-detect logs and metrics: Detects hosts (with metrics and logs)
  • Kubernetes: Detects hosts, containers and services
  • Elastic APM / OpenTelemetry / Synthetic Monitor: Detects services

@alaudazzi
Copy link
Contributor Author

@roshan-elastic
Thank you for your feedback! I integrated your comments and added some more edits.
Here is the latest doc preview.

Copy link
Contributor

@roshan-elastic roshan-elastic left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @alaudazzi

roshan-elastic
roshan-elastic previously approved these changes Oct 4, 2024
Copy link
Contributor

@mdbirnstiehl mdbirnstiehl left a comment

Choose a reason for hiding this comment

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

Just a few suggestions for you to take or leave.

docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
docs/en/serverless/inventory.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

@mdbirnstiehl mdbirnstiehl left a comment

Choose a reason for hiding this comment

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

🚀

@alaudazzi alaudazzi merged commit c618242 into elastic:main Oct 6, 2024
4 checks passed
alaudazzi added a commit that referenced this pull request Oct 31, 2024
* Add new page on inventory

* Update docs/en/observability/apm/view-and-analyze/inventory.asciidoc

Co-authored-by: DeDe Morton <[email protected]>

* Crop images to remove nav pane

---------

Co-authored-by: DeDe Morton <[email protected]>
mergify bot pushed a commit that referenced this pull request Oct 31, 2024
* Add new page on inventory

* Update docs/en/observability/apm/view-and-analyze/inventory.asciidoc

Co-authored-by: DeDe Morton <[email protected]>

* Crop images to remove nav pane

---------

Co-authored-by: DeDe Morton <[email protected]>
(cherry picked from commit c54a48d)
mergify bot pushed a commit that referenced this pull request Oct 31, 2024
* Add new page on inventory

* Update docs/en/observability/apm/view-and-analyze/inventory.asciidoc

Co-authored-by: DeDe Morton <[email protected]>

* Crop images to remove nav pane

---------

Co-authored-by: DeDe Morton <[email protected]>
(cherry picked from commit c54a48d)
alaudazzi added a commit that referenced this pull request Oct 31, 2024
* Add new page on inventory

* Update docs/en/observability/apm/view-and-analyze/inventory.asciidoc

Co-authored-by: DeDe Morton <[email protected]>

* Crop images to remove nav pane

---------

Co-authored-by: DeDe Morton <[email protected]>
(cherry picked from commit c54a48d)

Co-authored-by: Arianna Laudazzi <[email protected]>
alaudazzi added a commit that referenced this pull request Oct 31, 2024
* Add new page on inventory

* Update docs/en/observability/apm/view-and-analyze/inventory.asciidoc

Co-authored-by: DeDe Morton <[email protected]>

* Crop images to remove nav pane

---------

Co-authored-by: DeDe Morton <[email protected]>
(cherry picked from commit c54a48d)

Co-authored-by: Arianna Laudazzi <[email protected]>
eedugon pushed a commit to eedugon/observability-docs that referenced this pull request Nov 6, 2024
…c#4438)

* Add new page on inventory

* Update docs/en/observability/apm/view-and-analyze/inventory.asciidoc

Co-authored-by: DeDe Morton <[email protected]>

* Crop images to remove nav pane

---------

Co-authored-by: DeDe Morton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-skip Skip notification from the automated backport with mergify needs-writer-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants