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

Issue: Unnamed docker volumes #478

Closed
max06 opened this issue Apr 1, 2023 · 5 comments
Closed

Issue: Unnamed docker volumes #478

max06 opened this issue Apr 1, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@max06
Copy link

max06 commented Apr 1, 2023

(This is probably in the wrong repository - I'm not sure where to put this, please move it to the right people)

Specifying a mountpoint without a source-attribute creates a named volume named undefined. According to the spec (and to the same mount used in a devcontainer.json), it should create an unnamed volume.

Example:

    "mounts": [
        {
            "target": "/home/vscode/.vscode-server",
            "type": "volume"
        }
    ]

ends up as:

            {
                "Type": "volume",
                "Name": "undefined",
                "Source": "/var/lib/docker/volumes/undefined/_data",
                "Destination": "/home/vscode/.vscode-server",
                "Driver": "local",
                "Mode": "z",
                "RW": true,
                "Propagation": ""
            }
@samruddhikhandale samruddhikhandale transferred this issue from devcontainers/spec Apr 3, 2023
@samruddhikhandale
Copy link
Member

Hi 👋

Thanks for reporting.

According to this docker documentation,

Anonymous volumes are not given an explicit name when they are first mounted into a container, so Docker gives them a random name that is guaranteed to be unique within a given Docker host.

Looks like a bug in the CLI - if the source is omitted, then I expect a new volume with random unique name should be created for volume mounts. This should definitely not be named as undefined

@samruddhikhandale samruddhikhandale added the bug Something isn't working label Apr 3, 2023
@max06
Copy link
Author

max06 commented Apr 3, 2023

Hey 👋🏼

if I'm not mistaken, your schema requires the source-parameter for mounts. The bug in the cli might be its result.

@max06
Copy link
Author

max06 commented May 15, 2023

Hey there, @samruddhikhandale
not trying to pressure anyone, just interested if there's any progress on this topic?

@samruddhikhandale
Copy link
Member

Thanks for the ping on this issue.

@alexander-smolyakov can you help prioritize this issue? We need a bug fix for this issue for the dev containers CLI.

@max06
Copy link
Author

max06 commented Jun 29, 2023

Closing this issue as it was fixed in #537 some time ago. Passed right by me unnoticed.
I just tested it, I can confirm the solution, and I'm very happy about it! Thank you very much! 🥳

@max06 max06 closed this as completed Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants