Skip to content

PLTF-2905: Expand Laminar observability and config guidance for OHE#579

Merged
juanmichelini merged 6 commits into
mainfrom
pltf-2905-laminar-observability-guide
Jun 23, 2026
Merged

PLTF-2905: Expand Laminar observability and config guidance for OHE#579
juanmichelini merged 6 commits into
mainfrom
pltf-2905-laminar-observability-guide

Conversation

@juanmichelini

@juanmichelini juanmichelini commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • merge the strongest Laminar guidance from the SDK observability and OHE analytics docs
  • make the SDK guide the canonical tracing reference with official Laminar links
  • keep the OHE analytics page focused on deployment, forwarded LMNR_/LLM_ variables, and post-setup Laminar workflows
  • document how to configure Laminar endpoint settings and choose LLM provider/model sizes through an OpenAI-compatible proxy

Testing

  • git diff --check

This PR description was created by an AI agent (OpenHands) on behalf of the user.

Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
@juanmichelini juanmichelini changed the title PLTF-2905: Expand Laminar observability guidance for OHE PLTF-2905: Expand Laminar observability and config guidance for OHE Jun 19, 2026
@enyst

enyst commented Jun 21, 2026

Copy link
Copy Markdown
Member

@juanmichelini Just a quick run to see if the agent sees something. Please feel free to not consider it if it has its moment of exaggerations 😅

@OpenHands Take a look at this PR and verify Laminar docs and clone yourself agent-sdk and openhands repos from the same org if you need. Review it. Post your feedback as a gh review with gh api.

@openhands-ai

openhands-ai Bot commented Jun 21, 2026

Copy link
Copy Markdown

I'm on it! enyst can track my progress at all-hands.dev

@enyst enyst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed the docs and cross-checked the Laminar env vars against OpenHands/software-agent-sdk (LMNR_PROJECT_API_KEY, LMNR_BASE_URL, LMNR_FORCE_HTTP) and OpenHands env forwarding (AUTO_FORWARD_PREFIXES). The Laminar doc URLs resolve and git diff --check passes. I left one inline concern about the Enterprise endpoint example using a concrete all-hands.dev host in a guide that otherwise uses <your-base-domain>.

This review was created by an AI agent (OpenHands) on behalf of the user.

Comment thread enterprise/analytics.mdx Outdated
@openhands-ai

This comment was marked as duplicate.

Updated base URLs in configuration examples to use placeholders for user-defined domains.
Co-authored-by: openhands <openhands@all-hands.dev>

@rajshah4 rajshah4 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great work pull this all together, looks good to me

@juanmichelini juanmichelini merged commit bd803ca into main Jun 23, 2026
5 checks passed
@juanmichelini juanmichelini deleted the pltf-2905-laminar-observability-guide branch June 23, 2026 00:12
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.

4 participants