Zhou Zheng Sheng has posted comments on this change.
Change subject: storage functional test with multiple storage domains and images
......................................................................
Patch Set 6: (5 inline comments)
YAML needs a new dependency on third party lib. If the conf in the code
continues to grow and becomes more complicated, I will try to use YAML. For now
I think plain python dict is OK.
....................................................
File tests/functional/xmlrpcTests.py
Line 31:
Line 32: from vdsm.config import config
Line 33: from vdsm.constants import VDSM_USER, VDSM_GROUP
Line 34: from storage.sd import BLANK_UUID
Line 35: from storage.sd import name2class
Done
Line 36: from storage.sd import name2type as sdname2type
Line 37: from storage.volume import name2type as volname2type
Line 38: from vdsm import vdscli
Line 39: from storage.misc import execCmd
Line 77: os.chmod(path, 0644)
Line 78: return path
Line 79:
Line 80:
Line 81: def rollbackManager(transaction):
Done
Line 82: @contextmanager
Line 83: def wrapper(*args, **kwargs):
Line 84: rollback = []
Line 85: exception = None
Line 85: exception = None
Line 86: traceback = None
Line 87: try:
Line 88: yield transaction(rollback=rollback, *args, **kwargs)
Line 89: except Exception, e:
Done
Line 90: # keep the original exception and traceback info
Line 91: exception = e
Line 92: traceback = sys.exc_info()[2]
Line 93: finally:
Line 209: with _localfsStore(conf['conn']),
self._vdsmStorageLayout(conf):
Line 210: pass
Line 211:
Line 212: @rollbackManager
Line 213: def _vdsmStorageLayout(self, conf, rollback):
Done
Line 214: connections = conf['conn']
Line 215: storageDomains = conf['sd']
Line 216: storagePools = conf['sp']
Line 217: images = conf['img']
Line 334: storageLayouts = \
Line 335: {'localfs':
Line 336: {'conn': {'53acd629-47e6-42d8-ba99-cd0b12ff0e1e':
Line 337: {'type': 'localfs',
Line 338: 'params': {'path': '/tmp/teststorage0'}},
Done
Line 339: '87e618fe-587c-4704-a9f8-9fd9321fd907':
Line 340: {'type': 'localfs',
Line 341: 'params': {'path': '/tmp/teststorage1'}}},
Line 342: 'sd': {"def32ac7-1234-1234-8a8c-1c887333fe65":
--
To view, visit http://gerrit.ovirt.org/8182
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8287046046460f399f180d19e0717a91419297f8
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: Ryan Harper <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
