Skip to content

Simplify dashboard monitor metrics payload and switch stats sampling to /stats?blocks=100 - #2

Draft
edtubbs with Copilot wants to merge 13 commits into
dashb0rdfrom
copilot/update-pup-chart-monitor
Draft

Simplify dashboard monitor metrics payload and switch stats sampling to /stats?blocks=100#2
edtubbs with Copilot wants to merge 13 commits into
dashb0rdfrom
copilot/update-pup-chart-monitor

Conversation

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown

This update aligns monitor output with current chart ingestion expectations: metric meta is no longer emitted, and stats are now sourced from the REST endpoint using a fixed 100-block window instead of the prior 24hr-style call.

  • Stats source window update

    • In dashboard/monitor/monitor.go, collectMetrics() now fetches:
      • "/stats?blocks=100"
    • Replaces the previous "/stats24" call path.
  • Payload schema simplification

    • Removed meta from submitted stats_* and smpv_* metric objects.
    • Deleted the now-unused sampleMeta(...) helper and its callsite.
    • Metric keys and value fields remain unchanged.
  • Repo hygiene

    • Added root/pkg/mod/cache/lock to .gitignore to avoid committing local Go module cache lock artifacts.
// before
statsStr, err := fetchEndpoint(fmt.Sprintf("/stats24"))
"stats_tps": map[string]interface{}{"value": mustParseFloat(stats["tps"]), "meta": tsmeta},

// after
statsStr, err := fetchEndpoint("/stats?blocks=100")
"stats_tps": map[string]interface{}{"value": mustParseFloat(stats["tps"])},

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits March 5, 2026 15:51
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Copilot AI changed the title [WIP] Update monitor for pup chart and remove txmetadata Align dashboard monitor payload with float chart metrics; remove obsolete tx metadata; bump manifest version Mar 5, 2026
Copilot AI and others added 3 commits March 5, 2026 19:06
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Copilot AI changed the title Align dashboard monitor payload with float chart metrics; remove obsolete tx metadata; bump manifest version Align dashboard monitor chart payload with float metrics and timestamp-only metadata Mar 5, 2026
Copilot AI and others added 3 commits March 5, 2026 19:10
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Copilot AI changed the title Align dashboard monitor chart payload with float metrics and timestamp-only metadata Stop monitor-side synthetic timestamps; emit timestamp metadata only from source stats Mar 5, 2026
Copilot AI and others added 3 commits March 5, 2026 19:18
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Copilot AI changed the title Stop monitor-side synthetic timestamps; emit timestamp metadata only from source stats Remove monitor metric meta payload from dashboard submission Mar 5, 2026
Co-authored-by: edtubbs <84785904+edtubbs@users.noreply.github.com>
Copilot AI changed the title Remove monitor metric meta payload from dashboard submission Simplify dashboard monitor metrics payload and switch stats sampling to /stats?blocks=100 Mar 5, 2026
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