Merge pull request #6261 from underarmour/feature/add-datadog-environment-option

Add datadog environment as a configuration option

docs/user-guide/nginx-configuration/configmap.md

@@ -140,8 +140,9 @@ The following table shows a configuration option's name, type, and the default v
 |[jaeger-trace-baggage-header-prefix](#jaeger-trace-baggage-header-prefix)|string|uberctx-|
 |[datadog-collector-host](#datadog-collector-host)|string|""|
 |[datadog-collector-port](#datadog-collector-port)|int|8126|
-|[datadog-service-name](#datadog-service-name)|service|"nginx"|
+|[datadog-service-name](#datadog-service-name)|string|"nginx"|
-|[datadog-operation-name-override](#datadog-operation-name-override)|service|"nginx.handle"|
+|[datadog-environment](#datadog-environment)|string|"prod"|
+|[datadog-operation-name-override](#datadog-operation-name-override)|string|"nginx.handle"|
 |[datadog-priority-sampling](#datadog-priority-sampling)|bool|"true"|
 |[datadog-sample-rate](#datadog-sample-rate)|float|1.0|
 |[main-snippet](#main-snippet)|string|""|
@@ -861,6 +862,10 @@ Specifies the port to use when uploading traces. _**default:**_ 8126
 
 Specifies the service name to use for any traces created. _**default:**_ nginx
 
+## datadog-environment
+
+Specifies the environment this trace belongs to. _**default:**_ prod
+
 ## datadog-operation-name-override
 
 Overrides the operation naem to use for any traces crated. _**default:**_ nginx.handle

docs/user-guide/third-party-addons/opentracing.md

@@ -92,6 +92,9 @@ datadog-collector-port
 # specifies the service name to use for any traces created, Default: nginx
 datadog-service-name
 
+# specifies the environment this trace belongs to, Default: prod
+datadog-environment
+
 # specifies the operation name to use for any traces collected, Default: nginx.handle
 datadog-operation-name-override
 

internal/ingress/controller/config/config.go

@@ -584,6 +584,10 @@ type Configuration struct {
 	// Default: 8126
 	DatadogCollectorPort int `json:"datadog-collector-port"`
 
+	// DatadogEnvironment specifies the environment this trace belongs to.
+	// Default: prod
+	DatadogEnvironment string `json:"datadog-environment"`
+
 	// DatadogServiceName specifies the service name to use for any traces created
 	// Default: nginx
 	DatadogServiceName string `json:"datadog-service-name"`
@@ -823,6 +827,7 @@ func NewDefault() Configuration {
 		JaegerSamplerPort:            5778,
 		JaegerSamplerHost:            "http://127.0.0.1",
 		DatadogServiceName:           "nginx",
+		DatadogEnvironment:           "prod",
 		DatadogCollectorPort:         8126,
 		DatadogOperationNameOverride: "nginx.handle",
 		DatadogSampleRate:            1.0,

internal/ingress/controller/nginx.go

@@ -1046,6 +1046,7 @@ const datadogTmpl = `{
   "service": "{{ .DatadogServiceName }}",
   "agent_host": "{{ .DatadogCollectorHost }}",
   "agent_port": {{ .DatadogCollectorPort }},
+  "environment": "{{ .DatadogEnvironment }}",
   "operation_name_override": "{{ .DatadogOperationNameOverride }}",
   "sample_rate": {{ .DatadogSampleRate }},
   "dd.priority.sampling": {{ .DatadogPrioritySampling }}