[libvirt] [test-API][PATCH 2/2] Replace env.conf with global.conf in env_parser

Guannan Ren gren at redhat.com
Tue Aug 7 13:18:03 UTC 2012


On 08/06/2012 03:46 PM, Wayne Sun wrote:
>    Only update the expection info when parsing.
>
> Signed-off-by: Wayne Sun <gsun at redhat.com>
> ---
>   src/env_parser.py |   22 +++++++++++-----------
>   1 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/src/env_parser.py b/src/env_parser.py
> index f02af57..ddda727 100644
> --- a/src/env_parser.py
> +++ b/src/env_parser.py
> @@ -30,7 +30,7 @@ class Envparser(object):
>               self.cfg.read(configfile)
>           else:
>               raise exception.FileDoesNotExist(
> -            "env.conf is not a regular file or nonexist")
> +            "global.conf is not a regular file or nonexist")
>   
>       def has_section(self, section):
>           if self.cfg.has_section(section):
> @@ -46,7 +46,7 @@ class Envparser(object):
>                   return False
>           else:
>               raise exception.SectionDoesNotExist(
> -            "In env.conf, the section %s is nonexist" % section)
> +            "In global.conf, the section %s is nonexist" % section)
>   
>       def sections_list(self):
>           return self.cfg.sections()
> @@ -56,7 +56,7 @@ class Envparser(object):
>               return self.cfg.options(section)
>           else:
>               raise exception.SectionDoesNotExist(
> -            "In env.conf, the section %s is nonexist" % section)
> +            "In global.conf, the section %s is nonexist" % section)
>   
>       def get_value(self, section, option):
>           if self.has_section:
> @@ -64,17 +64,17 @@ class Envparser(object):
>                   return self.cfg.get(section, option)
>               else:
>                   raise exception.OptionDoesNotExist(
> -                "In env.conf, the option %s is nonexist" % option)
> +                "In global.conf, the option %s is nonexist" % option)
>           else:
>               raise exception.SectionDoesNotExist(
> -            "In env.conf, the section %s is nonexist" % section)
> +            "In global.conf, the section %s is nonexist" % section)
>   
>       def get_items(self, section):
>           if self.has_section:
>               return self.cfg.items(section)
>           else:
>               raise exception.SectionDoesNotExist(
> -            "In env.conf, the section %s is nonexist" % section)
> +            "In global.conf, the section %s is nonexist" % section)
>   
>       def add_section(self, section):
>           if self.has_section:
> @@ -91,10 +91,10 @@ class Envparser(object):
>                   return True
>               else:
>                   raise exception.OptionDoesNotExist(
> -                "In env.conf, the option %s is nonexist" % option)
> +                "In global.conf, the option %s is nonexist" % option)
>           else:
>               raise exception.SectionDoesNotExist(
> -            "In env.conf, the section %s is nonexist" % section)
> +            "In global.conf, the section %s is nonexist" % section)
>   
>       def remove_section(self, section):
>           if self.has_section:
> @@ -102,7 +102,7 @@ class Envparser(object):
>               return True
>           else:
>               raise exception.SectionDoesNotExist(
> -            "In env.conf, the section %s is nonexist" % section)
> +            "In global.conf, the section %s is nonexist" % section)
>   
>       def set_value(self, section, option, value):
>           if self.has_section:
> @@ -111,6 +111,6 @@ class Envparser(object):
>                   return True
>               else:
>                   raise exception.OptionDoesNotExist(
> -                "In env.conf, the option %s is nonexist" % option)
> +                "In global.conf, the option %s is nonexist" % option)
>           raise exception.SectionDoesNotExist(
> -        "In env.conf, the section %s is nonexist" % section)
> +        "In global.conf, the section %s is nonexist" % section)

      It should be global.cfg, but dont' worry let me change them.
      ACK and pushed.

      Guannan




More information about the libvir-list mailing list