The GitHub Actions job "npm_and_yarn in /nodejs for js-yaml - Update 
#1487654190" on rocketmq-clients.git/master has failed.
Run started by GitHub user dependabot[bot] (triggered by dependabot[bot]).

Head commit for run:
5939e374500b4b1572acd23979a61a5f6426f876 / guyinyou 
<[email protected]>
[Go] Fix resource leaks on client close (#1305)

1. ClientManager shutdown: call shutdown() instead of UnRegisterClient()
   in GracefulStop to stop background goroutines and close RPC connections
   (regression from #235 which made ClientManager per-client but forgot
   to add shutdown call).

2. Telemetry session cleanup: release all sessions in GracefulStop before
   shutting down the client manager to properly close gRPC streams.

3. release() nil check: guard against nil observer when telemetry
   connection failed during startup.

4. Per-client metrics isolation: use view.NewMeter() to give each client
   its own Meter, Measures, Views, and Exporter. On client close, the
   Meter is stopped and views are unregistered, preventing metrics
   cardinality explosion from accumulated client_id tag values.
   The client_id tag is preserved in exported metric data.

5. Fix meter lifecycle: call Start() before Register() to avoid deadlock,
   and register/unregister the exporter with the per-client Meter so
   metrics are actually exported.

Co-authored-by: guyinyou <[email protected]>

Report URL: https://github.com/apache/rocketmq-clients/actions/runs/30293494100

With regards,
GitHub Actions via GitBox

Reply via email to