Skip to content

chore(rfc-0014): sweep 12 'future PR-B' comments when WithMetrics bridge lands #380

Description

@trilamsr

Context

Post-wave-audit finding #7 (see docs/v1-rc1-post-wave-audit.md).

10 "future PR-B" comments in module/ + 2 in docs/ reference the RFC-0014 WithMetrics bridge that has not shipped. They are factually accurate today, but become stale the day PR-B lands.

Locations:

  • module/processor/patterndetectorprocessor/patterndetector.go:357,366,447,642
  • module/processor/patterndetectorprocessor/pcie_aer_test.go:47
  • module/processor/patterndetectorprocessor/thermal_throttle_test.go:252
  • module/pkg/patterns/thermal_throttle.go:42
  • module/pkg/patterns/cuda_oom.go:80
  • module/pkg/patterns/pcie_aer.go:70
  • docs/patterns/pattern-4-thermal-throttle.md:145
  • docs/integrations/prometheus-scrape.md:381

Fix

When RFC-0014 PR-B lands, sweep all 12 sites in the same PR (or as the first follow-up).

This issue is the tracker, not the fix.

Acceptance

  • All 12 comments either deleted or rewritten to reflect post-PR-B reality.
  • Closes with PR-B's merge commit.

Metadata

Metadata

Assignees

No one assigned

    Labels

    documentationImprovements or additions to documentationpost-wave-auditFiled by docs/v1-rc1-post-wave-audit.md (2026-06-01)

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions