Skip to content

adding services dashboard discrepancy#2709

Open
revmag wants to merge 2 commits intomainfrom
services-dashboard
Open

adding services dashboard discrepancy#2709
revmag wants to merge 2 commits intomainfrom
services-dashboard

Conversation

@revmag
Copy link
Collaborator

@revmag revmag commented Feb 12, 2026

No description provided.

@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
signoz-web Ready Ready Preview, Comment Feb 12, 2026 11:06am

Request Review

@claude
Copy link
Contributor

claude bot commented Feb 12, 2026

Claude finished @revmag's task in 1m 37s —— View job


Documentation Review Complete ✅

I've reviewed this PR which adds a new FAQ entry about the discrepancy between custom rate queries and the Services dashboard.


Summary

Overall Assessment: The content is technically accurate and well-explained. The FAQ addresses a common user confusion about why custom queries don't match the Services dashboard values. I found only one minor formatting issue.

Documentation Review

✅ Strengths

  • Technical accuracy: The explanation correctly identifies that the Services dashboard filters to show only top-level operations, while direct queries to signoz_calls_total include all operations (nested/child operations)
  • Clear structure: Good use of Q&A format with collapsible detailed explanation
  • Helpful cross-references: Links to both the APM Metrics dashboard template and the APM metrics guide are appropriate and verified to be working
  • Good explanation flow: Progresses logically from problem → understanding → solution

Issues Found

Low Priority:

  • Missing newline at end of file (see inline comment on line 41)

Technical Verification

I verified the technical claims against SigNoz documentation:


Recommendation

Approve with minor fix: The content is solid and ready to merge after fixing the newline issue. This is a helpful addition to the troubleshooting FAQs.

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.

1 participant