chore(supervisor): add supervisor grafana dashboard to kurtosis

[dhyaniarun1993]

optimism-package grafana expects the dashboard json inside the dashboards folder in the provided sources

[claude]

Claude finished @dhyaniarun1993's task —— View job

---

PR Review: Supervisor Grafana Dashboard Integration

Reviewing the changes to add supervisor Grafana dashboard to Kurtosis configuration.

Todo List:

• Read and analyze all 4 changed files
• Review grafana dashboard configuration files
• Analyze devnet YAML modifications
• Check for security, performance, and quality issues
• Provide comprehensive review feedback

Review Summary

This PR adds Grafana dashboard integration for the Kona supervisor component by:

1. Renaming/moving dashboard files from dashboard/ to dashboards/ directory (correcting the path structure)
2. Adding observability configuration to devnet YAML files to include the dashboard sources

Detailed Analysis

✅ Positive Changes

1. Correct Directory Structure: The rename from dashboard/ to dashboards/ follows Grafana's standard directory naming conventions for dashboard provisioning.

2. Comprehensive Dashboard: The kona-supervisor.json contains a well-structured Grafana dashboard with:

   • System metrics (version, build info, resource usage)
   • Supervisor-specific metrics (reorgs, block processing, RPC performance)
   • Storage performance metrics
   • Proper Prometheus datasource configuration

3. Configuration Consistency: Both devnet configurations (preinterop-supervisor.yaml and simple-supervisor.yaml) are updated consistently.

4. Valid YAML Structure: The dashboard provisioning configuration in dashboard.yml follows correct Grafana API v1 format.

📝 Minor Observations

1. No Breaking Changes: The file renames maintain the same content, ensuring no functional changes to the dashboard itself.

2. GitHub URL Reference: The dashboard source uses github.com/op-rs/kona/docker/recipes/kona-supervisor/grafana which correctly points to the new directory structure.

🔍 Security & Performance

• No Security Concerns: Configuration files contain no sensitive data or credentials
• Performance Impact: Minimal - only adds dashboard provisioning to existing Grafana instances
• Resource Usage: Dashboard queries use standard Prometheus metrics without expensive operations

🧪 Test Coverage Considerations

The changes primarily affect configuration files and don't introduce new code logic. However, consider:

• Testing that the dashboard loads correctly in the Kurtosis environment
• Verifying that all metrics referenced in the dashboard are actually exposed by the supervisor

Recommendations

1. ✅ LGTM: The changes are well-structured and follow best practices
2. Consider: Adding a brief description to the PR explaining the directory structure correction
3. Future Enhancement: Consider adding dashboard screenshots or documentation for the metrics being tracked

Final Assessment

Code Quality: ✅ Excellent
Security: ✅ No concerns
Performance: ✅ No impact
Best Practices: ✅ Follows Grafana conventions

This is a clean, straightforward configuration change that properly integrates supervisor monitoring dashboards into the Kurtosis testing environment.

---

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.9%. Comparing base (0ff8bf9) to head (98b5075).
⚠️ Report is 1 commits behind head on main.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Copilot]

Pull Request Overview

This PR adds Grafana dashboard configuration for the Kona supervisor to Kurtosis devnet configurations. The change enables observability by configuring the optimism-package to include supervisor-specific Grafana dashboards from the kona repository.

• Adds observability configuration with Grafana dashboard sources to devnet YAML files
• Points to the kona-supervisor Grafana dashboard located in the op-rs/kona repository

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File	Description
tests/devnets/simple-supervisor.yaml	Adds observability configuration with Grafana dashboard source for supervisor monitoring
tests/devnets/preinterop-supervisor.yaml	Adds observability configuration with Grafana dashboard source for supervisor monitoring

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[claude]

Claude finished @dhyaniarun1993's task —— View job

---

PR Review: Supervisor Grafana Dashboard Integration

Reviewing the changes to add supervisor Grafana dashboard to Kurtosis configuration.

Todo List:

• Read and analyze all 4 changed files
• Review grafana dashboard configuration files
• Analyze devnet YAML modifications
• Check for security, performance, and quality issues
• Provide comprehensive review feedback

Review Summary

This PR adds Grafana dashboard integration for the Kona supervisor component by:

1. Renaming/moving dashboard files from dashboard/ to dashboards/ directory (correcting the path structure)
2. Adding observability configuration to devnet YAML files to include the dashboard sources

Detailed Analysis

✅ Positive Changes

1. Correct Directory Structure: The rename from dashboard/ to dashboards/ follows Grafana's standard directory naming conventions for dashboard provisioning.

2. Comprehensive Dashboard: The kona-supervisor.json contains a well-structured Grafana dashboard with:

   • System metrics (version, build info, resource usage)
   • Supervisor-specific metrics (reorgs, block processing, RPC performance)
   • Storage performance metrics
   • Proper Prometheus datasource configuration

3. Configuration Consistency: Both devnet configurations (preinterop-supervisor.yaml and simple-supervisor.yaml) are updated consistently.

4. Valid YAML Structure: The dashboard provisioning configuration in dashboard.yml follows correct Grafana API v1 format.

📝 Minor Observations

1. No Breaking Changes: The file renames maintain the same content, ensuring no functional changes to the dashboard itself.

2. GitHub URL Reference: The dashboard source uses github.com/op-rs/kona/docker/recipes/kona-supervisor/grafana which correctly points to the new directory structure.

🔍 Security & Performance

• No Security Concerns: Configuration files contain no sensitive data or credentials
• Performance Impact: Minimal - only adds dashboard provisioning to existing Grafana instances
• Resource Usage: Dashboard queries use standard Prometheus metrics without expensive operations

🧪 Test Coverage Considerations

The changes primarily affect configuration files and don't introduce new code logic. However, consider:

• Testing that the dashboard loads correctly in the Kurtosis environment
• Verifying that all metrics referenced in the dashboard are actually exposed by the supervisor

Recommendations

1. ✅ LGTM: The changes are well-structured and follow best practices
2. Consider: Adding a brief description to the PR explaining the directory structure correction
3. Future Enhancement: Consider adding dashboard screenshots or documentation for the metrics being tracked

Final Assessment

Code Quality: ✅ Excellent
Security: ✅ No concerns
Performance: ✅ No impact
Best Practices: ✅ Follows Grafana conventions

This is a clean, straightforward configuration change that properly integrates supervisor monitoring dashboards into the Kurtosis testing environment.

---