recursive read-only (RRO) mount: support the Docker v25 form #2651

Activity

1. AkihiroSuda commented on Nov 21, 2023

   

   AkihiroSuda

   on Nov 21, 2023

   Last edited by AkihiroSuda

   MemberAuthor

   More actions

   May depend on:

   • ParseFlagMount should not depend on ParseFlagV (but ParseFlagV may depend on ParseFlagMount) #1390
2. AkihiroSuda

   added

   enhancementNew feature or requestNew feature or request

   on Nov 21, 2023
3. thaJeztah commented on Nov 22, 2023

   

   thaJeztah

   on Nov 22, 2023

   Member

   More actions

   Hope I didn't make life hard on you, removing the boolean format ❤️ (I was looking at it again, and thought "now" is the moment to still change it; and it's easier to add back later, than to ship it and remove later).
4. AkihiroSuda

   added a parent issue on May 7, 2025

   • Update the list of the unimplemented Docker features (and implement them) #3867
5. AkihiroSuda commented on Jul 1, 2026

   

   AkihiroSuda

   on Jul 1, 2026

   MemberAuthor

   More actions

   nerdctl/pkg/mountutil/mountutil_linux.go

   Lines 383 to 402 in 4608753

   	case "bind-recursive":
   	// bind-recursive is the Docker option that supersedes bind-nonrecursive.
   	valS := value
   	// Allow boolean as an alias to "enabled" or "disabled"
   	if b, err := strconv.ParseBool(valS); err == nil {
   	if b {
   	valS = "enabled"
   	} else {
   	valS = "disabled"
   	}
   	}
   	switch valS {
   	case "enabled":
   	bindNonRecursive = false
   	case "disabled":
   	bindNonRecursive = true
   	default:
   	// TODO: support "writable", "readonly"
   	return nil, fmt.Errorf("invalid value for %s: %s (must be \"enabled\" or \"disabled\")", key, value)
   	}

   It also has to support bind-recursive=(readonly|writeonly)
6. AkihiroSuda

   closed this as completedin #5043on Jul 9, 2026