Plugins: Refactoring: Move plugins tracing env vars to separate func (#66170)

pkg/plugins/manager/loader/initializer/initializer.go

@@ -67,27 +67,32 @@ func (i *Initializer) envVars(plugin *plugins.Plugin) []string {
 	hostEnv = append(hostEnv, i.awsEnvVars()...)
 	hostEnv = append(hostEnv, i.secureSocksProxyEnvVars()...)
 	hostEnv = append(hostEnv, azsettings.WriteToEnvStr(i.cfg.Azure)...)
+	hostEnv = append(hostEnv, i.tracingEnvVars(plugin)...)
 
-	// Tracing
+	ev := getPluginSettings(plugin.ID, i.cfg).asEnvVar("GF_PLUGIN", hostEnv)
+	return ev
+}
+
+func (i *Initializer) tracingEnvVars(plugin *plugins.Plugin) []string {
 	var pluginTracingEnabled bool
 	if v, exists := i.cfg.PluginSettings[plugin.ID]["tracing"]; exists {
 		pluginTracingEnabled = v == "true"
 	}
-	if i.cfg.Tracing.IsEnabled() && pluginTracingEnabled {
+	if !i.cfg.Tracing.IsEnabled() || !pluginTracingEnabled {
+		return nil
+	}
+
+	var vars []string
 	if plugin.Info.Version != "" {
-			hostEnv = append(hostEnv, fmt.Sprintf("GF_PLUGIN_VERSION=%s", plugin.Info.Version))
+		vars = append(vars, fmt.Sprintf("GF_PLUGIN_VERSION=%s", plugin.Info.Version))
 	}
-		hostEnv = append(
-			hostEnv,
+	return append(
+		vars,
 		fmt.Sprintf("GF_INSTANCE_OTLP_ADDRESS=%s", i.cfg.Tracing.OpenTelemetry.Address),
 		fmt.Sprintf("GF_INSTANCE_OTLP_PROPAGATION=%s", i.cfg.Tracing.OpenTelemetry.Propagation),
 	)
 }
 
-	ev := getPluginSettings(plugin.ID, i.cfg).asEnvVar("GF_PLUGIN", hostEnv)
-	return ev
-}
-
 func (i *Initializer) awsEnvVars() []string {
 	var variables []string
 	if i.cfg.AWSAssumeRoleEnabled {