Sandro Bonazzola has uploaded a new change for review.

Change subject: packaging: setup: avoid traceback adding data domain
......................................................................

packaging: setup: avoid traceback adding data domain

Fixed the call to to ovirtsdk api using kwargs instead
of positional arguments ensuring the path is passed to
the correct variable.

Change-Id: I617b30d16e0eb28d60a3ce9feb32c47df38eff1d
Signed-off-by: Sandro Bonazzola <[email protected]>
---
M packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/10/17210/1

diff --git a/packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py 
b/packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py
index 765e9ac..60069f5 100644
--- a/packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py
+++ b/packaging/setup/plugins/ovirt-engine-setup/all-in-one/vdsm.py
@@ -248,9 +248,9 @@
         else:
             self.logger.debug('Adding local storage domain')
             storage = self._ovirtsdk_xml.params.Storage(
-                self.environment[
+                path=self.environment[
                     osetupcons.AIOEnv.STORAGE_DOMAIN_DIR
-                ].rstrip('/')
+                ].rstrip('/'),
             )
             storage.set_type('localfs')
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I617b30d16e0eb28d60a3ce9feb32c47df38eff1d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to