Remove DD_HOSTNAME from agent and move it back to check runner (#139)

DataDog · Aug 27, 2020 · 3b8bf72 · 3b8bf72
1 parent e4f7a0a
commit 3b8bf72
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 18 deletions.
diff --git a/Makefile b/Makefile
@@ -42,7 +42,7 @@ build-plugin: ${ARTIFACT_PLUGIN}
 ${ARTIFACT_PLUGIN}: ${SOURCES}
 	CGO_ENABLED=0 go build -i -installsuffix cgo ${LDFLAGS} -o ${ARTIFACT_PLUGIN} ./cmd/${ARTIFACT_PLUGIN}/main.go
 
-container:
+container: bin/operator-sdk
 	./bin/operator-sdk build $(PREFIX):$(TAG)
     ifeq ($(KINDPUSH), true)
 	 kind load docker-image --name $(KIND_CLUSTER_NAME) $(PREFIX):$(TAG)
@@ -54,7 +54,7 @@ container-ci:
 test:
 	./go.test.sh
 
-e2e:
+e2e: bin/operator-sdk
 	kubectl apply -f deploy/datadoghq_v1alpha1_extendeddaemonset_crd.yaml
 	./bin/operator-sdk test local ./test/e2e --go-test-flags '-v' --image $(PREFIX):$(TAG)
 

diff --git a/pkg/controller/datadogagent/agent_test.go b/pkg/controller/datadogagent/agent_test.go
@@ -236,14 +236,6 @@ func defaultEnvVars() []corev1.EnvVar {
 				},
 			},
 		},
-		{
-			Name: datadoghqv1alpha1.DDHostname,
-			ValueFrom: &corev1.EnvVarSource{
-				FieldRef: &corev1.ObjectFieldSelector{
-					FieldPath: FieldPathSpecNodeName,
-				},
-			},
-		},
 		{
 			Name:  "KUBERNETES",
 			Value: "yes",

diff --git a/pkg/controller/datadogagent/clusterchecksrunner.go b/pkg/controller/datadogagent/clusterchecksrunner.go
@@ -373,6 +373,14 @@ func getEnvVarsForClusterChecksRunner(dda *datadoghqv1alpha1.DatadogAgent) []cor
 				},
 			},
 		},
+		{
+			Name: datadoghqv1alpha1.DDHostname,
+			ValueFrom: &corev1.EnvVarSource{
+				FieldRef: &corev1.ObjectFieldSelector{
+					FieldPath: FieldPathSpecNodeName,
+				},
+			},
+		},
 	}
 
 	if spec.ClusterChecksRunner.Config.LogLevel != nil {

diff --git a/pkg/controller/datadogagent/clusterchecksrunner_test.go b/pkg/controller/datadogagent/clusterchecksrunner_test.go
@@ -197,6 +197,14 @@ func clusterChecksRunnerDefaultEnvVars() []corev1.EnvVar {
 				},
 			},
 		},
+		{
+			Name: datadoghqv1alpha1.DDHostname,
+			ValueFrom: &corev1.EnvVarSource{
+				FieldRef: &corev1.ObjectFieldSelector{
+					FieldPath: FieldPathSpecNodeName,
+				},
+			},
+		},
 	}
 }
 

diff --git a/pkg/controller/datadogagent/utils.go b/pkg/controller/datadogagent/utils.go
@@ -443,14 +443,6 @@ func getEnvVarsCommon(dda *datadoghqv1alpha1.DatadogAgent, needAPIKey bool) ([]c
 				},
 			},
 		},
-		{
-			Name: datadoghqv1alpha1.DDHostname,
-			ValueFrom: &corev1.EnvVarSource{
-				FieldRef: &corev1.ObjectFieldSelector{
-					FieldPath: FieldPathSpecNodeName,
-				},
-			},
-		},
 		{
 			Name:  datadoghqv1alpha1.KubernetesEnvvarName,
 			Value: "yes",