mirror of
				https://github.com/optim-enterprises-bv/kubernetes.git
				synced 2025-11-04 04:08:16 +00:00 
			
		
		
		
	e2e: remove workarounds for etcd2 test jobs
Now we've removed support for etcd2, we should make sure we're testing chunking fully.
This commit is contained in:
		@@ -82,10 +82,6 @@ var _ = SIGDescribe("Servers with support for API chunking", func() {
 | 
				
			|||||||
				list, err := client.List(opts)
 | 
									list, err := client.List(opts)
 | 
				
			||||||
				Expect(err).ToNot(HaveOccurred())
 | 
									Expect(err).ToNot(HaveOccurred())
 | 
				
			||||||
				framework.Logf("Retrieved %d/%d results with rv %s and continue %s", len(list.Items), opts.Limit, list.ResourceVersion, list.Continue)
 | 
									framework.Logf("Retrieved %d/%d results with rv %s and continue %s", len(list.Items), opts.Limit, list.ResourceVersion, list.Continue)
 | 
				
			||||||
				// TODO: kops PR job is still using etcd2, which prevents this feature from working. Remove this check when kops is upgraded to etcd3
 | 
					 | 
				
			||||||
				if len(list.Items) > int(opts.Limit) {
 | 
					 | 
				
			||||||
					framework.Skipf("ERROR: This cluster does not support chunking, which means it is running etcd2 and not supported.")
 | 
					 | 
				
			||||||
				}
 | 
					 | 
				
			||||||
				Expect(len(list.Items)).To(BeNumerically("<=", opts.Limit))
 | 
									Expect(len(list.Items)).To(BeNumerically("<=", opts.Limit))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
				if len(lastRV) == 0 {
 | 
									if len(lastRV) == 0 {
 | 
				
			||||||
@@ -120,10 +116,6 @@ var _ = SIGDescribe("Servers with support for API chunking", func() {
 | 
				
			|||||||
		opts := metav1.ListOptions{}
 | 
							opts := metav1.ListOptions{}
 | 
				
			||||||
		opts.Limit = oneTenth
 | 
							opts.Limit = oneTenth
 | 
				
			||||||
		list, err := client.List(opts)
 | 
							list, err := client.List(opts)
 | 
				
			||||||
		// TODO: kops PR job is still using etcd2, which prevents this feature from working. Remove this check when kops is upgraded to etcd3
 | 
					 | 
				
			||||||
		if len(list.Items) > int(opts.Limit) {
 | 
					 | 
				
			||||||
			framework.Skipf("ERROR: This cluster does not support chunking, which means it is running etcd2 and not supported.")
 | 
					 | 
				
			||||||
		}
 | 
					 | 
				
			||||||
		Expect(err).ToNot(HaveOccurred())
 | 
							Expect(err).ToNot(HaveOccurred())
 | 
				
			||||||
		firstToken := list.Continue
 | 
							firstToken := list.Continue
 | 
				
			||||||
		firstRV := list.ResourceVersion
 | 
							firstRV := list.ResourceVersion
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -108,10 +108,6 @@ var _ = SIGDescribe("Servers with support for Table transformation", func() {
 | 
				
			|||||||
			SetHeader("Accept", "application/json;as=Table;v=v1beta1;g=meta.k8s.io").
 | 
								SetHeader("Accept", "application/json;as=Table;v=v1beta1;g=meta.k8s.io").
 | 
				
			||||||
			Do().Into(pagedTable)
 | 
								Do().Into(pagedTable)
 | 
				
			||||||
		Expect(err).NotTo(HaveOccurred())
 | 
							Expect(err).NotTo(HaveOccurred())
 | 
				
			||||||
		// TODO: kops PR job is still using etcd2, which prevents this feature from working. Remove this check when kops is upgraded to etcd3
 | 
					 | 
				
			||||||
		if len(pagedTable.Rows) > 2 {
 | 
					 | 
				
			||||||
			framework.Skipf("ERROR: This cluster does not support chunking, which means it is running etcd2 and not supported.")
 | 
					 | 
				
			||||||
		}
 | 
					 | 
				
			||||||
		Expect(len(pagedTable.Rows)).To(Equal(2))
 | 
							Expect(len(pagedTable.Rows)).To(Equal(2))
 | 
				
			||||||
		Expect(pagedTable.ResourceVersion).ToNot(Equal(""))
 | 
							Expect(pagedTable.ResourceVersion).ToNot(Equal(""))
 | 
				
			||||||
		Expect(pagedTable.SelfLink).ToNot(Equal(""))
 | 
							Expect(pagedTable.SelfLink).ToNot(Equal(""))
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user