-
Notifications
You must be signed in to change notification settings - Fork 166
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
[apm] Clarify requirements in the Fleet-managed APM Server guide #4714
Conversation
…ting started guide
A documentation preview will be available soon. Request a new doc build by commenting
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. |
Your explanation makes sense to me. I'm supportive of this approach. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the changes, and I'm totally ok with them. Thanks!
…ting started guide (#4714) (#4721) (cherry picked from commit 33da1d1) Co-authored-by: Colleen McGinnis <[email protected]>
…ting started guide (#4714) (#4722) (cherry picked from commit 33da1d1) Co-authored-by: Colleen McGinnis <[email protected]>
…ting started guide (#4714) (#4723) (cherry picked from commit 33da1d1) Co-authored-by: Colleen McGinnis <[email protected]>
Description
Updates the Fleet-managed APM Server guide to address feedback from @eedugon including:
There are some points that @eedugon brought up in #4668 that are not addressed here including:
The content for both the Prerequisites and Step 1: Set up Fleet sections come from the Fleet docs (here and here). We could link out to the Fleet guide instead of importing this content into this APM doc, but in my opinion we should only do that if it's the best user experience, not because of maintenance concerns. However, it would probably be a good idea to review the content in the elastic/ingest-docs repo to make sure it's up to date and as clear as possible.
@eedugon @bmorelli25 thoughts on this approach?
Documentation sets edited in this PR
Check all that apply.
docs/en/observability/*
)docs/en/serverless/*
)docs/en/integrations/*
)Related issue
Closes #4668
Checklist
Follow-up tasks
Select one.