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

[8.17](backport #4713) [apm] Update APM decision tree diagram #4717

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 19, 2024

Description

Updates the APM decision tree to address feedback from @eedugon including:

  • Clarifies that when we ask about using other integrations, we mean other integrations with APM Server.
  • Clarifies what we mean by running APM Server on the edge vs centrally.
    • Adds a brief description to the diagram itself.
    • Adds to the list of pros of using a Fleet-managed APM Server that you can centrally manage multiple APM Servers running on edge machines.

Note: In #4669 @eedugon mentioned that where the user is running APM Server (edge machine vs centrally) doesn't technically determine whether they must use Fleet-managed or APM Server binary. However, the goal of this section is to help a new user make a decision based on several factors not to be a complete reference of all implementations that are or aren't possible with Elastic APM. Is there anything we can change about the intro to the "Help me decide" section to make that clearer? @bmorelli25 any thoughts here?

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 issue

Closes #4669

Checklist

  • 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:
    • Port to stateful docs: <link to PR or tracking issue>
    • Port to serverless docs: <link to PR or tracking issue>

This is an automatic backport of pull request #4713 done by [Mergify](https://mergify.com).

* update apm decision tree diagram

* add pro to fleet-managed list

* address feedback from @eedugon

* address feedback from @bmorelli25

(cherry picked from commit f1ce0e1)
@mergify mergify bot added the backport label Dec 19, 2024
@mergify mergify bot requested a review from a team as a code owner December 19, 2024 21:40
Copy link
Contributor

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.

@colleenmcginnis colleenmcginnis enabled auto-merge (squash) December 19, 2024 21:57
@colleenmcginnis colleenmcginnis merged commit 4202c42 into 8.17 Dec 19, 2024
8 checks passed
@colleenmcginnis colleenmcginnis deleted the mergify/bp/8.17/pr-4713 branch December 19, 2024 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant