From 5ce0a259ec3ca7a67a9534eba3b6499b4aa71f06 Mon Sep 17 00:00:00 2001 From: Mark McLoughlin Date: Mon, 4 Feb 2013 16:26:39 +0000 Subject: Be explicit about set_default() parameters We probably only want to support people using set_defaults() for a limited set of config options, so explicitly list those as parameters to the method. This is what we do for rpc and log options too. Change-Id: I3bfc5355e82eec17480b9547662e847dc0d74138 --- openstack/common/db/sqlalchemy/session.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'openstack/common') diff --git a/openstack/common/db/sqlalchemy/session.py b/openstack/common/db/sqlalchemy/session.py index a49a5a8..3b2d9df 100644 --- a/openstack/common/db/sqlalchemy/session.py +++ b/openstack/common/db/sqlalchemy/session.py @@ -317,9 +317,11 @@ _ENGINE = None _MAKER = None -def set_defaults(**kwargs): +def set_defaults(sql_connection, sqlite_db): """Set defaults for configuration variables.""" - cfg.set_defaults(sql_opts, **kwargs) + cfg.set_defaults(sql_opts, + sql_connection=sql_connection, + sqlite_db=sqlite_db) def get_session(autocommit=True, expire_on_commit=False): -- cgit