Skip to content

Conversation

@lucaspimentel
Copy link
Member

@lucaspimentel lucaspimentel commented Oct 29, 2025

What does this PR do? What is the motivation?

Adds a Prerequisites section and improves cross-referencing between the AWS Lambda .NET documentation and the main .NET Core APM docs.

Changes:

  • Adds Prerequisites section at the beginning of the Lambda .NET instrumentation page
  • Links to .NET Core compatibility requirements and main tracing documentation
  • Clarifies Lambda-specific differences (layers vs MSI, version matching, environment variable configuration)
  • Adds bidirectional link from Lambda docs back to traditional host setup
  • Adds Configuration section explaining Lambda-specific configuration notes
  • Adds Troubleshooting section with links to diagnostic tools

Motivation:
Users coming to the Lambda documentation often lack context about .NET tracing in general or miss important Lambda-specific configuration differences. Similarly, users who land on the main .NET Core docs might be deploying to Lambda but not realize they need different setup instructions. This PR creates better navigation between these related docs and sets proper expectations upfront.

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:

Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

[6/5/2025] Merge queue has been disabled on the documentation repo. If you have write access to the repo, the PR has been reviewed by a Documentation team member, and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #documentation channel in Slack.

Additional notes

This PR complements the decision tree changes in #32476 by ensuring users who reach the Lambda docs understand the prerequisites and can easily navigate back to general .NET tracing docs for conceptual understanding.

Copy link
Member Author

lucaspimentel commented Oct 29, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 29, 2025

Preview links (active after the build_preview check completes)

Modified Files

Copy link
Member Author

@lucaspimentel lucaspimentel left a comment

Choose a reason for hiding this comment

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

Claude Code Review: Reviewing for style guide compliance and documentation best practices.

@lucaspimentel lucaspimentel force-pushed the lpimentel/apm-dotnet-lambda-prerequisites branch from d2293ca to 1a4fdd1 Compare October 29, 2025 21:09
@lucaspimentel lucaspimentel force-pushed the lpimentel/apm-dotnet-decision-trees branch from 9a4474a to ac00373 Compare October 30, 2025 02:39
@lucaspimentel lucaspimentel changed the base branch from lpimentel/apm-dotnet-decision-trees to graphite-base/32477 October 30, 2025 02:54
@lucaspimentel lucaspimentel force-pushed the lpimentel/apm-dotnet-lambda-prerequisites branch from 1a4fdd1 to 0b1f029 Compare October 30, 2025 02:54
@lucaspimentel lucaspimentel changed the base branch from graphite-base/32477 to lpimentel/apm-dotnet-fix-windows-link October 30, 2025 02:54
@lucaspimentel lucaspimentel force-pushed the lpimentel/apm-dotnet-fix-windows-link branch from ad02060 to 5778445 Compare October 30, 2025 21:37
lucaspimentel and others added 2 commits November 3, 2025 12:29
Add a Prerequisites section to help users understand Lambda-specific
requirements before they begin setup. Add Configuration and
Troubleshooting sections with cross-references to main APM docs.

Key additions:
- Prerequisites section explaining Lambda-specific differences
- Links to .NET Core compatibility requirements
- Links to main .NET Core APM docs for background concepts
- Configuration section with Lambda-specific notes
- Troubleshooting section with diagnostic tool references

This improves navigation between Lambda-specific docs and foundational
APM documentation, helping users understand how Lambda differs from
traditional host-based setups.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Change "Before instrumenting" to "Before you begin" for direct language
- Clarify NuGet version must match Lambda layer version
- Remove "you're" contraction per style guide
- Replace "traditional host-based" with specific deployment types
- Clarify environment variable configuration location

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@lucaspimentel lucaspimentel force-pushed the lpimentel/apm-dotnet-fix-windows-link branch from 5778445 to e7475f2 Compare November 3, 2025 17:29
@lucaspimentel lucaspimentel force-pushed the lpimentel/apm-dotnet-lambda-prerequisites branch from 0b1f029 to 1881ff0 Compare November 3, 2025 17:29
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