mirror of
https://github.com/kubernetes-sigs/descheduler.git
synced 2026-01-28 06:29:29 +01:00
profile: fix span attribute typo
Fix profile attribute "prpfile" typo in trace spans. Signed-off-by: Antoine Deschênes <antoine.deschenes@linux.com>
This commit is contained in:
@@ -304,7 +304,7 @@ func (d profileImpl) RunDeschedulePlugins(ctx context.Context, nodes []*v1.Node)
|
|||||||
errs := []error{}
|
errs := []error{}
|
||||||
for _, pl := range d.deschedulePlugins {
|
for _, pl := range d.deschedulePlugins {
|
||||||
var span trace.Span
|
var span trace.Span
|
||||||
ctx, span = tracing.Tracer().Start(ctx, pl.Name(), trace.WithAttributes(attribute.String("plugin", pl.Name()), attribute.String("prpfile", d.profileName), attribute.String("operation", tracing.DescheduleOperation)))
|
ctx, span = tracing.Tracer().Start(ctx, pl.Name(), trace.WithAttributes(attribute.String("plugin", pl.Name()), attribute.String("profile", d.profileName), attribute.String("operation", tracing.DescheduleOperation)))
|
||||||
defer span.End()
|
defer span.End()
|
||||||
evicted := d.podEvictor.TotalEvicted()
|
evicted := d.podEvictor.TotalEvicted()
|
||||||
// TODO: strategyName should be accessible from within the strategy using a framework
|
// TODO: strategyName should be accessible from within the strategy using a framework
|
||||||
@@ -337,7 +337,7 @@ func (d profileImpl) RunBalancePlugins(ctx context.Context, nodes []*v1.Node) *f
|
|||||||
errs := []error{}
|
errs := []error{}
|
||||||
for _, pl := range d.balancePlugins {
|
for _, pl := range d.balancePlugins {
|
||||||
var span trace.Span
|
var span trace.Span
|
||||||
ctx, span = tracing.Tracer().Start(ctx, pl.Name(), trace.WithAttributes(attribute.String("plugin", pl.Name()), attribute.String("prpfile", d.profileName), attribute.String("operation", tracing.BalanceOperation)))
|
ctx, span = tracing.Tracer().Start(ctx, pl.Name(), trace.WithAttributes(attribute.String("plugin", pl.Name()), attribute.String("profile", d.profileName), attribute.String("operation", tracing.BalanceOperation)))
|
||||||
defer span.End()
|
defer span.End()
|
||||||
evicted := d.podEvictor.TotalEvicted()
|
evicted := d.podEvictor.TotalEvicted()
|
||||||
// TODO: strategyName should be accessible from within the strategy using a framework
|
// TODO: strategyName should be accessible from within the strategy using a framework
|
||||||
|
|||||||
Reference in New Issue
Block a user