Skip to content

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Aug 5, 2025

relates to:

The bind-nonrecursive option was replaced with the bind-recursive option (see cli-4316, cli-4671). The option was still accepted, but printed a deprecation warning:

bind-nonrecursive is deprecated, use bind-recursive=disabled instead

In the v29.0 release, this warning is removed, and returned as an error. Users should use the equivalent bind-recursive=disabled option instead.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

Remove deprecated `bind-nonrecursive` option for `--mount`

- A picture of a cute animal (not mandatory but encouraged)

The `bind-nonrecursive` option was replaced with the [`bind-recursive`]
option (see [cli-4316], [cli-4671]). The option was still accepted, but
printed a deprecation warning:

    bind-nonrecursive is deprecated, use bind-recursive=disabled instead

In the v29.0 release, this warning is removed, and returned as an error.
Users should use the equivalent `bind-recursive=disabled` option instead.

[`bind-recursive`]: https://docs.docker.com/engine/storage/bind-mounts/#recursive-mounts
[cli-4316]: docker#4316
[cli-4671]: docker#4671

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
opts/mount.go 33.33% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah
Copy link
Member Author

For the record; at the time we chose to "soft" deprecate this option (keep it, but warn). However, having multiple options doing the same thing is confusing, and we did mark it as deprecated, so I'm removing it in this PR.

This options would be used rarely, but it also doesn't take a massive amount of code to maintain, so if there's significant outfall, we could still consider adding back a stub.

@thaJeztah thaJeztah merged commit f2c64c1 into docker:master Aug 6, 2025
114 of 121 checks passed
@thaJeztah thaJeztah deleted the deprecate_bind_nonrecursive branch August 6, 2025 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants