diff options
| author | Vishvananda Ishaya <vishvananda@gmail.com> | 2011-01-10 12:19:59 +0000 |
|---|---|---|
| committer | Tarmac <> | 2011-01-10 12:19:59 +0000 |
| commit | d968ce5b92181a9551f9cd36c899595ca726fb99 (patch) | |
| tree | 4a062fe4e927fc538d94e489808cdb6ad1b5e397 | |
| parent | c556d071ce02d258adeb7af4aada4536defed21a (diff) | |
| parent | 5eb5373af5dd8f062975b4c42e12f95569f7e41b (diff) | |
| download | nova-d968ce5b92181a9551f9cd36c899595ca726fb99.tar.gz nova-d968ce5b92181a9551f9cd36c899595ca726fb99.tar.xz nova-d968ce5b92181a9551f9cd36c899595ca726fb99.zip | |
Checks for existence of volume group using vgs instead of checking to see if /dev/nova-volumes exists. The dev is created by udev and isn't always there even if the volume group does exist.
| -rw-r--r-- | nova/volume/driver.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/nova/volume/driver.py b/nova/volume/driver.py index 477e0abf4..6bc925f3e 100644 --- a/nova/volume/driver.py +++ b/nova/volume/driver.py @@ -20,7 +20,6 @@ Drivers for volumes. """ -import os import time from nova import exception @@ -80,7 +79,9 @@ class VolumeDriver(object): def check_for_setup_error(self): """Returns an error if prerequisites aren't met""" - if not os.path.isdir("/dev/%s" % FLAGS.volume_group): + out, err = self._execute("sudo vgs --noheadings -o name") + volume_groups = out.split() + if not FLAGS.volume_group in volume_groups: raise exception.Error(_("volume group %s doesn't exist") % FLAGS.volume_group) |