Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

volume: fix WCOW volume mounts #10090

Merged
merged 1 commit into from
Dec 16, 2022
Merged

volume: fix WCOW volume mounts #10090

merged 1 commit into from
Dec 16, 2022

Commits on Dec 16, 2022

  1. volume: fix WCOW volume mounts

    Do not use the older `Volumes` field in the API; instead rely on
    the more robust `Mounts`. For Linux containers, it seems that it's
    fine to set both of these. For Windows containers (WCOW), however,
    there appears to be a Moby bug that causes it to normalize the
    anonymous (`Volumes`) variant to lowercase, which can result in
    duplicative volume definitions and an error when trying to start
    the container.
    
    Fixes docker#9577.
    
    Signed-off-by: Milas Bowman <milas.bowman@docker.com>
    milas committed Dec 16, 2022
    Configuration menu
    Copy the full SHA
    ffb9544 View commit details
    Browse the repository at this point in the history