Use a less confusing name

elastic · Aug 31, 2023 · 07f6c45 · 07f6c45
1 parent fe46878
commit 07f6c45
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/cmd/apmbench/config.go b/cmd/apmbench/config.go
@@ -19,11 +19,11 @@ var cfg struct {
 	Benchtime time.Duration
 	RunRE     *regexp.Regexp
 	// Sorted list of agents count to be used for benchmarking
-	AgentsList          []int
-	CollectorConfigYaml string
-	ServerMode          bool
-	APMServerURL        string
-	APMSecretToken      string
+	AgentsList               []int
+	CollectorConfigYaml      string
+	ServerMode               bool
+	MonitoringAPMServerURL   string
+	MonitoringAPMSecretToken string
 }
 
 func init() {
@@ -62,6 +62,6 @@ func init() {
 	)
 	flag.StringVar(&cfg.CollectorConfigYaml, "collector-config-yaml", "", "configuration for otel collector")
 	flag.BoolVar(&cfg.ServerMode, "server-mode", false, "continue running otel collector post benchmark run")
-	flag.StringVar(&cfg.APMServerURL, "apm-server-url", "", "monitoring APM server URL")
-	flag.StringVar(&cfg.APMSecretToken, "apm-secret-token", "", "monitoring APM secret token")
+	flag.StringVar(&cfg.MonitoringAPMServerURL, "monitoring-apm-server-url", "", "monitoring APM server URL")
+	flag.StringVar(&cfg.MonitoringAPMSecretToken, "monitoring-apm-secret-token", "", "monitoring APM secret token")
 }
diff --git a/cmd/apmbench/main.go b/cmd/apmbench/main.go
@@ -31,18 +31,18 @@ func main() {
 
 	// Create otel collector
 	collectorCfg := otelcollector.DefaultConfig()
-	if cfg.APMServerURL != "" {
-		apmURL, err := url.Parse(cfg.APMServerURL)
+	if cfg.MonitoringAPMServerURL != "" {
+		apmURL, err := url.Parse(cfg.MonitoringAPMServerURL)
 		if err != nil {
 			logger.Fatal("invalid apm-server-url specified")
 		}
 		collectorCfg.OTLPExporterEndpoint = apmURL.Host
 		if apmURL.Port() == "" {
 			collectorCfg.OTLPExporterEndpoint += ":443"
 		}
-		if cfg.APMSecretToken != "" {
+		if cfg.MonitoringAPMSecretToken != "" {
 			collectorCfg.OTLPExporterHeaders = map[string]string{
-				"Authorization": "Bearer " + cfg.APMSecretToken,
+				"Authorization": "Bearer " + cfg.MonitoringAPMSecretToken,
 			}
 		}
 	}