Duplicate agent_submit spans in agent and lb_agent

[bharadwr]

There's two spans with the same name, namely in agent.go and lb_agent.go.

fn/api/agent/agent.go

Line 250 in 4ccd8fa

ctx, span := trace.StartSpan(ctx, "agent_submit")

fn/api/agent/lb_agent.go

Line 200 in 1fb78ed

ctx, span := trace.StartSpan(call.req.Context(), "agent_submit")

Should this be renamed? Someone might find a span on jaeger and go down the wrong rabbit hole if theyre debugging (I certainly did)

[rdallman]

yes, please - lb_agent likely has a few duplicates from agent, lb_agent_submit is probably sufficient?

[bharadwr]

Yep, I'll make that change. Lemme see if there's a few more duplicate traces, and I'll add them all to one ticket.

[bharadwr]

It's gonna look like this, added some more span attributes that are probably useful
Couldn't find any duplicate spans besides this