[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