Skip to content

docs: define app performance metrics for monitoring collection (issue #518)#519

Draft
Chulhee1Lee with Copilot wants to merge 4 commits into
mainfrom
copilot/define-app-performance-metrics
Draft

docs: define app performance metrics for monitoring collection (issue #518)#519
Chulhee1Lee with Copilot wants to merge 4 commits into
mainfrom
copilot/define-app-performance-metrics

Conversation

Copilot AI commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

MonitoringServer already delivers node-level metrics to PolicyManager via ReportNodeMetrics gRPC, but app-level metrics (StressMonitoringMetric: FPS, latency, per-core CPU load) have no forwarding path to PolicyManager — confirming the core gap identified in #518.

Changes

Both documents are fully bilingual, with each section written in Korean and English side by side.

  • doc/architecture/EN/HLD/app_performance_metrics.md — Bilingual design document (English primary, Korean alongside) defining:

    • Candidate metric research: 16 candidate metrics evaluated beyond the representative examples in the issue (FPS, Latency, Per-core CPU Load, GPU Utilization, Frame Drop Count, Latency Jitter, Throughput, Error/Fault Rate, Temperature, Power Consumption, Context Switch Rate, Real-time Deadline Violation, and node-level metrics)
    • Feasibility and collection complexity assessment for each candidate
    • Current state: node-level metrics path (✅ exists), app-level metrics path to PolicyManager (❌ missing)
    • Confirmed metric list with units, collection intervals, and scopes (per-Node vs per-App), selected by High feasibility rating
    • Gap analysis table and next-step tasks (collection method decision → interface design → E2E verification)
  • doc/architecture/KR/HLD/app_performance_metrics.md — Bilingual counterpart (Korean primary, English alongside) with identical content

Key findings

Path Status
Node metrics → MonitoringServer → PolicyManager ✅ Implemented (ReportNodeMetrics gRPC + threshold-based offloading)
App metrics → MonitoringServer → etcd ✅ Implemented (SendStressMonitoringMetric gRPC)
App metrics → PolicyManager Missing

App-level metrics confirmed for collection (High feasibility):

Metric Unit Collection Scope
FPS frames/s Push (on-demand) Per App
Latency ms Push (on-demand) Per App
Per-core CPU load % Push (on-demand) Per App
Frame Drop Count count/interval Push (on-demand) Per App
Latency Jitter ms Push (on-demand) Per App

Metrics with Medium or Low feasibility (GPU Utilization, Throughput, Error Rate, Temperature, Power Consumption, Context Switch Rate, Real-time Deadline Violation) are deferred to future iterations.

Copilot AI changed the title [WIP] Design app performance metrics for monitoring collection docs: define app performance metrics for monitoring collection (issue #518) Jul 8, 2026
Copilot AI requested a review from Chulhee1Lee July 8, 2026 11:03
@Chulhee1Lee

Copy link
Copy Markdown
Contributor

@copilot 연결된 Issue에서 언급한 모든 항목이 다 커버되고 있는지 확인해줘.

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot 연결된 Issue에서 언급한 모든 항목이 다 커버되고 있는지 확인해줘.

이슈 #518의 모든 항목을 다시 점검하고, 누락된 부분을 추가하였습니다 (커밋: 최신 커밋).

이슈 Tasks 대비 커버리지:

이슈 Task 이전 상태 현재 상태
후보 메트릭 유형 조사 (이슈에서 "폭넓게 검토" 요구) ❌ 기존 구현 메트릭만 기술 ✅ 16개 후보 메트릭 조사 표 추가 (FPS/Latency/CPU 외 GPU 활용률, 처리량, 오류율, 온도, 전력 소비량, 컨텍스트 스위치, 데드라인 위반 등 포함)
각 후보 메트릭의 활용 가능성 및 수집 난이도 검토 ❌ 미포함 ✅ 후보 표에 Feasibility/Collection Complexity 컬럼 추가
수집 대상 메트릭 최종 확정 ✅ (높음 등급 기준으로 확정 명시, Frame Drop Count / Latency Jitter 추가)
각 메트릭의 단위/수집 주기/수집 범위 정의
문서화

Acceptance Criteria 모두 충족:

  • 수집 대상 메트릭 목록 확정 및 문서화 ✅
  • 단위, 수집 주기, 수집 범위 정의 ✅
  • 수집 주체 결정 Task 진행 가능 상태 ✅ (Next Steps 섹션 참조)

@Chulhee1Lee

Copy link
Copy Markdown
Contributor

@copilot Architecture 및 Document는 항상 한글과 영문을 병기해줘. 업데이트하도록.

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Architecture 및 Document는 항상 한글과 영문을 병기해줘. 업데이트하도록.

두 문서(doc/architecture/EN/HLD/app_performance_metrics.md, doc/architecture/KR/HLD/app_performance_metrics.md) 모두 각 섹션에 한글과 영문을 병기하도록 업데이트했습니다. EN 문서는 영문을 주 언어로 하고 한국어를 병기하며, KR 문서는 한국어를 주 언어로 하고 영문을 병기합니다.

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.

[Design] Define App Performance Metrics for Monitoring Collection

2 participants