On Wed, Sep 24, 2008 at 03:43:33PM +0200, Michael Hanselmann wrote:
> 
> Future patches will add even more variables to the cluster config.
> Adding more parameters wouldn't make the function easier to use and
> it doesn't make sense to pass them to another function, as it's
> only done once in bootstrap.py on cluster initialization.
> ---
>  lib/bootstrap.py               |   18 +++++++++++++++-
>  lib/config.py                  |   41 ++++++++++++++-------------------------
>  test/ganeti.config_unittest.py |   17 +++++++++++++--
>  3 files changed, 45 insertions(+), 31 deletions(-)

LGTM.

iustin

Reply via email to