daeuniverse / dae

eBPF-based Linux high-performance transparent proxy solution.
GNU Affero General Public License v3.0
2.62k stars 165 forks source link

chore: Add a global config to enable pprof #532

Closed jschwinger233 closed 1 week ago

jschwinger233 commented 2 weeks ago

Background

This PR adds a new global config pprof_port to specify (and enable/disable) pprof server. Users can also enable or disable pprof on the fly using dae reload.

Checklist

Full Changelogs

Issue Reference

Closes #[issue number]

Test Result

mzz2017 commented 2 weeks ago

我之前测试的时候发现引入 http 包会带来比较大的二进制大小膨胀,可以麻烦测试一下吗?

jschwinger233 commented 1 week ago

@mzz2017 我对比了下 PR build: https://github.com/daeuniverse/dae/actions/runs/9441996291?pr=532

Arch With pprof Without pprof
arm64 11MB 10.7MB
x64 11.6MB 11.3MB

会增加 0.3MB 大小

dae-prow[bot] commented 1 week ago

❌ Your branch is currently out-of-sync to main. No worry, I will fix it for you.

dae-prow[bot] commented 1 week ago

❌ Your branch is currently out-of-sync to main. No worry, I will fix it for you.

mzz2017 commented 1 week ago

LGTM