Azure / azurefile-dockervolumedriver

Docker Volume Driver for Azure File Service over SMB/CIFS :whale:
Apache License 2.0
169 stars 55 forks source link

List volumes does not work #20

Closed sascha-egerer closed 8 years ago

sascha-egerer commented 8 years ago

The docker command docker volume ls does not list azurefile volumes.

ahmetb commented 8 years ago

@sascha-egerer Hi, could you explain the problem a bit more? Does it list native docker volumes or other driver volumes? What version of azurefile-dockervolumedriver and what version of docker are you running?

sascha-egerer commented 8 years ago

@ahmetalpbalkan Here are some more infos:

$ docker info
Containers: 2
 Running: 2
 Paused: 0
 Stopped: 0
Images: 3
Server Version: 1.10.2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 32
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local azurefile
 Network: bridge null host
Kernel Version: 3.19.0-49-generic
Operating System: Ubuntu 14.04.4 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.36 GiB
$ azurefile-dockervolumedriver --version
azurefile-dockervolumedriver version 0.1
$ docker volume ls
DRIVER              VOLUME NAME
local               952644d6e9b77a7902312130caf0b655909ae27c9412965f82c65df5b50ca230
$ docker volume create -d azurefile -o share=foo --name=foo
foo
$ docker volume ls
DRIVER              VOLUME NAME
local               952644d6e9b77a7902312130caf0b655909ae27c9412965f82c65df5b50ca230
$ sudo ls /etc/docker/plugins/azurefile/volumes/
foo

Deleting the Volume does also not work. The volume stays there...

$ docker volume rm foo
foo
$ sudo ls /etc/docker/plugins/azurefile/volumes/
foo
$ docker volume rm foo
foo
ahmetb commented 8 years ago

I can repro this. thanks.

ahmetb commented 8 years ago

This is now fixed in master, feel free to compile yourself until I release a new version. (I'm planning to include the mode/uid feature in v1.1 as soon as I fully understand the use case.) Thanks for reporting.

sascha-egerer commented 8 years ago

Tested it and it does work now.

ahmetb commented 8 years ago

@sascha-egerer thanks for testing.