diff options
author | Cole Robinson <crobinso@redhat.com> | 2013-10-06 12:55:21 -0400 |
---|---|---|
committer | Cole Robinson <crobinso@redhat.com> | 2013-10-06 13:00:30 -0400 |
commit | 102417ca0c9913505de156f889920b3d3f99fa62 (patch) | |
tree | 14c19d673495eb424dfa880880334b08fba788b2 /virtManager/addhardware.py | |
parent | 5945b8c4a1d7f571853dd9c2f422434aa4e224fb (diff) | |
download | virt-manager-102417ca0c9913505de156f889920b3d3f99fa62.tar.gz |
addhw: Make sure to set device defaults
Diffstat (limited to 'virtManager/addhardware.py')
-rw-r--r-- | virtManager/addhardware.py | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/virtManager/addhardware.py b/virtManager/addhardware.py index f2a8dc8a..37a2190d 100644 --- a/virtManager/addhardware.py +++ b/virtManager/addhardware.py @@ -1338,7 +1338,7 @@ class vmmAddHardware(vmmGObjectUI): # Page validation methods # ########################### - def validate(self, page_num): + def _validate(self, page_num): if page_num == PAGE_ERROR: self._dev = None return True @@ -1371,6 +1371,13 @@ class vmmAddHardware(vmmGObjectUI): elif page_num == PAGE_RNG: return self.validate_page_rng() + def validate(self, page_num): + ret = self._validate(page_num) + if ret is not False and self._dev: + self._dev.set_defaults() + self._dev.validate() + return ret + def validate_page_storage(self): bus, device = self.get_config_disk_target() cache = self.get_config_disk_cache() @@ -1420,9 +1427,6 @@ class vmmAddHardware(vmmGObjectUI): not self.vm.is_hvm() and util.is_blktap_capable(self.conn.get_backend())): disk.driver_name = virtinst.VirtualDisk.DRIVER_TAP - - disk.validate() - except Exception, e: return self.err.val_err(_("Storage parameter error."), e) @@ -1688,7 +1692,6 @@ class vmmAddHardware(vmmGObjectUI): try: self._dev = VirtualSmartCardDevice(conn) self._dev.mode = mode - self._dev.validate() except Exception, e: return self.err.val_err(_("Smartcard device parameter error"), e) |