Skip to content

Commit f59eb7f

Browse files
committed
debug: add log for debug discovery
1 parent e95eee3 commit f59eb7f

3 files changed

Lines changed: 10 additions & 0 deletions

File tree

client/middlewares.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ func newResolveMWBuilder(lbf *lbcache.BalancerFactory) endpoint.MiddlewareBuilde
9595
if remote.GetInstance() != nil {
9696
return next(ctx, request, response)
9797
}
98+
klog.CtxErrorf(ctx, "KITEX-DEBUG: start get balancer")
9899
lb, err := lbf.Get(ctx, dest)
99100
if err != nil {
100101
return kerrors.ErrServiceDiscovery.WithCause(err)
@@ -108,10 +109,13 @@ func newResolveMWBuilder(lbf *lbcache.BalancerFactory) endpoint.MiddlewareBuilde
108109
default:
109110
}
110111

112+
klog.CtxErrorf(ctx, "KITEX-DEBUG: start get picker")
111113
// we always need to get a new picker every time, because when downstream update deployment,
112114
// we may get an old picker that include all outdated instances which will cause connect always failed.
113115
picker := lb.GetPicker()
116+
klog.CtxErrorf(ctx, "KITEX-DEBUG: start get instance")
114117
ins := picker.Next(ctx, request)
118+
klog.CtxErrorf(ctx, "KITEX-DEBUG: end get instance")
115119
if ins == nil {
116120
err = kerrors.ErrNoMoreInstance.WithCause(fmt.Errorf("last error: %w", lastErr))
117121
} else {

pkg/loadbalance/lbcache/cache.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,9 @@ func (b *BalancerFactory) Get(ctx context.Context, target rpcinfo.EndpointInfo)
163163
return val.(*Balancer), nil
164164
}
165165
val, err, _ := b.sfg.Do(desc, func() (interface{}, error) {
166+
klog.CtxErrorf(ctx, "KITEX-DEBUG: start Resolve")
166167
res, err := b.resolver.Resolve(ctx, desc)
168+
klog.CtxErrorf(ctx, "KITEX-DEBUG: end Resolve")
167169
if err != nil {
168170
return nil, err
169171
}
@@ -173,7 +175,9 @@ func (b *BalancerFactory) Get(ctx context.Context, target rpcinfo.EndpointInfo)
173175
target: desc,
174176
}
175177
bl.res.Store(res)
178+
klog.CtxErrorf(ctx, "KITEX-DEBUG: start getSharedTicker")
176179
bl.sharedTicker = getSharedTicker(bl, b.opts.RefreshInterval)
180+
klog.CtxErrorf(ctx, "KITEX-DEBUG: end getSharedTicker")
177181
b.cache.Store(desc, bl)
178182
return bl, nil
179183
})

pkg/loadbalance/weighted_balancer.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,9 @@ func (wb *weightedBalancer) GetPicker(e discovery.Result) Picker {
6363
picker, ok := wb.pickerCache.Load(e.CacheKey)
6464
if !ok {
6565
picker, _, _ = wb.sfg.Do(e.CacheKey, func() (interface{}, error) {
66+
klog.Error("KITEX-DEBUG: start createPicker")
6667
p := wb.createPicker(e)
68+
klog.Error("KITEX-DEBUG: end createPicker")
6769
wb.pickerCache.Store(e.CacheKey, p)
6870
return p, nil
6971
})

0 commit comments

Comments
 (0)