diff options
author | Thomas Hipp <thomashipp@gmail.com> | 2017-12-08 13:23:00 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-08 13:23:00 +0100 |
commit | 79ad36eed30a9664b31709b599d984f365060190 (patch) | |
tree | cf2b3e358db8741ea666fbf0c79a0709c0656dfe /libvirt/resource_libvirt_volume.go | |
parent | a20d2a92668b2d5e077267c50074bf853ef371dd (diff) | |
parent | 96f4032f04b2a625b12d188cae4315887bf9fc66 (diff) | |
download | terraform-provider-libvirt-79ad36eed30a9664b31709b599d984f365060190.tar terraform-provider-libvirt-79ad36eed30a9664b31709b599d984f365060190.tar.gz |
Merge pull request #256 from monstermunchkin/unique-volume-name
check uniqueness of storage volume name
Diffstat (limited to 'libvirt/resource_libvirt_volume.go')
-rw-r--r-- | libvirt/resource_libvirt_volume.go | 24 |
1 files changed, 7 insertions, 17 deletions
diff --git a/libvirt/resource_libvirt_volume.go b/libvirt/resource_libvirt_volume.go index 7a7f6a40..4f9c3322 100644 --- a/libvirt/resource_libvirt_volume.go +++ b/libvirt/resource_libvirt_volume.go @@ -105,12 +105,15 @@ func resourceLibvirtVolumeCreate(d *schema.ResourceData, meta interface{}) error return pool.Refresh(0) }) - volumeDef := newDefVolume() - - if name, ok := d.GetOk("name"); ok { - volumeDef.Name = name.(string) + // Check whether the storage volume already exists. Its name needs to be + // unique. + if _, err := pool.LookupStorageVolByName(d.Get("name").(string)); err == nil { + return fmt.Errorf("storage volume '%s' already exists", d.Get("name").(string)) } + volumeDef := newDefVolume() + volumeDef.Name = d.Get("name").(string) + volumeFormat := "qcow2" if _, ok := d.GetOk("format"); ok { volumeFormat = d.Get("format").(string) @@ -136,19 +139,6 @@ func resourceLibvirtVolumeCreate(d *schema.ResourceData, meta interface{}) error return fmt.Errorf("'base_volume_name' can't be specified when also 'source' is given") } - // Check if we already have this image in the pool - if len(volumeDef.Name) > 0 { - if v, err := pool.LookupStorageVolByName(volumeDef.Name); err != nil { - log.Printf("could not find image %s in pool %s", volumeDef.Name, poolName) - } else { - volume = v - volumeDef, err = newDefVolumeFromLibvirt(volume) - if err != nil { - return fmt.Errorf("could not get a volume definition from XML for %s: %s", volumeDef.Name, err) - } - } - } - if img, err = newImage(source.(string)); err != nil { return err } |