From 6dcaa7bdc0a8b7e8114914dbf87b20492b861c70 Mon Sep 17 00:00:00 2001
From: Yulia Portnova <yportnova@mirantis.com>
Date: Mon, 30 Sep 2013 13:18:42 +0300
Subject: [PATCH] Fixed values passed to share_rpcapi.create_share.

Values passed to share_rpcapi.create_share were in incorrect order.

Fixed bug 1232126

Change-Id: Icaa9dcaea9d0ee7f325620380180994ce4e838dc
---
 manila/scheduler/simple.py | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/manila/scheduler/simple.py b/manila/scheduler/simple.py
index 1acb6ea61d..a19b2a1f90 100644
--- a/manila/scheduler/simple.py
+++ b/manila/scheduler/simple.py
@@ -64,8 +64,10 @@ class SimpleScheduler(chance.ChanceScheduler):
             self.share_rpcapi.create_share(context,
                                            updated_share,
                                            host,
-                                           snapshot_id,
-                                           None)
+                                           request_spec,
+                                           None,
+                                           snapshot_id=snapshot_id
+                                           )
             return None
 
         results = db.service_get_all_share_sorted(elevated)
@@ -83,7 +85,9 @@ class SimpleScheduler(chance.ChanceScheduler):
                 self.share_rpcapi.create_share(context,
                                                updated_share,
                                                service['host'],
-                                               snapshot_id, None)
+                                               request_spec,
+                                               None,
+                                               snapshot_id=snapshot_id)
                 return None
         msg = _("Is the appropriate service running?")
         raise exception.NoValidHost(reason=msg)