Merge pull request #121880 from borg-land/skip-a-broken-test
Add kubeup label instead of a feature label
This commit is contained in:
		@@ -29,7 +29,6 @@ import (
 | 
				
			|||||||
	"k8s.io/apimachinery/pkg/labels"
 | 
						"k8s.io/apimachinery/pkg/labels"
 | 
				
			||||||
	"k8s.io/apimachinery/pkg/util/wait"
 | 
						"k8s.io/apimachinery/pkg/util/wait"
 | 
				
			||||||
	clientset "k8s.io/client-go/kubernetes"
 | 
						clientset "k8s.io/client-go/kubernetes"
 | 
				
			||||||
	"k8s.io/kubernetes/test/e2e/feature"
 | 
					 | 
				
			||||||
	"k8s.io/kubernetes/test/e2e/framework"
 | 
						"k8s.io/kubernetes/test/e2e/framework"
 | 
				
			||||||
	e2enode "k8s.io/kubernetes/test/e2e/framework/node"
 | 
						e2enode "k8s.io/kubernetes/test/e2e/framework/node"
 | 
				
			||||||
	e2epod "k8s.io/kubernetes/test/e2e/framework/pod"
 | 
						e2epod "k8s.io/kubernetes/test/e2e/framework/pod"
 | 
				
			||||||
@@ -106,7 +105,7 @@ var _ = SIGDescribe("DNS horizontal autoscaling", func() {
 | 
				
			|||||||
	// This test is separated because it is slow and need to run serially.
 | 
						// This test is separated because it is slow and need to run serially.
 | 
				
			||||||
	// Will take around 5 minutes to run on a 4 nodes cluster.
 | 
						// Will take around 5 minutes to run on a 4 nodes cluster.
 | 
				
			||||||
	// TODO(upodroid) This test will be removed in 1.33 when kubeup is removed
 | 
						// TODO(upodroid) This test will be removed in 1.33 when kubeup is removed
 | 
				
			||||||
	f.It(f.WithSerial(), f.WithSlow(), feature.KubeUp, "kube-dns-autoscaler should scale kube-dns pods when cluster size changed", func(ctx context.Context) {
 | 
						f.It(f.WithSerial(), f.WithSlow(), f.WithLabel("KubeUp"), "kube-dns-autoscaler should scale kube-dns pods when cluster size changed", func(ctx context.Context) {
 | 
				
			||||||
		numNodes, err := e2enode.TotalRegistered(ctx, c)
 | 
							numNodes, err := e2enode.TotalRegistered(ctx, c)
 | 
				
			||||||
		framework.ExpectNoError(err)
 | 
							framework.ExpectNoError(err)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -88,7 +88,8 @@ var _ = SIGDescribe("Restart", framework.WithDisruptive(), func() {
 | 
				
			|||||||
		}
 | 
							}
 | 
				
			||||||
	})
 | 
						})
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	ginkgo.It("should restart all nodes and ensure all nodes and pods recover", func(ctx context.Context) {
 | 
						// TODO(upodroid) This test will be removed in 1.33 when kubeup is removed
 | 
				
			||||||
 | 
						f.It(f.WithLabel("KubeUp"), "should restart all nodes and ensure all nodes and pods recover", func(ctx context.Context) {
 | 
				
			||||||
		ginkgo.By("restarting all of the nodes")
 | 
							ginkgo.By("restarting all of the nodes")
 | 
				
			||||||
		err := common.RestartNodes(f.ClientSet, originalNodes)
 | 
							err := common.RestartNodes(f.ClientSet, originalNodes)
 | 
				
			||||||
		framework.ExpectNoError(err)
 | 
							framework.ExpectNoError(err)
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -71,7 +71,6 @@ var (
 | 
				
			|||||||
	KubeProxyDaemonSetDowngrade             = framework.WithFeature(framework.ValidFeatures.Add("KubeProxyDaemonSetDowngrade"))
 | 
						KubeProxyDaemonSetDowngrade             = framework.WithFeature(framework.ValidFeatures.Add("KubeProxyDaemonSetDowngrade"))
 | 
				
			||||||
	KubeProxyDaemonSetUpgrade               = framework.WithFeature(framework.ValidFeatures.Add("KubeProxyDaemonSetUpgrade"))
 | 
						KubeProxyDaemonSetUpgrade               = framework.WithFeature(framework.ValidFeatures.Add("KubeProxyDaemonSetUpgrade"))
 | 
				
			||||||
	KubeProxyDaemonSetMigration             = framework.WithFeature(framework.ValidFeatures.Add("KubeProxyDaemonSetMigration"))
 | 
						KubeProxyDaemonSetMigration             = framework.WithFeature(framework.ValidFeatures.Add("KubeProxyDaemonSetMigration"))
 | 
				
			||||||
	KubeUp                                  = framework.WithFeature(framework.ValidFeatures.Add("KubeUp"))
 | 
					 | 
				
			||||||
	LabelSelector                           = framework.WithFeature(framework.ValidFeatures.Add("LabelSelector"))
 | 
						LabelSelector                           = framework.WithFeature(framework.ValidFeatures.Add("LabelSelector"))
 | 
				
			||||||
	LocalStorageCapacityIsolation           = framework.WithFeature(framework.ValidFeatures.Add("LocalStorageCapacityIsolation"))
 | 
						LocalStorageCapacityIsolation           = framework.WithFeature(framework.ValidFeatures.Add("LocalStorageCapacityIsolation"))
 | 
				
			||||||
	LocalStorageCapacityIsolationQuota      = framework.WithFeature(framework.ValidFeatures.Add("LocalStorageCapacityIsolationQuota"))
 | 
						LocalStorageCapacityIsolationQuota      = framework.WithFeature(framework.ValidFeatures.Add("LocalStorageCapacityIsolationQuota"))
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user