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 <prashantkumat.mis...@citrix.com> Authored: Wed Mar 25 11:43:05 2015 +0530 Committer: sanjeev <sanj...@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
