Ayal Baron has posted comments on this change.
Change subject: Read pool metadata once in StoragePool.getInfo() ......................................................................
Patch Set 4:
(1 comment)
.................................................... File vdsm/storage/sp.py Line 1495: poolInfo = { Line 1496: 'type': msdInfo['type'], Line 1497: 'name': pmd[PMDK_POOL_DESCRIPTION], Line 1498: 'domains': domainListEncoder(pmd[PMDK_DOMAINS]), Line 1499: 'master_uuid': self.masterDomain.sdUUID, nit: why change this from msdUUID? (e.g. msdUUID could be a property and in any event is shorter). Anyway, function should be consistent. Line 1500: 'master_ver': pmd[PMDK_MASTER_VER], Line 1501: 'lver': pmd[PMDK_LVER], Line 1502: 'spm_id': pmd[PMDK_SPM_ID], Line 1503: 'pool_status': 'uninitialized',