On Thu, Nov 06, 2014 at 12:30:35AM +0200, Alex Pyrgiotis wrote: > From: Ilias Tsitsimpis <[email protected]> > > Implement 'GetAllDisksInfo' function which returns the configuration of > all disks. > > Signed-off-by: Ilias Tsitsimpis <[email protected]> > Signed-off-by: Alex Pyrgiotis <[email protected]> > > diff --git a/lib/config.py b/lib/config.py > index 893a19c..09ea190 100644 > --- a/lib/config.py > +++ b/lib/config.py > @@ -587,6 +587,35 @@ class ConfigWriter(object): > """ > return self._UnlockedGetDiskInfo(disk_uuid) > > + def _UnlockedGetDiskList(self): > + """Get the list of disks. > + > + @return: array of disks, ex. ['disk2-uuid', 'disk1-uuid'] > + > + """ > + return self._ConfigData().disks.keys() > + > + @_ConfigSync(shared=1) > + def GetAllDisksInfo(self): > + """Get the configuration of all disks. > + > + This is a simple wrapper over L{_UnlockedGetAllDisksInfo}. > + > + """ > + return self._UnlockedGetAllDisksInfo() > + > + def _UnlockedGetAllDisksInfo(self): > + """Get the configuration of all disks. > + > + @rtype: dict > + @return: dict of (disk, disk_info), where disk_info is what > + would GetDiskInfo return for the node > + > + """ > + my_dict = dict([(disk_uuid, self._UnlockedGetDiskInfo(disk_uuid)) > + for disk_uuid in self._UnlockedGetDiskList()]) > + return my_dict > + > def _AllInstanceNodes(self, inst_uuid): > """Compute the set of all disk-related nodes for an instance. >
LGTM > -- > 1.7.10.4 > -- Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
