11644: Add DeviceID() to Volume interface.
authorTom Clegg <tom@curoverse.com>
Thu, 18 May 2017 17:26:36 +0000 (13:26 -0400)
committerTom Clegg <tom@curoverse.com>
Thu, 18 May 2017 17:26:36 +0000 (13:26 -0400)
services/keepstore/volume.go

index 07b497cc2afd4b70a185e7354001137d3146bd58..86b3da61fa34802b232987126e0c490f96fc7fe8 100644 (file)
@@ -232,6 +232,10 @@ type Volume interface {
        // EmptyTrash looks for trashed blocks that exceeded TrashLifetime
        // and deletes them from the volume.
        EmptyTrash()
+
+       // Return a globally unique ID of the underlying storage
+       // device if possible, otherwise "".
+       DeviceID() string
 }
 
 // A VolumeWithExamples provides example configs to display in the
@@ -320,17 +324,12 @@ func MakeRRVolumeManager(volumes []Volume) *RRVolumeManager {
        for _, v := range volumes {
                mnt := &VolumeMount{
                        UUID:        (*VolumeMount)(nil).generateUUID(),
-                       DeviceID:    "",
+                       DeviceID:    v.DeviceID(),
                        ReadOnly:    !v.Writable(),
                        Replication: v.Replication(),
                        Tier:        1,
                        volume:      v,
                }
-               if v, ok := v.(interface {
-                       DeviceID() string
-               }); ok {
-                       mnt.DeviceID = v.DeviceID()
-               }
                vm.iostats[v] = &ioStats{}
                vm.mounts = append(vm.mounts, mnt)
                vm.mountMap[mnt.UUID] = mnt