Emulate "bind" mounts using the bind filter

Signed-off-by: Gabriel Adrian Samfira <[email protected]>

Author: gabriel-samfira

snapshot/localmounter_windows.go

@@ -3,16 +3,18 @@ package snapshot
 import (
 	"os"
 
+	"github.com/Microsoft/go-winio/pkg/bindfilter"
 	"github.com/containerd/containerd/errdefs"
 	"github.com/containerd/containerd/mount"
 	"github.com/pkg/errors"
+	"golang.org/x/sys/windows"
 )
 
 func (lm *localMounter) Mount() (string, error) {
 	lm.mu.Lock()
 	defer lm.mu.Unlock()
 
-	if lm.mounts == nil {
+	if lm.mounts == nil && lm.mountable != nil {
 		mounts, release, err := lm.mountable.Mount()
 		if err != nil {
 			return "", err
@@ -33,9 +35,18 @@ func (lm *localMounter) Mount() (string, error) {
 		return "", errors.Wrap(err, "failed to create temp dir")
 	}
 
-	if err := m.Mount(dir); err != nil {
-		return "", errors.Wrapf(err, "failed to mount %v: %+v", m, err)
+	if m.Type == "bind" || m.Type == "rbind" {
+		// The Windows snapshotter does not have any notion of bind mounts. We emulate
+		// bind mounts here using the bind filter.
+		if err := bindfilter.ApplyFileBinding(dir, m.Source, m.ReadOnly()); err != nil {
+			return "", errors.Wrapf(err, "failed to mount %v: %+v", m, err)
+		}
+	} else {
+		if err := m.Mount(dir); err != nil {
+			return "", errors.Wrapf(err, "failed to mount %v: %+v", m, err)
+		}
 	}
+
 	lm.target = dir
 	return lm.target, nil
 }
@@ -44,9 +55,32 @@ func (lm *localMounter) Unmount() error {
 	lm.mu.Lock()
 	defer lm.mu.Unlock()
 
+	// NOTE(gabriel-samfira): Should we just return nil if len(lm.mounts) == 0?
+	// Calling Mount() would fail on an instance of the localMounter where mounts contains
+	// anything other than 1 mount.
+	if len(lm.mounts) != 1 {
+		return errors.Wrapf(errdefs.ErrNotImplemented, "request to mount %d layers, only 1 is supported", len(lm.mounts))
+	}
+	m := lm.mounts[0]
+
 	if lm.target != "" {
-		if err := mount.Unmount(lm.target, 0); err != nil {
-			return err
+		if m.Type == "bind" || m.Type == "rbind" {
+			if err := bindfilter.RemoveFileBinding(lm.target); err != nil {
+				// The following two errors denote that lm.target is not a mount point.
+				if !errors.Is(err, windows.ERROR_INVALID_PARAMETER) && !errors.Is(err, windows.ERROR_NOT_FOUND) {
+					return errors.Wrapf(err, "failed to unmount %v: %+v", lm.target, err)
+				}
+			}
+		} else {
+			// The containerd snapshotter uses the bind filter internally to mount windows-layer
+			// volumes. We use same bind filter here to emulate bind mounts. In theory we could
+			// simply call mount.Unmount() here, without the extra check for bind mounts and explicit
+			// call to bindfilter.RemoveFileBinding() (above), but this would operate under the
+			// assumption that the internal implementation in containerd will always be based on the
+			// bind filter, which feels brittle.
+			if err := mount.Unmount(lm.target, 0); err != nil {
+				return errors.Wrapf(err, "failed to unmount %v: %+v", lm.target, err)
+			}
 		}
 		os.RemoveAll(lm.target)
 		lm.target = ""