Skip to content

ci: simplify DNS/SCI PR summaries and scope policy smart SCI#25

Merged
MaurUppi merged 3 commits intomainfrom
feat/policy-smart
Feb 25, 2026
Merged

ci: simplify DNS/SCI PR summaries and scope policy smart SCI#25
MaurUppi merged 3 commits intomainfrom
feat/policy-smart

Conversation

@MaurUppi
Copy link
Copy Markdown
Owner

Summary

  • DNS Benchmark Compare: keep full artifact report, but PR bot comment now posts concise summary only (before/after + suite status + overall)
  • Policy Smart SCI: restrict execution to policy-smart head branches on pull requests
  • Policy Smart SCI: keep/update a single PR comment with concise summary (scope + overall + per-job status)

Why

  • Current benchmark bot comment is too verbose for review
  • SCI workflow should be scoped to policy-smart branch verification only

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 25, 2026

Policy Smart SCI Summary

Job Status

  • ✅ SCI-1 Strategy Correctness: success
  • ✅ SCI-2 TCP Retry Semantics: success
  • ✅ SCI-3 Concurrency Race: success

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 25, 2026

DNS Benchmark Compare Summary

  • Before (base): origin/main
  • After (head): 517760075c9173db046d6abac9ffe26273347962
  • Base strategy: merge-base
  • Suite profile: dns-module
  • Overall: pass (5/5 passed)
  • Workflow run: https://github.com/MaurUppi/dae/actions/runs/22400806967
  • Full report artifact: dns-benchmark-compare-22400806967

Suite Status

  • component_upstream_hotpath: pass
  • control_cache_hotpath: pass
  • control_cache_structures: pass
  • control_core_flow: pass
  • control_singleflight_scale: pass

Suite Delta (time/op geomean)

  • component_upstream_hotpath: 6.370n -> 6.059n-4.89%)
  • control_cache_hotpath: 12.86n -> 12.92n+0.45%)
  • control_cache_structures: 43.42n -> 43.45n+0.08%)
  • control_core_flow: 241.4n -> 241.7n+0.13%)
  • control_singleflight_scale: 113.5µ -> 113.4µ-0.09%)

@MaurUppi MaurUppi merged commit 280a60c into main Feb 25, 2026
5 checks passed
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.

1 participant