diff --git a/npc/controller_test.go b/npc/controller_test.go index 9557f443..ca690ac1 100644 --- a/npc/controller_test.go +++ b/npc/controller_test.go @@ -305,6 +305,6 @@ func TestDefaultAllow(t *testing.T) { require.True(t, m.Exist(defaultAllowIPSetName, fooPodIP)) controller.DeletePod(podFoo) - // Should remove foo pod to default-allow + // Should remove foo pod from default-allow require.False(t, m.Exist(defaultAllowIPSetName, fooPodIP)) } diff --git a/test/840_weave_kube_3_test.sh b/test/840_weave_kube_3_test.sh index 1736dc17..e45fe476 100755 --- a/test/840_weave_kube_3_test.sh +++ b/test/840_weave_kube_3_test.sh @@ -170,12 +170,12 @@ assert_raises 'wait_for_x check_all_pods_communicate pods' # nettest-deny should still not be able to reach nettest pods assert_raises "! $SSH $HOST1 $KUBECTL exec $denyPodName -- curl -s -S -f -m 2 http://$DOMAIN:8080/status >/dev/null" -# allow access for all +# allow access for nettest-deny run_on $HOST1 "$KUBECTL apply -f -" </dev/null" +# remove the access for nettest-deny +run_on $HOST1 "$KUBECTL delete netpol allow-nettest-deny" + +# nettest-deny should still not be able to reach nettest pods +assert_raises "! $SSH $HOST1 $KUBECTL exec $denyPodName -- curl -s -S -f -m 2 http://$DOMAIN:8080/status >/dev/null" + +# allow access for all +run_on $HOST1 "$KUBECTL apply -f -" </dev/null" + tear_down_kubeadm # Destroy our test ipset, and implicitly check it is still there