Merge pull request #45062 from FengyunPan/fix-e2e-daemonset-secret
Automatic merge from submit-queue [federation][e2e] Distinguish local vars and global vars None
This commit is contained in:
		@@ -245,7 +245,8 @@ func waitForDaemonSetOrFail(clientset *kubeclientset.Clientset, namespace string
 | 
				
			|||||||
	By(fmt.Sprintf("Fetching a federated daemonset shard of daemonset %q in namespace %q from cluster", daemonset.Name, namespace))
 | 
						By(fmt.Sprintf("Fetching a federated daemonset shard of daemonset %q in namespace %q from cluster", daemonset.Name, namespace))
 | 
				
			||||||
	var clusterDaemonSet *v1beta1.DaemonSet
 | 
						var clusterDaemonSet *v1beta1.DaemonSet
 | 
				
			||||||
	err := wait.PollImmediate(framework.Poll, timeout, func() (bool, error) {
 | 
						err := wait.PollImmediate(framework.Poll, timeout, func() (bool, error) {
 | 
				
			||||||
		clusterDaemonSet, err := clientset.Extensions().DaemonSets(namespace).Get(daemonset.Name, metav1.GetOptions{})
 | 
							var err error
 | 
				
			||||||
 | 
							clusterDaemonSet, err = clientset.Extensions().DaemonSets(namespace).Get(daemonset.Name, metav1.GetOptions{})
 | 
				
			||||||
		if (!present) && errors.IsNotFound(err) { // We want it gone, and it's gone.
 | 
							if (!present) && errors.IsNotFound(err) { // We want it gone, and it's gone.
 | 
				
			||||||
			By(fmt.Sprintf("Success: shard of federated daemonset %q in namespace %q in cluster is absent", daemonset.Name, namespace))
 | 
								By(fmt.Sprintf("Success: shard of federated daemonset %q in namespace %q in cluster is absent", daemonset.Name, namespace))
 | 
				
			||||||
			return true, nil // Success
 | 
								return true, nil // Success
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -225,7 +225,8 @@ func waitForSecretOrFail(clientset *kubeclientset.Clientset, nsName string, secr
 | 
				
			|||||||
	By(fmt.Sprintf("Fetching a federated secret shard of secret %q in namespace %q from cluster", secret.Name, nsName))
 | 
						By(fmt.Sprintf("Fetching a federated secret shard of secret %q in namespace %q from cluster", secret.Name, nsName))
 | 
				
			||||||
	var clusterSecret *v1.Secret
 | 
						var clusterSecret *v1.Secret
 | 
				
			||||||
	err := wait.PollImmediate(framework.Poll, timeout, func() (bool, error) {
 | 
						err := wait.PollImmediate(framework.Poll, timeout, func() (bool, error) {
 | 
				
			||||||
		clusterSecret, err := clientset.Core().Secrets(nsName).Get(secret.Name, metav1.GetOptions{})
 | 
							var err error
 | 
				
			||||||
 | 
							clusterSecret, err = clientset.Core().Secrets(nsName).Get(secret.Name, metav1.GetOptions{})
 | 
				
			||||||
		if (!present) && errors.IsNotFound(err) { // We want it gone, and it's gone.
 | 
							if (!present) && errors.IsNotFound(err) { // We want it gone, and it's gone.
 | 
				
			||||||
			By(fmt.Sprintf("Success: shard of federated secret %q in namespace %q in cluster is absent", secret.Name, nsName))
 | 
								By(fmt.Sprintf("Success: shard of federated secret %q in namespace %q in cluster is absent", secret.Name, nsName))
 | 
				
			||||||
			return true, nil // Success
 | 
								return true, nil // Success
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user