[blivet:master] Add a getRaidLevel() convenience method to raid.py

Vratislav Podzimek vpodzime at redhat.com
Wed Jun 4 10:42:18 UTC 2014


On Mon, 2014-06-02 at 15:33 -0400, mulhern wrote:
> Signed-off-by: mulhern <amulhern at redhat.com>
> ---
>  blivet/devicefactory.py   |  4 ++--
>  blivet/devicelibs/raid.py | 14 ++++++++++++++
>  2 files changed, 16 insertions(+), 2 deletions(-)
> 
> diff --git a/blivet/devicefactory.py b/blivet/devicefactory.py
> index a5717c1..00e3bc4 100644
> --- a/blivet/devicefactory.py
> +++ b/blivet/devicefactory.py
> @@ -320,7 +320,7 @@ class DeviceFactory(object):
>          if value is None:
>              self._raid_level = None
>          else:
> -            self._raid_level = raid.ALL_LEVELS.raidLevel(value)
> +            self._raid_level = raid.getRaidLevel(value)
>  
>      @property
>      def container_raid_level(self):
> @@ -338,7 +338,7 @@ class DeviceFactory(object):
>          if value is None:
>              self._container_raid_level = None
>          else:
> -            self._container_raid_level = raid.ALL_LEVELS.raidLevel(value)
> +            self._container_raid_level = raid.getRaidLevel(value)
>      #
>      # methods related to device size and disk space requirements
>      #
> diff --git a/blivet/devicelibs/raid.py b/blivet/devicelibs/raid.py
> index 896fba4..b10a736 100644
> --- a/blivet/devicelibs/raid.py
> +++ b/blivet/devicelibs/raid.py
> @@ -631,3 +631,17 @@ class Dup(RAIDLevel):
>  
>  Dup = Dup()
>  ALL_LEVELS.addRaidLevel(Dup)
> +
> +def getRaidLevel(descriptor):
> +    """ Convenience function to return a RAID level for the descriptor.
> +
> +        :param object descriptor: a RAID level descriptor
> +        :rtype: RAIDLevel
> +        :returns: The RAIDLevel object for this descriptor
> +
> +        Note that descriptor may be any object that identifies a
> +        RAID level, including the RAID object itself.
> +
> +        Raises a RaidError is there is no RAID object for the descriptor.
> +    """
> +    return ALL_LEVELS.raidLevel(descriptor)
ACK.

-- 
Vratislav Podzimek

Anaconda Rider | RHCE | Red Hat, Inc. | Brno - Czech Republic




More information about the anaconda-patches mailing list