mosn / holmes

self-aware Golang profile dumper
Apache License 2.0
1.06k stars 135 forks source link

EnableDump: current cpu percent [1] greater than the CPUMaxPercent[90] #139

Closed z760087139 closed 1 year ago

z760087139 commented 1 year ago

I get "[INFO] [Holmes] unable to dump: current cpu percent [1] is greater than the CPUMaxPercent[90]" in dump log

this is my holmes service coding like:

        ctx, _ := context.WithTimeout(context.Background(), 10 * time.Hour)
    h, err := New(
        WithTextDump(),
        WithCollectInterval("5s"),
        WithCGroup(true),
        WithCPUDump(20,50,70,5*time.Minute),
        WithCPUMax(90),
        WithMemDump(20,50,70,5*time.Minute),
        WithGoroutineDump(6000, 50, 8000, 10 * 1000, 5*time.Minute),
        )
    if err != nil {
        return 
    }
    h = h.EnableCPUDump().EnableMemDump().EnableGoroutineDump()
    h.Start()
    go func(){
        <-ctx.Done()
        h.Stop()
    }()
Jun10ng commented 1 year ago

We fixed this issue on 08/22. Here are PR, and we release it in v1.2 tag.

Could you provide the holmes version in your application and check it?

z760087139 commented 1 year ago

We fixed this issue on 08/22. Here are PR, and we release it in v1.2 tag.

Could you provide the holmes version in your application and check it?

thx, im using holmes v1.1.0