Skip to content

Commit

Permalink
connectivity: Add op=Exists toleration for test-conn-disrupt
Browse files Browse the repository at this point in the history
It was observed on a Kind cluster, that during Cilium upgrade (v1.13.4
-> main) kubelet set the taint node.kubernetes.io/network-unavailable
which evicted test-conn-disrupt pods. This resulted in a false negative
in the upgrade tests.

Fix the issue by tolerating any taint.

Signed-off-by: Martynas Pumputis <[email protected]>
  • Loading branch information
brb authored and michi-covalent committed Jun 28, 2023
1 parent 52038de commit 127d9b8
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions connectivity/check/deployment.go
Original file line number Diff line number Diff line change
Expand Up @@ -600,6 +600,9 @@ func (ct *ConnectivityTest) deploy(ctx context.Context) error {
Labels: map[string]string{"app": "test-conn-disrupt-server"},
Command: []string{"tcd-server", "8000"},
Port: 8000,
Tolerations: []corev1.Toleration{
{Operator: corev1.TolerationOpExists},
},
})
_, err = ct.clients.src.CreateServiceAccount(ctx, ct.params.TestNamespace, k8s.NewServiceAccount(testConnDisruptServerDeploymentName), metav1.CreateOptions{})
if err != nil {
Expand Down Expand Up @@ -646,6 +649,9 @@ func (ct *ConnectivityTest) deploy(ctx context.Context) error {
fmt.Sprintf("test-conn-disrupt.%s.svc.cluster.local.:8000", ct.params.TestNamespace),
},
ReadinessProbe: readinessProbe,
Tolerations: []corev1.Toleration{
{Operator: corev1.TolerationOpExists},
},
})

_, err = ct.clients.src.CreateServiceAccount(ctx, ct.params.TestNamespace, k8s.NewServiceAccount(testConnDisruptClientDeploymentName), metav1.CreateOptions{})
Expand Down

0 comments on commit 127d9b8

Please sign in to comment.