Skip to content

Conversation

colleenmcginnis
Copy link
Contributor

Cleans up observability apps section including:

  • In content that applies to both stateful and serverless, use APM Server or the managed intake service instead of APM Server as there is no APM Server in serverless.
  • Add missing serverless content (e.g. getting started, api keys).
  • Update syntax that leads to rendering issues.
  • Simplify internal links.
  • Clean up unnecessary white space.

@colleenmcginnis colleenmcginnis requested a review from a team April 7, 2025 15:35
@colleenmcginnis colleenmcginnis self-assigned this Apr 7, 2025
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I'm not creating a redirect here because in the final list of redirects for the web team, we're redirecting https://www.elastic.co/guide/en/observability/current/apm-performance-diagnostic.html to troubleshoot/observability/apm/apm-server-performance-diagnostic.md.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I'm not creating a redirect here because in the final list of redirects for the web team, we're redirecting https://www.elastic.co/guide/en/observability/current/apm-common-response-codes.html to troubleshoot/observability/apm/apm-server-response-codes.md.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I'm not creating a redirect here because in the final list of redirects for the web team, we're redirecting https://www.elastic.co/guide/en/observability/current/apm-enable-apm-server-debugging.html to
troubleshoot/observability/apm/enable-apm-server-binary-debugging.md.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I'm not creating a redirect here because in the final list of redirects for the web team, we're redirecting https://www.elastic.co/guide/en/observability/current/apm-processing-and-performance.html to troubleshoot/observability/apm/processing-performance.md.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I'm not creating a redirect here because in the final list of redirects for the web team, we're redirecting https://www.elastic.co/guide/en/observability/current/apm-server-es-down.html to troubleshoot/observability/apm/what-happens-when-apm-server-elasticsearch-is-down.md.

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 couple of small things, otherwise LGTM! 🦭

@colleenmcginnis colleenmcginnis enabled auto-merge (squash) April 9, 2025 15:13
@colleenmcginnis colleenmcginnis merged commit 6e72537 into main Apr 9, 2025
3 checks passed
@colleenmcginnis colleenmcginnis deleted the obs-apps-cleanup branch April 9, 2025 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants