Skip to content

Conversation

sadohert
Copy link
Contributor

@sadohert sadohert commented May 2, 2025

This started out small. I only wanted to weave in some pointers on setting up the Prometheus node-exporter tool to make OS/network metrics available in our Calls dashboard. I started into Claude Code... and ... well.. . it spiralled.

I couldn't get a local build to see what it all looked like. I read most of the text and it was okay, but I want to see how its formatted.

So hopefully this can be a draft here while I have a look.

sadohert and others added 3 commits February 2, 2024 13:24
I think they broke the build.
Sub pages were added. Claude seemed to expand on the documentation for
calls configuration, including deployment, metrics monitoring,
RTC setup, and troubleshooting.
@sadohert
Copy link
Contributor Author

sadohert commented May 2, 2025

Hi @cwarnermm - Is there a trick to make this draft PR build a lab setup for me to review?

@cwarnermm
Copy link
Member

Hi @sadohert! This PR is a draft currently; I suspect that's why a preview isn't being generated. If you're open to it, flip it to Ready for review and that should kick off a generated preview. Pretty sure you could move it back to draft afterward.

@sadohert sadohert marked this pull request as ready for review May 2, 2025 14:07
@cwarnermm
Copy link
Member

@sadohert - The generated preview is blocked until the merge conflicts in this PR are resolved.

@sadohert
Copy link
Contributor Author

sadohert commented May 7, 2025

Thanks... I think I had some old junk in my fork that messed thigns up. I'm going to close this PR and work through a clean one here - #7929

@sadohert sadohert closed this May 7, 2025
@sadohert sadohert deleted the stu-calls-claude-05-01 branch May 7, 2025 16:37
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