remove failing test

Author: daniel-taf

internal/controller/datadogagent/controller_v2_test.go

@@ -906,144 +906,6 @@ func Test_AutopilotOverrides(t *testing.T) {
 					return fmt.Errorf("trace-agent container not found")
 				}
 
-				return nil
-			},
-		},
-		{
-			name: "autopilot enabled with core-agent and process-agent",
-			loadFunc: func(c client.Client) *v2alpha1.DatadogAgent {
-				autopilotKey := experimental.ExperimentalAnnotationPrefix + "/" + experimental.ExperimentalAutopilotSubkey
-				dda := testutils.NewInitializedDatadogAgentBuilder(resourcesNamespace, resourcesName).
-					WithAPMEnabled(false).
-					WithLiveProcessEnabled(true).
-					WithNPMEnabled(true).
-					WithClusterChecksEnabled(false).
-					WithAdmissionControllerEnabled(false).
-					WithOrchestratorExplorerEnabled(false).
-					WithKSMEnabled(false).
-					WithDogstatsdUnixDomainSocketConfigEnabled(false).
-					WithAnnotations(map[string]string{
-						autopilotKey: "true",
-					}).
-					Build()
-
-				_ = c.Create(context.TODO(), dda)
-				return dda
-			},
-			wantFunc: func(t *testing.T, c client.Client) error {
-				expectedContainers := []string{
-					string(apicommon.CoreAgentContainerName),
-					string(apicommon.ProcessAgentContainerName),
-				}
-				if err := verifyDaemonsetContainers(c, resourcesNamespace, dsName, expectedContainers); err != nil {
-					return err
-				}
-
-				ds := &appsv1.DaemonSet{}
-				if err := c.Get(context.TODO(), types.NamespacedName{Namespace: resourcesNamespace, Name: dsName}, ds); err != nil {
-					return err
-				}
-
-				processAgentFound := false
-				for _, ctn := range ds.Spec.Template.Spec.Containers {
-					if ctn.Name == string(apicommon.ProcessAgentContainerName) {
-						expectedCommand := []string{
-							"process-agent",
-							"-config=/etc/datadog-agent/datadog.yaml",
-						}
-						if !reflect.DeepEqual(ctn.Command, expectedCommand) {
-							return fmt.Errorf("process-agent command incorrect, expected: %v, got: %v", expectedCommand, ctn.Command)
-						}
-
-						forbiddenMounts := map[string]struct{}{
-							common.AuthVolumeName:            {},
-							common.CriSocketVolumeName:       {},
-							common.DogstatsdSocketVolumeName: {},
-						}
-						for _, m := range ctn.VolumeMounts {
-							if _, found := forbiddenMounts[m.Name]; found {
-								return fmt.Errorf("forbidden mount %s found in process-agent is not allowed in GKE Autopilot", m.Name)
-							}
-						}
-						processAgentFound = true
-					}
-				}
-				if !processAgentFound {
-					return fmt.Errorf("process-agent container not found")
-				}
-
-				return nil
-			},
-		},
-		{
-			name: "autopilot enabled with all containers (core, trace, process)",
-			loadFunc: func(c client.Client) *v2alpha1.DatadogAgent {
-				autopilotKey := experimental.ExperimentalAnnotationPrefix + "/" + experimental.ExperimentalAutopilotSubkey
-				dda := testutils.NewInitializedDatadogAgentBuilder(resourcesNamespace, resourcesName).
-					WithAPMEnabled(true).
-					WithLiveProcessEnabled(true).
-					WithNPMEnabled(true).
-					WithClusterChecksEnabled(false).
-					WithAdmissionControllerEnabled(false).
-					WithOrchestratorExplorerEnabled(false).
-					WithKSMEnabled(false).
-					WithDogstatsdUnixDomainSocketConfigEnabled(false).
-					WithAnnotations(map[string]string{
-						autopilotKey: "true",
-					}).
-					Build()
-
-				_ = c.Create(context.TODO(), dda)
-				return dda
-			},
-			wantFunc: func(t *testing.T, c client.Client) error {
-				expectedContainers := []string{
-					string(apicommon.CoreAgentContainerName),
-					string(apicommon.TraceAgentContainerName),
-					string(apicommon.ProcessAgentContainerName),
-				}
-				if err := verifyDaemonsetContainers(c, resourcesNamespace, dsName, expectedContainers); err != nil {
-					return err
-				}
-
-				ds := &appsv1.DaemonSet{}
-				if err := c.Get(context.TODO(), types.NamespacedName{Namespace: resourcesNamespace, Name: dsName}, ds); err != nil {
-					return err
-				}
-
-				traceAgentFound := false
-				processAgentFound := false
-				for _, ctn := range ds.Spec.Template.Spec.Containers {
-					switch ctn.Name {
-					case string(apicommon.TraceAgentContainerName):
-						expectedCommand := []string{
-							"trace-agent",
-							"-config=/etc/datadog-agent/datadog.yaml",
-						}
-						if !reflect.DeepEqual(ctn.Command, expectedCommand) {
-							return fmt.Errorf("trace-agent command incorrect, expected: %v, got: %v", expectedCommand, ctn.Command)
-						}
-						traceAgentFound = true
-
-					case string(apicommon.ProcessAgentContainerName):
-						expectedCommand := []string{
-							"process-agent",
-							"-config=/etc/datadog-agent/datadog.yaml",
-						}
-						if !reflect.DeepEqual(ctn.Command, expectedCommand) {
-							return fmt.Errorf("process-agent command incorrect, expected: %v, got: %v", expectedCommand, ctn.Command)
-						}
-						processAgentFound = true
-					}
-				}
-
-				if !traceAgentFound {
-					return fmt.Errorf("trace-agent container not found")
-				}
-				if !processAgentFound {
-					return fmt.Errorf("process-agent container not found")
-				}
-
 				return nil
 			},
 		},