vdsm-patches August 2012

vdsm-patches@lists.fedorahosted.org

61 participants
390 discussions

Change in vdsm[master]: Move uninstall preun section to vdsm-unregister
by lilei＠linux.vnet.ibm.com 12 Jan '14

12 Jan '14

Lei Li has uploaded a new change for review. Change subject: Move uninstall preun section to vdsm-unregister ...................................................................... Move uninstall preun section to vdsm-unregister The specfile vdsm.spec.in has very long %post and %preun sections, move the install/uninstall hooks into vdsm-tool. This patch move unregister to a script which will be encapsulated into a vdsm-tool function stop-vdsm-service. Change-Id: I4e7b5dc969dfb51e6880b9bb209a363609f5e123 Signed-off-by: Lei Li <lilei(a)linux.vnet.ibm.com> --- M vdsm.spec.in M vdsm/Makefile.am A vdsm/vdsm-unregister.in 3 files changed, 36 insertions(+), 42 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/26/4526/1 -- To view, visit http://gerrit.ovirt.org/4526 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4e7b5dc969dfb51e6880b9bb209a363609f5e123 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Lei Li <lilei(a)linux.vnet.ibm.com>

9 35

Change in vdsm[master]: Adjust width for usage_command in vdsm-tool
by lilei＠linux.vnet.ibm.com 07 Jan '14

07 Jan '14

Lei Li has uploaded a new change for review. Change subject: Adjust width for usage_command in vdsm-tool ...................................................................... Adjust width for usage_command in vdsm-tool The default value of width is 70 characters. It may not be long enough for most of the usage_command comments. So adjust it to add flexibility. Change-Id: I800384e77452a76f651b73205b3335905274077c Signed-off-by: Lei Li <lilei(a)linux.vnet.ibm.com> --- M vdsm-tool/vdsm-tool 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/48/4648/1 -- To view, visit http://gerrit.ovirt.org/4648 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I800384e77452a76f651b73205b3335905274077c Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Lei Li <lilei(a)linux.vnet.ibm.com>

4 5

Change in vdsm[master]: [WIP] Move vm logic to it's own module
by smizrahi＠redhat.com 29 Dec '13

29 Dec '13

Saggi Mizrahi has uploaded a new change for review. Change subject: [WIP] Move vm logic to it's own module ...................................................................... [WIP] Move vm logic to it's own module Change-Id: Icc54c1d0ab7ad535d825dcd72fd2f4583b690f44 Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com> --- M vdsm/API.py M vdsm/Makefile.am M vdsm/clientIF.py A vdsm/vmContainer.py 4 files changed, 134 insertions(+), 79 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/23/7423/1 diff --git a/vdsm/API.py b/vdsm/API.py index e132fb9..818e00c 100644 --- a/vdsm/API.py +++ b/vdsm/API.py @@ -167,34 +167,36 @@ self.log.error("Error restoring VM parameters", exc_info=True) - requiredParams = ['vmId', 'memSize', 'display'] - for param in requiredParams: + for param in ['vmId', 'memSize', 'display']: if param not in vmParams: - self.log.error('Missing required parameter %s' % (param)) - return {'status': {'code': errCode['MissParam'] - ['status']['code'], - 'message': 'Missing required' + \ - 'parameter %s' % (param)}} + return {'status': { + 'code': errCode['MissParam']['status']['code'], + 'message': 'Missing required parameter %s' % (param,)} + } try: storage.misc.validateUUID(vmParams['vmId']) except: - return {'status': {'code': errCode['MissParam'] - ['status']['code'], - 'message': 'vmId must be a valid UUID'}} + return {'status': { + 'code': errCode['MissParam']['status']['code'], + 'message': 'vmId must be a valid UUID'} + } if vmParams['memSize'] == 0: - return {'status': {'code': errCode['MissParam'] - ['status']['code'], - 'message': 'Must specify nonzero memSize'}} + return {'status': { + 'code': errCode['MissParam']['status']['code'], + 'message': 'Must specify nonzero memSize'} + } if vmParams.get('boot') == 'c' and not 'hda' in vmParams \ and not vmParams.get('drives'): - return {'status': {'code': errCode['MissParam'] - ['status']['code'], - 'message': 'missing boot disk'}} + return {'status': { + 'code': errCode['MissParam']['status']['code'], + 'message': 'missing boot disk'} + } if 'vmType' not in vmParams: vmParams['vmType'] = 'kvm' - elif vmParams['vmType'] == 'kvm': + + if vmParams['vmType'] == 'kvm': if 'kvmEnable' not in vmParams: vmParams['kvmEnable'] = 'true' @@ -205,25 +207,27 @@ vmParams['volatileFloppy'] = True if caps.osversion()['name'] == caps.OSName.UNKNOWN: - return {'status': {'code': errCode['createErr'] - ['status']['code'], - 'message': 'Unknown host operating system'}} + return {'status': { + 'code': errCode['createErr']['status']['code'], + 'message': 'Unknown host operating system'} + } if 'sysprepInf' in vmParams: if not self._createSysprepFloppyFromInf(vmParams['sysprepInf'], vmParams['floppy']): - return {'status': {'code': errCode['createErr'] - ['status']['code'], - 'message': 'Failed to create ' - 'sysprep floppy image. ' - 'No space on /tmp?'}} - return errCode['createErr'] + return {'status': { + 'code': errCode['createErr']['status']['code'], + 'message': 'Failed to create sysprep floppy image. ' + 'No space on /tmp?'} + } if vmParams.get('display') not in ('vnc', 'qxl', 'qxlnc', 'local'): - return {'status': {'code': errCode['createErr'] - ['status']['code'], - 'message': 'Unknown display type %s' - % vmParams.get('display')}} + return {'status': { + 'code': errCode['createErr']['status']['code'], + 'message': 'Unknown display type %s' % + vmParams.get('display')} + } + if 'nicModel' not in vmParams: vmParams['nicModel'] = config.get('vars', 'nic_model') vmParams['displayIp'] = self._getNetworkIp(vmParams.get( diff --git a/vdsm/Makefile.am b/vdsm/Makefile.am index d4c52ed..574d762 100644 --- a/vdsm/Makefile.am +++ b/vdsm/Makefile.am @@ -53,6 +53,7 @@ tc.py \ vdsmDebugPlugin.py \ vmChannels.py \ + vmContainer.py \ vm.py \ $(NULL) diff --git a/vdsm/clientIF.py b/vdsm/clientIF.py index b9a4913..b2438fb 100644 --- a/vdsm/clientIF.py +++ b/vdsm/clientIF.py @@ -24,6 +24,7 @@ import pickle from xml.dom import minidom import uuid +import errno from storage.dispatcher import Dispatcher from storage.hsm import HSM @@ -40,9 +41,9 @@ import configNetwork import caps from vmChannels import Listener -from libvirtvm import LibvirtVm import blkid import supervdsm +import vmContainer try: import gluster.api as gapi _glusterEnabled = True @@ -66,7 +67,6 @@ :param log: a log object to be used for this object's logging. :type log: :class:`logging.Logger` """ - self.vmContainerLock = threading.Lock() self._networkSemaphore = threading.Semaphore() self._shutdownSemaphore = threading.Semaphore() self.log = log @@ -82,7 +82,7 @@ else: self.gluster = None try: - self.vmContainer = {} + self.vmContainer = vmContainer.VmContainer() ifids = netinfo.nics() + netinfo.bondings() ifrates = map(netinfo.speed, ifids) self._hostStats = utils.HostStatsThread( @@ -328,24 +328,19 @@ return res['status']['code'] def createVm(self, vmParams): - self.vmContainerLock.acquire() - self.log.info("vmContainerLock acquired by vm %s", - vmParams['vmId']) try: - if 'recover' not in vmParams: - if vmParams['vmId'] in self.vmContainer: - self.log.warning('vm %s already exists' % - vmParams['vmId']) - return errCode['exist'] - vm = LibvirtVm(self, vmParams) - self.vmContainer[vmParams['vmId']] = vm - finally: - container_len = len(self.vmContainer) - self.vmContainerLock.release() - vm.run() - self.log.debug("Total desktops after creation of %s is %d" % - (vmParams['vmId'], container_len)) - return {'status': doneCode, 'vmList': vm.status()} + vm = self.vmContainer.createVm(vmParams) + return {'status': doneCode, 'vmList': vm.status()} + except vmContainer.VmContainerError as e: + if e.errno == errno.EEXIST: + return errCode['exist'] + return + + def waitForShutdown(self, timeout=None): + self.shutdownEvent(timeout) + # Some versions of python have a bug where wait doesn't + # really return the correct value. + return self.shutdownEvent.isSet() def _recoverExistingVms(self): try: @@ -364,10 +359,11 @@ 'process with id: %s', vmId, exc_info=True) - while (self._enabled and - 'WaitForLaunch' in [v.lastStatus for v in - self.vmContainer.values()]): - time.sleep(1) + vmList = self.vmContainer.getVMs() + while ('WaitForLaunch' in (v.lastStatus for v in vmList)): + if self.waitForShutdown(1): + break + self._cleanOldFiles() self._recovery = False @@ -375,20 +371,23 @@ # and then prepare all volumes. # Actually, we need it just to get the resources for future # volumes manipulations - while self._enabled and self.vmContainer and \ - not self.irs.getConnectedStoragePoolsList()['poollist']: - time.sleep(5) + if len(vmList) > 0: + while not self.irs.getConnectedStoragePoolsList()['poollist']: + if self.waitForShutdown(5): + break - for vmId, vmObj in self.vmContainer.items(): - # Let's recover as much VMs as possible + # Let's recover as many VMs as possible + for vmObj in vmList: + # Do not prepare volumes when system goes down + if self.waitForShutdown(0): + break + try: - # Do not prepare volumes when system goes down - if self._enabled: - vmObj.preparePaths( - vmObj.getConfDevices()[vm.DISK_DEVICES]) + devices = vmObj.getConfDevices()[vm.DISK_DEVICES] + vmObj.preparePaths(devices) except: - self.log.error("Vm %s recovery failed", - vmId, exc_info=True) + self.log.error("Vm '%s' recovery failed", vmObj.id, + exc_info=True) except: self.log.error("Vm's recovery failed", exc_info=True) @@ -452,27 +451,27 @@ return recoveryFile except: self.log.debug("Error recovering VM", exc_info=True) - return None def _cleanOldFiles(self): for f in os.listdir(constants.P_VDSM_RUN): try: vmId, fileType = f.split(".", 1) - if fileType in ["guest.socket", "monitor.socket", "pid", + if fileType not in ["guest.socket", "monitor.socket", "pid", "stdio.dump", "recovery"]: - if vmId in self.vmContainer: - continue - if f == 'vdsmd.pid': - continue - if f == 'respawn.pid': - continue - if f == 'svdsm.pid': - continue - if f == 'svdsm.sock': - continue - else: continue - self.log.debug("removing old file " + f) + + if self.vmContainer.hasVM(vmId): + continue + + #FIXME: Move VM crud to a subdir so we don't need a + # whitelist. I remember spending a considerable amount of + # time while writing Super VDSM trying to figure out why my + # pid file was disappearing! + if f in ('vdsmd.pid', 'respawn.pid', + 'svdsm.pid', 'svdsm.sock'): + continue + + self.log.debug("Removing old file '%s'", f) utils.rmFile(constants.P_VDSM_RUN + f) except: pass diff --git a/vdsm/vmContainer.py b/vdsm/vmContainer.py new file mode 100644 index 0000000..f15b8eb --- /dev/null +++ b/vdsm/vmContainer.py @@ -0,0 +1,51 @@ +import logging +import threading +from libvirtvm import LibvirtVm +import os +import errno + + +class VmContainerError(OSError): + def __init__(self, errno): + OSError.__init__(self, errno, os.strerror(errno)) + + +class VmContainer(object): + log = logging.getLogger("VmContainer") + + def __init__(self): + self._syncroot = threading.Lock() + self._vms = {} + + def createVm(self, vmParams): + self.log.info("vmContainerLock acquired by vm %s", + vmParams['vmId']) + + with self._syncRoot: + if 'recover' not in vmParams: + if vmParams['vmId'] in self._vms: + self.log.warning('vm %s already exists' % vmParams['vmId']) + + raise VmContainerError(errno.EEXIST) + + vm = LibvirtVm(self, vmParams) + self._vms[vmParams['vmId']] = vm + + container_len = len(self._vms) + + vm.run() + self.log.debug("Total desktops after creation of %s is %d" % + (vmParams['vmId'], container_len)) + + return vm + + def getVMs(self): + return self._vms.values() + + def hasVM(self, vmId): + with self._syncRoot: + return vmId in self._vms + + def get(self, vmId): + with self._syncRoot: + return self._vms.get(vmId) -- To view, visit http://gerrit.ovirt.org/7423 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Icc54c1d0ab7ad535d825dcd72fd2f4583b690f44 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>

4 10

Change in vdsm[master]: add verb for vdsm to tuneCpu params
by lvroyce＠linux.vnet.ibm.com 12 Dec '13

12 Dec '13

Royce Lv has uploaded a new change for review. Change subject: add verb for vdsm to tuneCpu params ...................................................................... add verb for vdsm to tuneCpu params Engine and Mom need an interface to cap the max cpu and share. So add this verb to vdsm. tuneCpu: Dynamically tune cpu absolute max proportion and relative share. Input: vmId: VM to be tuned max_proportion=<value>: absolute max proportion for each vCPU can consume -1:no limit,[0.001,1]:valid proportion relative_share=<value>:relative share of CPU compared with other VMs output: success: return doneCode failure: return errCode tunecpuErr. Change-Id: I09e02e09ef06ad6de45be75c3f2f913a3025750f Signed-off-by: Royce Lv<lvroyce(a)linux.vnet.ibm.com> --- M vdsm/API.py M vdsm/BindingXMLRPC.py M vdsm/define.py M vdsm/libvirtvm.py M vdsm/storage/misc.py M vdsm_cli/vdsClient.py 6 files changed, 97 insertions(+), 8 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/92/7492/1 diff --git a/vdsm/API.py b/vdsm/API.py index 827f73b..f0c47d1 100644 --- a/vdsm/API.py +++ b/vdsm/API.py @@ -26,6 +26,7 @@ import time import threading import logging +import math from vdsm import utils from clientIF import clientIF @@ -37,6 +38,7 @@ import storage.volume import storage.sd import storage.image +import storage.storage_exception as se from vdsm.define import doneCode, errCode, Kbytes, Mbytes import caps from vdsm.config import config @@ -555,6 +557,33 @@ message = USER_SHUTDOWN_MESSAGE return v.shutdown(delay, message) + def tuneCpu(self, tuneParams): + try: + v = self._cif.vmContainer[self._UUID] + if 'relative_share' in tuneParams: + tuneParams['cpu_shares'] = storage.misc.validateN( + tuneParams['relative_share'], 'relative_share') + del(tuneParams['relative_share']) + if 'max_proportion' in tuneParams: + max = storage.misc.validateFloat( + tuneParams['max_proportion'], 'max_proportion') + tuneParams['vcpu_period'] = 1000000 + if max > 1: + raise se.InvalidParameterException('max_proportion', max) + elif max >= 0.001: + tuneParams['vcpu_quota'] = int( + round(tuneParams['vcpu_period'] * max)) + else: + tuneParams['vcpu_quota'] = int(math.floor(max)) + del(tuneParams['max_proportion']) + del(tuneParams['vmId']) + except KeyError: + return errCode['noVM'] + except se.InvalidParameterException: + return errCode['tunecpuErr'] + + return v.tuneCpu(tuneParams) + def _createSysprepFloppyFromInf(self, infFileBinary, floppyImage): try: rc, out, err = utils.execCmd([constants.EXT_MK_SYSPREP_FLOPPY, diff --git a/vdsm/BindingXMLRPC.py b/vdsm/BindingXMLRPC.py index 826c125..37b1f60 100644 --- a/vdsm/BindingXMLRPC.py +++ b/vdsm/BindingXMLRPC.py @@ -265,6 +265,10 @@ vm = API.VM(vmId) return vm.snapshot(snapDrives) + def vmTuneCpu(self, params): + vm = API.VM(params['vmId']) + return vm.tuneCpu(params) + def vmMerge(self, vmId, mergeDrives): vm = API.VM(vmId) return vm.merge(mergeDrives) @@ -765,7 +769,8 @@ (self.vmHotplugDisk, 'hotplugDisk'), (self.vmHotunplugDisk, 'hotunplugDisk'), (self.vmHotplugNic, 'hotplugNic'), - (self.vmHotunplugNic, 'hotunplugNic')) + (self.vmHotunplugNic, 'hotunplugNic'), + (self.vmTuneCpu, 'tuneCpu')) def getIrsMethods(self): return ((self.domainActivate, 'activateStorageDomain'), diff --git a/vdsm/define.py b/vdsm/define.py index 049d36c..d41d467 100644 --- a/vdsm/define.py +++ b/vdsm/define.py @@ -117,6 +117,9 @@ 'momErr': {'status': {'code': 54, 'message': 'Failed to set mom policy'}}, + 'tunecpuErr': {'status': + {'code': 55, + 'message': 'Failed to Tune CPU'}}, 'recovery': {'status': {'code': 99, 'message': diff --git a/vdsm/libvirtvm.py b/vdsm/libvirtvm.py index e95e94b..59c7ec6 100644 --- a/vdsm/libvirtvm.py +++ b/vdsm/libvirtvm.py @@ -1371,13 +1371,17 @@ if self._initTimePauseCode == 'ENOSPC': self.cont() self.conf['pid'] = self._getPid() - - nice = int(self.conf.get('nice', '0')) - nice = max(min(nice, 19), 0) - try: - self._dom.setSchedulerParameters({'cpu_shares': (20 - nice) * 51}) - except: - self.log.warning('failed to set Vm niceness', exc_info=True) + #do not trying to change cpu_shares for recovery and restore flows + if not 'recover' in self.conf and not 'restoreState' in self.conf: + nice = int(self.conf.get('nice', '0')) + nice = max(min(nice, 19), 0) + cpuShare = (20 - nice) * 51 + try: + self._dom.setSchedulerParameters({'cpu_shares': cpuShare}) + except: + self.log.warning('failed to set Vm niceness', exc_info=True) + self.conf['cpu_shares'] = cpuShare + self.saveState() def _run(self): self.log.info("VM wrapper has started") @@ -1931,6 +1935,23 @@ (snapFlags & libvirt.VIR_DOMAIN_SNAPSHOT_CREATE_QUIESCE == libvirt.VIR_DOMAIN_SNAPSHOT_CREATE_QUIESCE)} + def tuneCpu(self, tuneParams): + try: + self._dom.setSchedulerParameters(tuneParams) + self.conf['cpu_shares'] = tuneParams.get('cpu_shares') + self.conf['vcpu_quota'] = tuneParams.get('vcpu_quota') + self.conf['vcpu_period'] = tuneParams.get('vcpu_period') + self.saveState() + return {'status': doneCode} + except libvirt.libvirtError, e: + self.log.error("TuneCpu failed:%s", + e.message, exc_info=True) + return errCode['tunecpuErr'] + except LookupError: + self.log.error('TuneCpu failed: unrecongnized params', + exc_info=True) + return errCode['tunecpuErr'] + def _runMerge(self): for mergeStatus in self.conf.get('liveMerge', []): if mergeStatus['status'] != MERGESTATUS.NOT_STARTED: diff --git a/vdsm/storage/misc.py b/vdsm/storage/misc.py index 161726b..1f5ed1a 100644 --- a/vdsm/storage/misc.py +++ b/vdsm/storage/misc.py @@ -503,6 +503,13 @@ return n +def validateFloat(floatNumber, name): + try: + return float(floatNumber) + except: + raise se.InvalidParameterException(floatNumber, name) + + def rotateFiles(directory, prefixName, gen, cp=False, persist=False): log.debug("dir: %s, prefixName: %s, versions: %s" % (directory, prefixName, gen)) diff --git a/vdsm_cli/vdsClient.py b/vdsm_cli/vdsClient.py index a164c20..ab0fa4c 100644 --- a/vdsm_cli/vdsClient.py +++ b/vdsm_cli/vdsClient.py @@ -204,6 +204,18 @@ return self.ExecAndExit(self.s.create(params)) + def tuneCpu(self, args): + params = {} + confLines = [] + params['vmId'] = args[0] + if len(args) > 1: + confLines.extend(args[1:]) + for line in confLines: + if '=' in line: + param, value = line.split("=", 1) + params[param] = value + return self.ExecAndExit(self.s.tuneCpu(params)) + def hotplugNic(self, args): nic = self._parseDriveSpec(args[1]) nic['type'] = 'interface' @@ -1802,6 +1814,18 @@ 'Optional additional parameters in dictionary format,' ' name:value,name:value' )), + 'tuneCpu': (serv.tuneCpu, + ('<vmId>' + '[max_proportion=value relative_share=value]', + 'Parameters list: r=required, o=optional', + 'r vmId: The vm to be tuned', + 'o max_proportion=<value>: ' + 'max proportion for each vCPU can be consumed, ' + '-1 is no limit,[0.001,1] is valid proportion', + 'o relative_share=<value>:relative share of CPU' + 'compared with other VMs', + 'Dynamically tune cpu absolute max proportion and relative share' + )), 'migrate': (serv.do_migrate, ('vmId=<id> method=<offline|online> src=<host:[port]> ' 'dst=<host:[port]>', -- To view, visit http://gerrit.ovirt.org/7492 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I09e02e09ef06ad6de45be75c3f2f913a3025750f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv <lvroyce(a)linux.vnet.ibm.com>

8 18

Change in vdsm[master]: adjust betterPopenTest.py
by xiawenc＠linux.vnet.ibm.com 08 Dec '13

08 Dec '13

Wenchao Xia has uploaded a new change for review. Change subject: adjust betterPopenTest.py ...................................................................... adjust betterPopenTest.py The main function in betterPopenTest is used to check result after popen, it should not include the betterPopen and test class code. If it include and the directory it lies on changes, the import will fail, which means additional python settings should be passed to betterPopen making things more complicate. Change-Id: I3059df6abb5c261f45720fc1a3b5266314c62e79 Signed-off-by: wenchao xia <xiawenc(a)linux.vnet.ibm.com> --- M tests/betterPopenTests.py 1 file changed, 50 insertions(+), 51 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/85/4585/1 -- To view, visit http://gerrit.ovirt.org/4585 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3059df6abb5c261f45720fc1a3b5266314c62e79 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia <xiawenc(a)linux.vnet.ibm.com>

7 13

Change in vdsm[master]: bz#831938 raise exception when error occurs in connectStorag...
by lvroyce＠linux.vnet.ibm.com 04 Dec '13

04 Dec '13

Royce Lv has uploaded a new change for review. Change subject: bz#831938 raise exception when error occurs in connectStorageServer ...................................................................... bz#831938 raise exception when error occurs in connectStorageServer connectStorageServer returned OK when error occurs: iscsi login fails, local directory lack of permission,etc change it to raise error to make early alarm Change-Id: I4c363f41d08669eec022797eb1eb440fe193f9c2 Signed-off-by: Royce Lv <lvroyce(a)linux.vnet.ibm.com> --- M vdsm/storage/hsm.py 1 file changed, 2 insertions(+), 8 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/37/6137/1 -- To view, visit http://gerrit.ovirt.org/6137 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4c363f41d08669eec022797eb1eb440fe193f9c2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv <lvroyce(a)linux.vnet.ibm.com>

3 3

Change in vdsm[master]: [WIP] rest-api: Model Networks
by lilei＠linux.vnet.ibm.com 26 Nov '13

26 Nov '13

Lei Li has uploaded a new change for review. Change subject: [WIP] rest-api: Model Networks ...................................................................... [WIP] rest-api: Model Networks API Implemented: ---------------- /api/networks * list, create, confirm and edit configuration for network Actions: /api/networks/add - Add a new network /api/networks/confirm - Mark the current network configurations as safe /api/networks/<id>/delete - DELETE a network /api/networks/<id>/edit - Add a new network and replace the old one API as TODO: --------------- /api/networks/<id...>/setup - Add/Edit/Remove configuration for networks Change-Id: Ic7eca74c16699a5cc35c4077192fb6ecfe706470 Signed-off-by: Lei Li <lilei(a)linux.vnet.ibm.com> --- M vdsm/rest/Controller.py M vdsm/rest/templates/Makefile.am M vdsm/rest/templates/api.xsd A vdsm/rest/templates/network.json.x A vdsm/rest/templates/network.xml.x A vdsm/rest/templates/networks.json.x A vdsm/rest/templates/networks.xml.x M vdsm/rest/templates/root.json.x M vdsm/rest/templates/root.xml.x M vdsm/rest/templates/rsdl.xml 10 files changed, 377 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/81/5881/1 -- To view, visit http://gerrit.ovirt.org/5881 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic7eca74c16699a5cc35c4077192fb6ecfe706470 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Lei Li <lilei(a)linux.vnet.ibm.com>

3 4

Change in vdsm[master]: Add deathSignal options to better peopen
by smizrahi＠redhat.com 17 Nov '13

17 Nov '13

Saggi Mizrahi has uploaded a new change for review. Change subject: Add deathSignal options to better peopen ...................................................................... Add deathSignal options to better peopen This allows VDSM to specify as signal that will be sent by the kernel to the child when VDSM dies. - Also fixes problem in the makefile that would cause make not to recompile the SO when the C file was changed Change-Id: I9f987129cea112e2a75d6f02477369417cc50dc7 Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com> --- M vdsm/betterPopen/Makefile.am M vdsm/betterPopen/__init__.py M vdsm/betterPopen/createprocess.c 3 files changed, 35 insertions(+), 5 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/41/6241/1 -- To view, visit http://gerrit.ovirt.org/6241 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I9f987129cea112e2a75d6f02477369417cc50dc7 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>

8 18

Change in vdsm[master]: Fix vm block stats lost after recovery
by wudxw＠linux.vnet.ibm.com 30 Oct '13

30 Oct '13

Mark Wu has uploaded a new change for review. Change subject: Fix vm block stats lost after recovery ...................................................................... Fix vm block stats lost after recovery We don't need call preparePaths() in the code path of vm recovery, because the 'path' has been prepared on vm creatation and saved in recovery file. But the flag _volumesPrepared still needs to be set to make the block stats can be collected. Change-Id: I59a90cf909a07967f642d348e47b4928f7516c61 Signed-off-by: Mark Wu <wudxw(a)linux.vnet.ibm.com> --- M vdsm/libvirtvm.py 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/56/6056/1 -- To view, visit http://gerrit.ovirt.org/6056 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I59a90cf909a07967f642d348e47b4928f7516c61 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu <wudxw(a)linux.vnet.ibm.com>

3 6

Change in vdsm[master]: minimalistic (incomplete) test for mkimage.py with fail for ...
by josef.pacula＠gmail.com 25 Oct '13

25 Oct '13

Josef Pacula has uploaded a new change for review. Change subject: minimalistic (incomplete) test for mkimage.py with fail for non-64base data ...................................................................... minimalistic (incomplete) test for mkimage.py with fail for non-64base data Change-Id: I1a7656359beb5d116ad1919e237fe084a1208d71 Signed-off-by: Josef Pacula <josef.pacula(a)gmail.com> --- M tests/Makefile.am A tests/mkimageTests.py M vdsm/mkimage.py 3 files changed, 20 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/75/6275/1 -- To view, visit http://gerrit.ovirt.org/6275 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1a7656359beb5d116ad1919e237fe084a1208d71 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Josef Pacula <josef.pacula(a)gmail.com>

4 10

← Newer
1
2
3
4
5
6
7
8
9
...
39
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

vdsm-patches August 2012