You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by sa...@apache.org on 2015/03/30 07:26:20 UTC

git commit: updated refs/heads/master to dede9d9

Repository: cloudstack
Updated Branches:
  refs/heads/master b6d2b62aa -> dede9d94f


CLOUDSTACK-8343:modifying update method in Configurations to take **kwargs


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/dede9d94
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/dede9d94
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/dede9d94

Branch: refs/heads/master
Commit: dede9d94fe302b9a63ae82357cb14cd9355f9b3b
Parents: b6d2b62
Author: prashant kumar mishra <pr...@citrix.com>
Authored: Wed Mar 25 11:43:05 2015 +0530
Committer: sanjeev <sa...@apache.org>
Committed: Mon Mar 30 10:55:31 2015 +0530

----------------------------------------------------------------------
 test/integration/component/maint/test_bugs.py | 4 ++--
 tools/marvin/marvin/lib/base.py               | 5 ++---
 2 files changed, 4 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/dede9d94/test/integration/component/maint/test_bugs.py
----------------------------------------------------------------------
diff --git a/test/integration/component/maint/test_bugs.py b/test/integration/component/maint/test_bugs.py
index 89b3bc9..96d0f77 100644
--- a/test/integration/component/maint/test_bugs.py
+++ b/test/integration/component/maint/test_bugs.py
@@ -164,8 +164,8 @@ class Test42xBugsMgmtSvr(cloudstackTestCase):
         # "apply.allocation.algorithm.to.pods" back to false
         Configurations.update(
             self.apiClient,
-            "apply.allocation.algorithm.to.pods",
-            "false"
+            name="apply.allocation.algorithm.to.pods",
+            value="false"
         )
         # TODO:cleanup: Restart management server
         return

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/dede9d94/tools/marvin/marvin/lib/base.py
----------------------------------------------------------------------
diff --git a/tools/marvin/marvin/lib/base.py b/tools/marvin/marvin/lib/base.py
index 7f609e7..3cb04fe 100755
--- a/tools/marvin/marvin/lib/base.py
+++ b/tools/marvin/marvin/lib/base.py
@@ -3636,12 +3636,11 @@ class Configurations:
     """Manage Configuration"""
 
     @classmethod
-    def update(cls, apiclient, name, value=None):
+    def update(cls, apiclient, **kwargs):
         """Updates the specified configuration"""
 
         cmd = updateConfiguration.updateConfigurationCmd()
-        cmd.name = name
-        cmd.value = value
+        [setattr(cmd, k, v) for k, v in kwargs.items()]
         apiclient.updateConfiguration(cmd)
 
     @classmethod