Skip flaky tests (#7002)

aws · Nov 9, 2023 · 4e9fecf · 4e9fecf
1 parent ba2ee55
commit 4e9fecf
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 1 deletion.
diff --git a/pkg/files/reader_test.go b/pkg/files/reader_test.go
@@ -96,7 +96,8 @@ func TestReaderReadFileHTTPSSuccess(t *testing.T) {
 }
 
 func TestReaderReadFileHTTPSProxySuccess(t *testing.T) {
-	t.Skip("Flaky test, needs work")
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/5775)")
+
 	g := NewWithT(t)
 	filePath := "testdata/file.yaml"
 	// It's important to use example.com because the certificate created for

diff --git a/pkg/providers/cloudstack/reconciler/reconciler_test.go b/pkg/providers/cloudstack/reconciler/reconciler_test.go
@@ -42,6 +42,8 @@ const (
 )
 
 func TestReconcilerReconcileSuccess(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6996)")
+
 	tt := newReconcilerTest(t)
 	// We want to check that the cluster status is cleaned up if validations are passed
 	tt.cluster.SetFailure(anywherev1.FailureReasonType("InvalidCluster"), "invalid cluster")
@@ -192,6 +194,8 @@ func TestReconcilerValidateMachineConfigFail(t *testing.T) {
 }
 
 func TestReconcilerControlPlaneIsNotReady(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/7000)")
+
 	tt := newReconcilerTest(t)
 
 	kcp := test.KubeadmControlPlane(func(kcp *controlplanev1.KubeadmControlPlane) {
@@ -234,6 +238,8 @@ func TestReconcilerControlPlaneIsNotReady(t *testing.T) {
 }
 
 func TestReconcileControlPlaneUnstackedEtcdSuccess(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/7001)")
+
 	tt := newReconcilerTest(t)
 	tt.cluster.Spec.ExternalEtcdConfiguration = &anywherev1.ExternalEtcdConfiguration{
 		Count: 1,

diff --git a/pkg/providers/docker/reconciler/reconciler_test.go b/pkg/providers/docker/reconciler/reconciler_test.go
@@ -39,6 +39,8 @@ const (
 )
 
 func TestReconcilerReconcileSuccess(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6996)")
+
 	tt := newReconcilerTest(t)
 	logger := test.NewNullLogger()
 	kcp := test.KubeadmControlPlane(func(kcp *controlplanev1.KubeadmControlPlane) {

diff --git a/pkg/providers/snow/reconciler/reconciler_test.go b/pkg/providers/snow/reconciler/reconciler_test.go
@@ -38,6 +38,8 @@ const (
 )
 
 func TestReconcilerReconcileSuccess(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6996)")
+
 	tt := newReconcilerTest(t)
 	// We want to check that the cluster status is cleaned up if validations are passed
 	tt.cluster.SetFailure(anywherev1.FailureReasonType("InvalidCluster"), "invalid cluster")
@@ -190,6 +192,8 @@ func TestReconcilerValidateMachineConfigsMachineConfigNotValidated(t *testing.T)
 }
 
 func TestReconcilerReconcileWorkers(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6999)")
+
 	tt := newReconcilerTest(t)
 	capiCluster := test.CAPICluster(func(c *clusterv1.Cluster) {
 		c.Name = tt.cluster.Name

diff --git a/pkg/providers/tinkerbell/reconciler/reconciler_test.go b/pkg/providers/tinkerbell/reconciler/reconciler_test.go
@@ -56,6 +56,8 @@ func TestReconcilerGenerateSpec(t *testing.T) {
 }
 
 func TestReconcilerReconcileSuccess(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6996)")
+
 	tt := newReconcilerTest(t)
 
 	capiCluster := test.CAPICluster(func(c *clusterv1.Cluster) {
@@ -584,6 +586,8 @@ func TestReconcilerValidateHardwareCountRollingUpdateFail(t *testing.T) {
 }
 
 func TestReconcilerValidateHardwareScalingUpdateFail(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6997)")
+
 	tt := newReconcilerTest(t)
 	tt.createAllObjs()
 
@@ -715,6 +719,8 @@ func TestReconcilerDetectOperationK8sVersionUpgradeCPOnly(t *testing.T) {
 }
 
 func TestReconcilerDetectOperationK8sVersionUpgradeWorkerOnly(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6998)")
+
 	tt := newReconcilerTest(t)
 	tt.createAllObjs()
 

diff --git a/pkg/providers/vsphere/reconciler/reconciler_test.go b/pkg/providers/vsphere/reconciler/reconciler_test.go
@@ -46,6 +46,8 @@ const (
 )
 
 func TestReconcilerReconcileSuccess(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/6996)")
+
 	tt := newReconcilerTest(t)
 	// We want to check that the cluster status is cleaned up if validations are passed
 	tt.cluster.SetFailure(anywherev1.FailureReasonType("InvalidCluster"), "invalid cluster")
@@ -184,6 +186,8 @@ func TestSetupEnvVars(t *testing.T) {
 }
 
 func TestReconcilerControlPlaneIsNotReady(t *testing.T) {
+	t.Skip("Flaky (https://github.com/aws/eks-anywhere/issues/7000)")
+
 	tt := newReconcilerTest(t)
 	kcp := test.KubeadmControlPlane(func(kcp *controlplanev1.KubeadmControlPlane) {
 		kcp.Name = tt.cluster.Name