slurm and dcgm docs

[revmag]

1. Added SLURM docs
2. Added DCGM docs

[vercel]

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

Project	Deployment	Review	Updated (UTC)
signoz-web	Ready	Preview, Comment	Jan 22, 2026 5:27am

[claude]

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

---

Review Complete: SLURM and DCGM Documentation

I've reviewed the PR that adds documentation for SLURM and NVIDIA DCGM metrics monitoring. Both documents provide solid foundations, but there are several areas that need attention to meet the repository's documentation standards.

Summary of Changes

• ✅ New files: 2 new MDX documentation files added
  • nvidia-dcgm-metrics.mdx - NVIDIA GPU monitoring via DCGM Exporter
  • slurm-metrics.mdx - SLURM cluster monitoring via Prometheus exporter
• ✅ Navigation: Both docs properly added to docsSideNav.ts
• ✅ UI Integration: Links added to MetricsQuickStartOverview.tsx component
• ✅ Technical accuracy: Verified against official sources

Key Issues to Address

Both Documents:

1. Missing Overview sections explaining what DCGM/SLURM are
2. Missing Next Steps sections to guide users after setup
3. Pipeline configuration should clarify appending vs replacing

SLURM Documentation:
4. Port correction needed: Default port is 8080, not :8080 in placeholder
5. Prerequisites: Should list specific CLI commands used (sinfo, squeue, sdiag, sacct, sshare)
6. Installation steps missing: No instructions on how to install/deploy the exporter

DCGM Documentation:
7. Prerequisites: Should mention driver/toolkit versions or link to NVIDIA docs
8. Localhost caveat: Needs clarification for containerized environments
9. Command context: Docker command needs explanation before showing it

Docs PR Checklist Status

Based on CONTRIBUTING.md checklist (lines 562-581):

• ✅ Frontmatter complete (date, id, title, description, doc_type)
• ✅ SEO keywords present in title and description
• ⚠️ Commands need more context/explanation (see inline comments)
• ⚠️ Missing Overview sections (optional per guidelines, but recommended for clarity)
• ⚠️ Missing Next Steps sections (required per line 573)
• ✅ Both properly added to sidebar navigation
• ✅ Dashboard integration included
• ✅ Troubleshooting sections present
• ✅ Images would use WebP format (dashboard previews via DashboardActions component)
• ✅ External links properly formatted with target="_blank" and security attributes

Technical Verification

I verified the technical accuracy against official sources:

• DCGM Exporter: Confirmed port 9400, Docker command, and Kubernetes deployment via NVIDIA's dcgm-exporter repository
• SLURM Exporter: Confirmed port 8080 and CLI commands via vpenso's prometheus-slurm-exporter

Label Added

✅ Added add-to-onboarding label since these are new documentation files for sending metrics to SigNoz

Recommendation

The inline comments provide specific suggestions with code blocks. Please address these to align with the repository's documentation standards before merging.

---

[github-actions]

📚 Onboarding issue created… see #2565