NetApp / netappdvp

A Docker volume plugin for NetApp storage
96 stars 33 forks source link

Fix issue of '_' not being valid for SF vol names #34

Closed j-griffith closed 8 years ago

j-griffith commented 8 years ago

This patch actually cleans up two things:

  1. SolidFire doesn't allow '' characters in volume names This is a problem because things like Compose can preface a volume name with 'project'

    Rather than try and map name changes, we'll instead just stuff the docker volume name into a volume attribute and use that as our handle going forward. That means ANY docker valid name is now a valid name for a SolidFire Volume.

  2. While working on this things got a bit wonky with the prefix NDVP by default uses a prefix for Volume and Snapshot names. Some times this is good, some times it's not. Rather than try and always keep that sort of thing tracked, we again just use a pure docker volume name in the attributes for things. So no matter what when we use the docker volume API and it's representation of the volume name things just work.

Addresses Github Issue: #32