Merge pull request #113789 from jsafrane/add-selinux-tests

Add multi-pod tests with SELinux mounts

test/e2e/storage/csi_mock/base.go

@@ -664,6 +664,11 @@ func startPausePodWithSELinuxOptions(cs clientset.Interface, pvc *v1.PersistentV
 			},
 		},
 	}
+	if node.Name != "" {
+		// Force schedule the pod to skip scheduler RWOP checks
+		framework.Logf("Forcing node name %s", node.Name)
+		pod.Spec.NodeName = node.Name
+	}
 	e2epod.SetNodeSelection(&pod.Spec, node)
 	return cs.CoreV1().Pods(ns).Create(context.TODO(), pod, metav1.CreateOptions{})
 }
@@ -821,22 +826,26 @@ func compareCSICalls(ctx context.Context, trackedCalls []string, expectedCallSeq
 
 // createSELinuxMountPreHook creates a hook that records the mountOptions passed in
 // through NodeStageVolume and NodePublishVolume calls.
-func createSELinuxMountPreHook(nodeStageMountOpts, nodePublishMountOpts *[]string) *drivers.Hooks {
+func createSELinuxMountPreHook(nodeStageMountOpts, nodePublishMountOpts *[]string, stageCalls, unstageCalls, publishCalls, unpublishCalls *atomic.Int32) *drivers.Hooks {
 	return &drivers.Hooks{
 		Pre: func(ctx context.Context, fullMethod string, request interface{}) (reply interface{}, err error) {
-			nodeStageRequest, ok := request.(*csipbv1.NodeStageVolumeRequest)
+			switch req := request.(type) {
-			if ok {
+			case *csipbv1.NodeStageVolumeRequest:
-				mountVolume := nodeStageRequest.GetVolumeCapability().GetMount()
+				stageCalls.Add(1)
+				mountVolume := req.GetVolumeCapability().GetMount()
 				if mountVolume != nil {
 					*nodeStageMountOpts = mountVolume.MountFlags
 				}
-			}
+			case *csipbv1.NodePublishVolumeRequest:
-			nodePublishRequest, ok := request.(*csipbv1.NodePublishVolumeRequest)
+				publishCalls.Add(1)
-			if ok {
+				mountVolume := req.GetVolumeCapability().GetMount()
-				mountVolume := nodePublishRequest.GetVolumeCapability().GetMount()
 				if mountVolume != nil {
 					*nodePublishMountOpts = mountVolume.MountFlags
 				}
+			case *csipbv1.NodeUnstageVolumeRequest:
+				unstageCalls.Add(1)
+			case *csipbv1.NodeUnpublishVolumeRequest:
+				unpublishCalls.Add(1)
 			}
 			return nil, nil
 		},

test/e2e/storage/csi_mock/csi_selinux_mount.go

@@ -18,10 +18,16 @@ package csi_mock
 
 import (
 	"context"
+	"sync/atomic"
 
 	"github.com/onsi/ginkgo/v2"
+	"github.com/onsi/gomega"
 	v1 "k8s.io/api/core/v1"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/fields"
+	"k8s.io/kubernetes/pkg/kubelet/events"
 	"k8s.io/kubernetes/test/e2e/framework"
+	e2eevents "k8s.io/kubernetes/test/e2e/framework/events"
 	e2epod "k8s.io/kubernetes/test/e2e/framework/pod"
 	e2eskipper "k8s.io/kubernetes/test/e2e/framework/skipper"
 	"k8s.io/kubernetes/test/e2e/storage/utils"
@@ -35,57 +41,92 @@ var _ = utils.SIGDescribe("CSI Mock selinux on mount", func() {
 
 	ginkgo.Context("SELinuxMount [LinuxOnly][Feature:SELinux][Feature:SELinuxMountReadWriteOncePod]", func() {
 		// Make sure all options are set so system specific defaults are not used.
-		seLinuxOpts := v1.SELinuxOptions{
+		seLinuxOpts1 := v1.SELinuxOptions{
 			User:  "system_u",
 			Role:  "object_r",
 			Type:  "container_file_t",
 			Level: "s0:c0,c1",
 		}
-		seLinuxMountOption := "context=\"system_u:object_r:container_file_t:s0:c0,c1\""
+		seLinuxMountOption1 := "context=\"system_u:object_r:container_file_t:s0:c0,c1\""
+		seLinuxOpts2 := v1.SELinuxOptions{
+			User:  "system_u",
+			Role:  "object_r",
+			Type:  "container_file_t",
+			Level: "s0:c98,c99",
+		}
+		seLinuxMountOption2 := "context=\"system_u:object_r:container_file_t:s0:c98,c99\""
 
 		tests := []struct {
 			name                       string
-			seLinuxEnabled       bool
+			csiDriverSELinuxEnabled    bool
-			seLinuxSetInPod      bool
+			firstPodSELinuxOpts        *v1.SELinuxOptions
+			startSecondPod             bool
+			secondPodSELinuxOpts       *v1.SELinuxOptions
 			mountOptions               []string
 			volumeMode                 v1.PersistentVolumeAccessMode
-			expectedMountOptions []string
+			expectedFirstMountOptions  []string
+			expectedSecondMountOptions []string
+			expectedUnstage            bool
 		}{
+			// Start just a single pod and check its volume is mounted correctly
 			{
 				name:                      "should pass SELinux mount option for RWOP volume and Pod with SELinux context set",
-				seLinuxEnabled:       true,
+				csiDriverSELinuxEnabled:   true,
-				seLinuxSetInPod:      true,
+				firstPodSELinuxOpts:       &seLinuxOpts1,
 				volumeMode:                v1.ReadWriteOncePod,
-				expectedMountOptions: []string{seLinuxMountOption},
+				expectedFirstMountOptions: []string{seLinuxMountOption1},
 			},
 			{
 				name:                      "should add SELinux mount option to existing mount options",
-				seLinuxEnabled:       true,
+				csiDriverSELinuxEnabled:   true,
-				seLinuxSetInPod:      true,
+				firstPodSELinuxOpts:       &seLinuxOpts1,
 				mountOptions:              []string{"noexec", "noatime"},
 				volumeMode:                v1.ReadWriteOncePod,
-				expectedMountOptions: []string{"noexec", "noatime", seLinuxMountOption},
+				expectedFirstMountOptions: []string{"noexec", "noatime", seLinuxMountOption1},
 			},
 			{
 				name:                      "should not pass SELinux mount option for RWO volume",
-				seLinuxEnabled:       true,
+				csiDriverSELinuxEnabled:   true,
-				seLinuxSetInPod:      true,
+				firstPodSELinuxOpts:       &seLinuxOpts1,
 				volumeMode:                v1.ReadWriteOnce,
-				expectedMountOptions: nil,
+				expectedFirstMountOptions: nil,
 			},
 			{
 				name:                      "should not pass SELinux mount option for Pod without SELinux context",
-				seLinuxEnabled:       true,
+				csiDriverSELinuxEnabled:   true,
-				seLinuxSetInPod:      false,
+				firstPodSELinuxOpts:       nil,
 				volumeMode:                v1.ReadWriteOncePod,
-				expectedMountOptions: nil,
+				expectedFirstMountOptions: nil,
 			},
 			{
 				name:                      "should not pass SELinux mount option for CSI driver that does not support SELinux mount",
-				seLinuxEnabled:       false,
+				csiDriverSELinuxEnabled:   false,
-				seLinuxSetInPod:      true,
+				firstPodSELinuxOpts:       &seLinuxOpts1,
 				volumeMode:                v1.ReadWriteOncePod,
-				expectedMountOptions: nil,
+				expectedFirstMountOptions: nil,
+			},
+			// Start two pods in a sequence and check their volume is / is not unmounted in between
+			{
+				name:                       "should not unstage volume when starting a second pod with the same SELinux context",
+				csiDriverSELinuxEnabled:    true,
+				firstPodSELinuxOpts:        &seLinuxOpts1,
+				startSecondPod:             true,
+				secondPodSELinuxOpts:       &seLinuxOpts1,
+				volumeMode:                 v1.ReadWriteOncePod,
+				expectedFirstMountOptions:  []string{seLinuxMountOption1},
+				expectedSecondMountOptions: []string{seLinuxMountOption1},
+				expectedUnstage:            false,
+			},
+			{
+				name:                       "should unstage volume when starting a second pod with different SELinux context",
+				csiDriverSELinuxEnabled:    true,
+				firstPodSELinuxOpts:        &seLinuxOpts1,
+				startSecondPod:             true,
+				secondPodSELinuxOpts:       &seLinuxOpts2,
+				volumeMode:                 v1.ReadWriteOncePod,
+				expectedFirstMountOptions:  []string{seLinuxMountOption1},
+				expectedSecondMountOptions: []string{seLinuxMountOption2},
+				expectedUnstage:            true,
 			},
 		}
 		for _, t := range tests {
@@ -95,27 +136,103 @@ var _ = utils.SIGDescribe("CSI Mock selinux on mount", func() {
 					e2eskipper.Skipf("SELinuxMount is only applied on linux nodes -- skipping")
 				}
 				var nodeStageMountOpts, nodePublishMountOpts []string
+				var unstageCalls, stageCalls, unpublishCalls, publishCalls atomic.Int32
 				m.init(ctx, testParameters{
 					disableAttach:      true,
 					registerDriver:     true,
-					enableSELinuxMount: &t.seLinuxEnabled,
+					enableSELinuxMount: &t.csiDriverSELinuxEnabled,
-					hooks:              createSELinuxMountPreHook(&nodeStageMountOpts, &nodePublishMountOpts),
+					hooks:              createSELinuxMountPreHook(&nodeStageMountOpts, &nodePublishMountOpts, &stageCalls, &unstageCalls, &publishCalls, &unpublishCalls),
 				})
 				ginkgo.DeferCleanup(m.cleanup)
 
+				// Act
+				ginkgo.By("Starting the initial pod")
 				accessModes := []v1.PersistentVolumeAccessMode{t.volumeMode}
-				var podSELinuxOpts *v1.SELinuxOptions
+				_, claim, pod := m.createPodWithSELinux(ctx, accessModes, t.mountOptions, t.firstPodSELinuxOpts)
-				if t.seLinuxSetInPod {
-					// Make sure all options are set so system specific defaults are not used.
-					podSELinuxOpts = &seLinuxOpts
-				}
-
-				_, _, pod := m.createPodWithSELinux(ctx, accessModes, t.mountOptions, podSELinuxOpts)
 				err := e2epod.WaitForPodNameRunningInNamespace(ctx, m.cs, pod.Name, pod.Namespace)
-				framework.ExpectNoError(err, "failed to start pod")
+				framework.ExpectNoError(err, "starting the initial pod")
 
-				framework.ExpectEqual(nodeStageMountOpts, t.expectedMountOptions, "Expect NodeStageVolumeRequest.VolumeCapability.MountVolume. to equal %q; got: %q", t.expectedMountOptions, nodeStageMountOpts)
+				// Assert
-				framework.ExpectEqual(nodePublishMountOpts, t.expectedMountOptions, "Expect NodePublishVolumeRequest.VolumeCapability.MountVolume.VolumeMountGroup to equal %q; got: %q", t.expectedMountOptions, nodeStageMountOpts)
+				ginkgo.By("Checking the initial pod mount options")
+				framework.ExpectEqual(nodeStageMountOpts, t.expectedFirstMountOptions, "NodeStage MountFlags for the initial pod")
+				framework.ExpectEqual(nodePublishMountOpts, t.expectedFirstMountOptions, "NodePublish MountFlags for the initial pod")
+
+				ginkgo.By("Checking the CSI driver calls for the initial pod")
+				gomega.Expect(unstageCalls.Load()).To(gomega.BeNumerically("==", 0), "NodeUnstage call count for the initial pod")
+				gomega.Expect(unpublishCalls.Load()).To(gomega.BeNumerically("==", 0), "NodeUnpublish call count for the initial pod")
+				gomega.Expect(stageCalls.Load()).To(gomega.BeNumerically(">", 0), "NodeStage for the initial pod")
+				gomega.Expect(publishCalls.Load()).To(gomega.BeNumerically(">", 0), "NodePublish for the initial pod")
+
+				if !t.startSecondPod {
+					return
+				}
+
+				// Arrange 2nd part of the test
+				ginkgo.By("Starting the second pod to check if a volume used by the initial pod is / is not unmounted based on SELinux context")
+
+				// Skip scheduler, it would block scheduling the second pod with ReadWriteOncePod PV.
+				pod, err = m.cs.CoreV1().Pods(pod.Namespace).Get(ctx, pod.Name, metav1.GetOptions{})
+				framework.ExpectNoError(err, "getting the initial pod")
+				nodeSelection := e2epod.NodeSelection{Name: pod.Spec.NodeName}
+				pod2, err := startPausePodWithSELinuxOptions(f.ClientSet, claim, nodeSelection, f.Namespace.Name, t.secondPodSELinuxOpts)
+				framework.ExpectNoError(err, "creating second pod with SELinux context %s", t.secondPodSELinuxOpts)
+				m.pods = append(m.pods, pod2)
+
+				// Delete the initial pod only after kubelet processes the second pod and adds its volumes to
+				// DesiredStateOfWorld.
+				// In this state, any volume UnPublish / UnStage must be done because of SELinux contexts and not
+				// because of random races because volumes of the second pod are not in DesiredStateOfWorld yet.
+				ginkgo.By("Waiting for the second pod to fail to start because of ReadWriteOncePod.")
+				eventSelector := fields.Set{
+					"involvedObject.kind":      "Pod",
+					"involvedObject.name":      pod2.Name,
+					"involvedObject.namespace": pod2.Namespace,
+					"reason":                   events.FailedMountVolume,
+				}.AsSelector().String()
+				var msg string
+				if t.expectedUnstage {
+					// This message is emitted before kubelet checks for ReadWriteOncePod
+					msg = "conflicting SELinux labels of volume"
+				} else {
+					msg = "volume uses the ReadWriteOncePod access mode and is already in use by another pod"
+				}
+				err = e2eevents.WaitTimeoutForEvent(ctx, m.cs, pod2.Namespace, eventSelector, msg, f.Timeouts.PodStart)
+				framework.ExpectNoError(err, "waiting for event %q in the second test pod", msg)
+
+				// count fresh CSI driver calls between the first and the second pod
+				nodeStageMountOpts = nil
+				nodePublishMountOpts = nil
+				unstageCalls.Store(0)
+				unpublishCalls.Store(0)
+				stageCalls.Store(0)
+				publishCalls.Store(0)
+
+				// Act 2nd part of the test
+				ginkgo.By("Deleting the initial pod")
+				err = e2epod.DeletePodWithWait(ctx, m.cs, pod)
+				framework.ExpectNoError(err, "deleting the initial pod")
+
+				// Assert 2nd part of the test
+				ginkgo.By("Waiting for the second pod to start")
+				err = e2epod.WaitForPodNameRunningInNamespace(ctx, m.cs, pod2.Name, pod2.Namespace)
+				framework.ExpectNoError(err, "starting the second pod")
+
+				ginkgo.By("Checking CSI driver calls for the second pod")
+				if t.expectedUnstage {
+					// Volume should be fully unstaged between the first and the second pod
+					gomega.Expect(unstageCalls.Load()).To(gomega.BeNumerically(">", 0), "NodeUnstage calls after the first pod is deleted")
+					gomega.Expect(stageCalls.Load()).To(gomega.BeNumerically(">", 0), "NodeStage calls for the second pod")
+					// The second pod got the right mount option
+					framework.ExpectEqual(nodeStageMountOpts, t.expectedSecondMountOptions, "NodeStage MountFlags for the second pod")
+				} else {
+					// Volume should not be fully unstaged between the first and the second pod
+					gomega.Expect(unstageCalls.Load()).To(gomega.BeNumerically("==", 0), "NodeUnstage calls after the first pod is deleted")
+					gomega.Expect(stageCalls.Load()).To(gomega.BeNumerically("==", 0), "NodeStage calls for the second pod")
+				}
+				// In both cases, Unublish and Publish is called, with the right mount opts
+				gomega.Expect(unpublishCalls.Load()).To(gomega.BeNumerically(">", 0), "NodeUnpublish calls after the first pod is deleted")
+				gomega.Expect(publishCalls.Load()).To(gomega.BeNumerically(">", 0), "NodePublish calls for the second pod")
+				framework.ExpectEqual(nodePublishMountOpts, t.expectedSecondMountOptions, "NodePublish MountFlags for the second pod")
 			})
 		}
 	})