[node-patches] Change in ovirt-node[master]: storage volume page should make sure _model is set

rbarry at redhat.com rbarry at redhat.com
Wed Jul 23 04:21:20 UTC 2014


Ryan Barry has uploaded a new change for review.

Change subject: storage volume page should make sure _model is set
......................................................................

storage volume page should make sure _model is set

Don't check for the presence of values in the storage volume page
if _model is empty

Change-Id: I716a9ea86f52ab3282c4be55f8dcad0890df0583
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/installer/core/storage_vol_page.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/09/30609/1

diff --git a/src/ovirt/node/installer/core/storage_vol_page.py b/src/ovirt/node/installer/core/storage_vol_page.py
index db6969c..a73ebca 100644
--- a/src/ovirt/node/installer/core/storage_vol_page.py
+++ b/src/ovirt/node/installer/core/storage_vol_page.py
@@ -42,7 +42,8 @@
 
     def model(self):
         drive_changed = (self._model["storage.install_drive"] !=
-                         self.__get_install_drive())
+                         self.__get_install_drive() if self._model
+                         else None)
         if not self._model or drive_changed:
             self._model = self.__get_default_sizes()
             self._model["storage.data_size"] = "%s" %\


-- 
To view, visit http://gerrit.ovirt.org/30609
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I716a9ea86f52ab3282c4be55f8dcad0890df0583
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list