chore: refactor (#3295)

core/trace/agent.go

@@ -33,7 +33,6 @@ var (
 
 // StartAgent starts an opentelemetry agent.
 func StartAgent(c Config) {
-
 	if c.Disabled {
 		return
 	}

core/trace/agent_test.go

@@ -73,6 +73,7 @@ func TestStartAgent(t *testing.T) {
 	StartAgent(c6)
 	StartAgent(c7)
 	StartAgent(c8)
+	defer StopAgent()
 
 	lock.Lock()
 	defer lock.Unlock()

core/trace/config.go

@@ -17,7 +17,6 @@ type Config struct {
 	// For example
 	// /v1/traces
 	OtlpHttpPath string `json:",optional"`
-
-	// Disabled indicates whether StartAgent should be called when starting the server.
-	Disabled bool
+	// Disabled indicates whether StartAgent starts the agent.
+	Disabled bool `json:",optional"`
 }