Ido Barkan has uploaded a new change for review.
Change subject: net: tests: clean legacy bonds from kernelConfig
......................................................................
net: tests: clean legacy bonds from kernelConfig
VdsProxy netinfo is built from getVdsCapabilities which reports fake
legacy bonds in el6. Remove them from here to avoid lying to
KernelConfig
Change-Id: I094eb615bbe05e4661b34e8374d105401b8db8b6
Signed-off-by: Ido Barkan <ibarkan(a)redhat.com>
Label: ovirt-3.5-only
Bug-Url: https://bugzilla.redhat.com/1203422
---
M tests/functional/networkTests.py
1 file changed, 12 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/13/43513/1
diff --git a/tests/functional/networkTests.py b/tests/functional/networkTests.py
index c29ff01..39fa251 100644
--- a/tests/functional/networkTests.py
+++ b/tests/functional/networkTests.py
@@ -17,6 +17,7 @@
# Refer to the README and COPYING files for full details of the license
#
from contextlib import contextmanager
+import copy
from functools import wraps
import os.path
import json
@@ -352,7 +353,7 @@
return status, msg
def _assert_kernel_config_matches_running_config(self):
- netinfo = self.vdsm_net.netinfo
+ netinfo = self._clean_legacy_bonds(self.vdsm_net.netinfo)
kernel_config = KernelConfig(netinfo)
running_config = self.vdsm_net.config
# do not use KernelConfig.__eq__ to get better exception if something
@@ -361,6 +362,16 @@
self.assertEqual(normalized_config.networks, kernel_config.networks)
self.assertEqual(normalized_config.bonds, kernel_config.bonds)
+ def _clean_legacy_bonds(self, netinfo):
+ """VdsProxy netinfo is built from getVdsCapabilities which reports
+ fake legacy bonds in el6. Remove them from here to avoid lying to
+ KernelConfig
+ """
+ netinfo = copy.deepcopy(netinfo)
+ for bond in caps._REQUIRED_BONDINGS:
+ netinfo.bondings.pop(bond, None)
+ return netinfo
+
@cleanupNet
@permutations([[True], [False]])
@RequireDummyMod
--
To view, visit https://gerrit.ovirt.org/43513
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I094eb615bbe05e4661b34e8374d105401b8db8b6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Ido Barkan <ibarkan(a)redhat.com>
Francesco Romani has posted comments on this change.
Change subject: vm: events: reorganize events
......................................................................
Patch Set 2:
(6 comments)
https://gerrit.ovirt.org/#/c/43538/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 377: self._lastStatus = value
Line 378:
Line 379: def send_status_event(self, **kwargs):
Line 380: vm_status = self._getVmStatus()
Line 381: if vm_status != vmstatus.WAIT_FOR_LAUNCH:
> I believe this could go away, we never send this now when events are not be
We do but as side effect. Please see comment on line 1810 below.
Line 382: stats = {'status': vm_status}
Line 383: stats.update(kwargs)
Line 384: self._notify('VM_status', stats)
Line 385:
Line 1147: with self._confLock:
Line 1148: del self.conf['pauseCode']
Line 1149: except KeyError:
Line 1150: pass
Line 1151: self.send_status_event()
move into an else: block; even better: move outside the lock
Line 1152: return {'status': doneCode, 'output': ['']}
Line 1153: finally:
Line 1154: if not guestCpuLocked:
Line 1155: self._guestCpuLock.release()
Line 1165: self._setGuestCpuRunning(self._isDomainRunning(),
Line 1166: guestCpuLocked=True)
Line 1167: self._logGuestCpuStatus('pause')
Line 1168: self._lastStatus = afterState
Line 1169: self.send_status_event()
same
Line 1170: return {'status': doneCode, 'output': ['']}
Line 1171: finally:
Line 1172: if not guestCpuLocked:
Line 1173: self._guestCpuLock.release()
Line 1530: try:
Line 1531: self._migrationSourceThread.stop()
Line 1532: self._migrationSourceThread.status['status']['message'] = \
Line 1533: 'Migration process cancelled'
Line 1534: self.send_status_event()
> did you test this flow? I wonder if we were handling this flow correctly. W
For cancellation, libvirt raises ABORTED error, so we should end up anyway in _recover. So this is probably redundant. Will check and most likely remove.
Line 1535: return self._migrationSourceThread.status
Line 1536: except libvirt.libvirtError as e:
Line 1537: if e.get_error_code() == libvirt.VIR_ERR_OPERATION_INVALID:
Line 1538: return errCode['migCancelErr']
Line 1806: # case VDSM is upgraded from 3.4 to 3.5 on the host without
Line 1807: # rebooting it. Evident on, but not limited to, the HE case.
Line 1808: self._fixLegacyConf()
Line 1809:
Line 1810: self.send_status_event()
this is where we send MIGRATION_DESTINATION and the reason for the check for WAIT_FOR_LAUNCH in send_status_event. AFAIU MIGRATION_DESTINATION is useless, so I'll remove this one.
Line 1811:
Line 1812: self._devices = self.devMapFromDevSpecMap(dev_spec_map)
Line 1813:
Line 1814: # We should set this event as a last part of drives initialization
Line 3649: # Terminate the VM's creation thread.
Line 3650: self._incomingMigrationFinished.set()
Line 3651: self.guestAgent.stop()
Line 3652:
Line 3653: self.send_status_event()
> As per discussion with Omer this is not needed on engine side and in fact c
Indeed seems redundant or even counterproductive. Will remove.
Line 3654:
Line 3655: if self._dom:
Line 3656: result = self._destroyVmGraceful()
Line 3657: if result['status']['code']:
--
To view, visit https://gerrit.ovirt.org/43538
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I37c198478bcc289a713ac61df82900ae2cd6c201
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
Michal Skrivanek has posted comments on this change.
Change subject: vm: events: reorganize events
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/43538/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 3649: # Terminate the VM's creation thread.
Line 3650: self._incomingMigrationFinished.set()
Line 3651: self.guestAgent.stop()
Line 3652:
Line 3653: self.send_status_event()
As per discussion with Omer this is not needed on engine side and in fact confuses the engine to process Down VMs twice. It should not be needed as onQemuDeath will emit event on its own anyway
Line 3654:
Line 3655: if self._dom:
Line 3656: result = self._destroyVmGraceful()
Line 3657: if result['status']['code']:
--
To view, visit https://gerrit.ovirt.org/43538
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I37c198478bcc289a713ac61df82900ae2cd6c201
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.activateHsm()
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43153
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I68eccd300b8ba0cc45b2485f978eb3ae367c16e2
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.removeVmData()
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43154
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic53b19a43850341ee488b50fbb339fdf68c1a048
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: events: reorganize events
......................................................................
Patch Set 2: Code-Review+1
(4 comments)
https://gerrit.ovirt.org/#/c/43538/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 377: self._lastStatus = value
Line 378:
Line 379: def send_status_event(self, **kwargs):
Line 380: vm_status = self._getVmStatus()
Line 381: if vm_status != vmstatus.WAIT_FOR_LAUNCH:
I believe this could go away, we never send this now when events are not being triggered by libvirt
Line 382: stats = {'status': vm_status}
Line 383: stats.update(kwargs)
Line 384: self._notify('VM_status', stats)
Line 385:
Line 868: self._guestEventTime = self._startTime
Line 869: self._guestEvent = vmstatus.REBOOT_IN_PROGRESS
Line 870: self._powerDownEvent.set()
Line 871: self.saveState()
Line 872: self.guestAgent.onReboot()
I suppose this triggers event already
Some refactoring around _guestEvent would be nice in future, I think we can do without it
Line 873: if self.conf.get('volatileFloppy'):
Line 874: self._ejectFloppy()
Line 875: self.log.debug('ejected volatileFloppy')
Line 876: except Exception:
Line 1451: return vmstatus.UP
Line 1452:
Line 1453: statuses = (vmstatus.SAVING_STATE, vmstatus.RESTORING_STATE,
Line 1454: vmstatus.MIGRATION_SOURCE, vmstatus.MIGRATION_DESTINATION,
Line 1455: vmstatus.PAUSED, vmstatus.DOWN)
I think this helps fixing a small race we have
Line 1456: if self.lastStatus in statuses:
Line 1457: return self.lastStatus
Line 1458: elif self.isMigrating():
Line 1459: if self._migrationSourceThread.hibernating:
Line 1530: try:
Line 1531: self._migrationSourceThread.stop()
Line 1532: self._migrationSourceThread.status['status']['message'] = \
Line 1533: 'Migration process cancelled'
Line 1534: self.send_status_event()
did you test this flow? I wonder if we were handling this flow correctly. Where is the lastStatus changed from MIGRATION_SOURCE to UP? For failed migrations it's in _recover(), but for cancellation - do we always get libvirt event? Or some other way?
Line 1535: return self._migrationSourceThread.status
Line 1536: except libvirt.libvirtError as e:
Line 1537: if e.get_error_code() == libvirt.VIR_ERR_OPERATION_INVALID:
Line 1538: return errCode['migCancelErr']
--
To view, visit https://gerrit.ovirt.org/43538
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I37c198478bcc289a713ac61df82900ae2cd6c201
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: StorageDomainManifest: Move clusterLock into Manifest
......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43549
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2c019a7fed6ba793d2b3a2459af221b1abbff2a3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: sdm: introduce the new createVolumeContainer command
......................................................................
Patch Set 16:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/38097
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If3ea624f81a04a830cdcbcfe2f777cdd7bd26c08
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: VolumeMetadata: Introduce class structure
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/41844
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I0552bea23b04b9e58e5d2cc7e016103d03194d1a
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: VolumeMetadata: Metadata read utils
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/41846
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5e02ad8138bae0b1d60b032662afbad15dd4feba
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: VolumeMetadata: metadata write utils
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/41847
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia245c589ea6bc2cf9ac1b2b59b9073d1b5fc9595
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: VolumeMetadata: Move the getParent API
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/41848
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8489e18c431c89728c180735938a51dfc278671b
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: VolumeMetadata: Move isLeaf, setLeaf and dependencies
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/41849
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I48e05ef7168ab835d31914e05f3abd969b6552b1
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: VolumeMetadata: Move recheckIfLeaf into *Metadata class
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/41850
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibddf8fc2feea49859924743f2b30e0f2fc71ebb7
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 12: Verified+1
trivial patch
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 13:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: events: reorganize events
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43538
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I37c198478bcc289a713ac61df82900ae2cd6c201
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: events: reorganize events
......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43538
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I37c198478bcc289a713ac61df82900ae2cd6c201
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Nir Soffer has uploaded a new change for review.
Change subject: testlib: Show fully qualified class name
......................................................................
testlib: Show fully qualified class name
We typically have multiple test case classes in the same module, and
looking in the tests results it is not clear where the tests are
located.
RoundTests
test_round(n=0, size=1024, result=0) OK
test_round(n=1, size=1024, result=1024) OK
test_round(n=1024, size=1024, result=1024) OK
test_round(n=1025, size=1024, result=2048) OK
test_round(n=3.14, size=1024, result=1024) OK
StopwatchTests
test_debug OK
test_info OK
test_notset OK
Now we show also the module name:
utilsTests.RoundTests
test_round(n=0, size=1024, result=0) OK
test_round(n=1, size=1024, result=1024) OK
test_round(n=1024, size=1024, result=1024) OK
test_round(n=1025, size=1024, result=2048) OK
test_round(n=3.14, size=1024, result=1024) OK
utilsTests.StopwatchTests
test_debug OK
test_info OK
test_notset OK
Change-Id: I560ba4a0b4efcc7c53d85a830b8a8d4f5365ce5e
Signed-off-by: Nir Soffer <nsoffer(a)redhat.com>
---
M tests/testlib.py
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/32/43432/1
diff --git a/tests/testlib.py b/tests/testlib.py
index 4e27dbb..c7dbd12 100644
--- a/tests/testlib.py
+++ b/tests/testlib.py
@@ -296,7 +296,8 @@
def startTest(self, test):
unittest.TestResult.startTest(self, test)
- current_case = test.test.__class__.__name__
+ current_case = "%s.%s" % (test.test.__module__,
+ test.test.__class__.__name__)
if self.showAll:
if current_case != self._last_case:
--
To view, visit https://gerrit.ovirt.org/43432
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I560ba4a0b4efcc7c53d85a830b8a8d4f5365ce5e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Nir Soffer has uploaded a new change for review.
Change subject: testlib: Use nose.case.Test.getDescription()
......................................................................
testlib: Use nose.case.Test.getDescription()
The version we have may be ok for Python 2.6, but node version has
additional code to support Python 2.7 changes, so we should use it.
Change-Id: I3a31a432fbeaa6f20f35247572aa995d1fae38d1
Signed-off-by: Nir Soffer <nsoffer(a)redhat.com>
---
M tests/testlib.py
1 file changed, 0 insertions(+), 3 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/31/43431/1
diff --git a/tests/testlib.py b/tests/testlib.py
index 04485e9..4e27dbb 100644
--- a/tests/testlib.py
+++ b/tests/testlib.py
@@ -249,9 +249,6 @@
result.TextTestResult.__init__(self, *args, **kwargs)
self._last_case = None
- def getDescription(self, test):
- return str(test)
-
def _writeResult(self, test, long_result, color, short_result, success):
if self.showAll:
colorWrite(self.stream, long_result, color)
--
To view, visit https://gerrit.ovirt.org/43431
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3a31a432fbeaa6f20f35247572aa995d1fae38d1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Francesco Romani has uploaded a new change for review.
Change subject: vm: run before_vm_create hook only in the VM boot
......................................................................
vm: run before_vm_create hook only in the VM boot
The 'before_vm_create' hook is running in two flows:
* as expected, in the VM creation flow.
* as may not be expected, in the VM dehybernation flow.
The latter is surprising and, most important,
contractdicting because in the dehibernation flow
* the domain XML returned by the before_vm_create
is never used
* just after the before_vm_create hook, we run
the before_vm_dehibernate hook with the stored
domain XML.
At risk to breaking some probably already incorrect
hook, this patch makes the before_vm_create hook
run only in the VM creation path.
Change-Id: I2ad05e39029acc7b898c7b2af2141bba1d7d3c8e
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 3 insertions(+), 4 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/30/34530/1
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 6e46119..751947e 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -2667,10 +2667,6 @@
# we need to complete the initialization, including
# domDependentInit, after the migration is completed.
- if not self.recovering and initDomain:
- domxml = hooks.before_vm_start(self._buildDomainXML(), self.conf)
- self.log.debug(domxml)
-
if self.recovering:
self._dom = NotifyingVirDomain(
self._connection.lookupByUUIDString(self.id),
@@ -2698,6 +2694,9 @@
self._connection.lookupByUUIDString(self.id),
self._timeoutExperienced)
else:
+ domxml = hooks.before_vm_start(self._buildDomainXML(), self.conf)
+ self.log.debug(domxml)
+
flags = libvirt.VIR_DOMAIN_NONE
if 'launchPaused' in self.conf:
flags |= libvirt.VIR_DOMAIN_START_PAUSED
--
To view, visit http://gerrit.ovirt.org/34530
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2ad05e39029acc7b898c7b2af2141bba1d7d3c8e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 11: Verified+1
verified that an events with extra data is indeed emitted.
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 12: Verified+1
verified that an events with extra data is indeed emitted.
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 11: Verified+1
verified that an events with extra data is indeed emitted.
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.activateHsm()
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43153
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I68eccd300b8ba0cc45b2485f978eb3ae367c16e2
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.removeVmData()
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43154
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic53b19a43850341ee488b50fbb339fdf68c1a048
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Ido Barkan has uploaded a new change for review.
Change subject: net: wait for restored devices to be up
......................................................................
net: wait for restored devices to be up
If restore-net-config does not wait for all devices to be up, it
might observe a transient kernel state where, for example, bonding
devices might report no slaves. This makes restoration re-restore a
bond because it is reported by the kernel as different from the
persisted device.
Change-Id: I3cd3de577e5d0bcf5e87c4894e94e03c209ce76a
Signed-off-by: Ido Barkan <ibarkan(a)redhat.com>
Bug-Url: https://bugzilla.redhat.com/1203422
---
M vdsm/vdsm-restore-net-config
1 file changed, 35 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/22/43222/1
diff --git a/vdsm/vdsm-restore-net-config b/vdsm/vdsm-restore-net-config
index 17957c0..7968da3 100755
--- a/vdsm/vdsm-restore-net-config
+++ b/vdsm/vdsm-restore-net-config
@@ -22,8 +22,10 @@
import logging
import logging.config
import os
+import time
from vdsm.config import config
+from vdsm import ipwrapper
from vdsm import netinfo
from vdsm.constants import P_VDSM_RUN
from vdsm.netconfpersistence import KernelConfig, BaseConfig
@@ -38,6 +40,7 @@
import pkgutil
_NETS_RESTORED_MARK = os.path.join(P_VDSM_RUN, 'nets_restored')
+_ALL_DEVICES_UP_TIMEOUT = 30
def ifcfg_restoration():
@@ -68,6 +71,8 @@
persistent_config = PersistentConfig()
available_config = _filter_available(persistent_config)
+ _wait_for_for_all_devices_up(
+ available_config.networks.keys() + available_config.bonds.keys())
changed_config = _filter_changed_nets_bonds(available_config)
nets = changed_config.networks
bonds = changed_config.bonds
@@ -168,6 +173,36 @@
return changed_or_missing
+def _wait_for_for_all_devices_up(links):
+ sleep_time = 1
+ time_left = _ALL_DEVICES_UP_TIMEOUT
+
+ down_links = _get_links_with_state_down(links)
+
+ # TODO: use netlink monitor here might be more elegant (not available in
+ # TODO: 3.5)
+ while down_links and time_left > 0:
+ logging.debug("waiting for %s to be up.", down_links)
+ time.sleep(sleep_time)
+ time_left -= sleep_time
+ down_links = _get_links_with_state_down(links)
+
+ if down_links:
+ logging.warning("Not all devices are up. VDSM might restore them "
+ "although they were not changed since they were "
+ "persisted.")
+ else:
+ logging.debug("All devices are up.")
+
+
+def _get_links_with_state_down(links):
+ def oper_up(link):
+ return bool(link.flags & 1 << 6)
+
+ return set([l.name for l in ipwrapper.getLinks() if
+ l.name in links and not oper_up(l)])
+
+
def _get_all_configurators():
"""Returns the class objects of all the configurators in the netconf pkg"""
prefix = configurators.__name__ + '.'
--
To view, visit https://gerrit.ovirt.org/43222
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3cd3de577e5d0bcf5e87c4894e94e03c209ce76a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Ido Barkan <ibarkan(a)redhat.com>
Hello Dan Kenigsberg,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/43276
to review the following change.
Change subject: no need to call setupNetworks with empty argument during network restoration
......................................................................
no need to call setupNetworks with empty argument during network restoration
It is not smart to trust setupNetwork implementation so that it is really a
no-op when called with empty input.
Change-Id: If7de328832c53ef40748c4e932b4340abf1ecf11
Signed-off-by: Ido Barkan <ibarkan(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38745
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M vdsm/vdsm-restore-net-config
1 file changed, 9 insertions(+), 7 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/76/43276/1
diff --git a/vdsm/vdsm-restore-net-config b/vdsm/vdsm-restore-net-config
index c4a9288..ca4ae5b 100755
--- a/vdsm/vdsm-restore-net-config
+++ b/vdsm/vdsm-restore-net-config
@@ -61,10 +61,11 @@
removeNetworks[network] = {'remove': True}
for bond in runningConfig.bonds:
removeBonds[bond] = {'remove': True}
- logging.debug('Removing all networks (%s) and bonds (%s) in running '
- 'config.', removeNetworks, removeBonds)
- setupNetworks(removeNetworks, removeBonds, connectivityCheck=False,
- _inRollback=True)
+ if removeNetworks or removeBonds:
+ logging.debug('Removing all networks (%s) and bonds (%s) in running '
+ 'config.', removeNetworks, removeBonds)
+ setupNetworks(removeNetworks, removeBonds, connectivityCheck=False,
+ _inRollback=True)
# Restore non-VDSM network devices (BZ#1188251)
configWriter = ifcfg.ConfigWriter()
@@ -78,9 +79,10 @@
nets = changed_config.networks
bonds = changed_config.bonds
_convert_to_blocking_dhcp(nets)
- logging.debug('Calling setupNetworks with networks (%s) and bond (%s).',
- nets, bonds)
- setupNetworks(nets, bonds, connectivityCheck=False, _inRollback=True)
+ if nets or bonds:
+ logging.debug('Calling setupNetworks with networks (%s) and bond (%s).',
+ nets, bonds)
+ setupNetworks(nets, bonds, connectivityCheck=False, _inRollback=True)
def _convert_to_blocking_dhcp(networks):
--
To view, visit https://gerrit.ovirt.org/43276
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If7de328832c53ef40748c4e932b4340abf1ecf11
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Ido Barkan <ibarkan(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Ido Barkan has uploaded a new change for review.
Change subject: net: ignore bondingOptions in persistent networks.
......................................................................
net: ignore bondingOptions in persistent networks.
Up until https://gerrit.ovirt.org/#/c/42146/ bondingOptions were
mistakenly part of persistent network attributes. So, if we are
upgrading from an older enough version, we should drop this entry
it the copied persistent networks.
Change-Id: I1c29d4e6c109d4fa7b2d293133b8551e423ec139
Signed-off-by: Ido Barkan <ibarkan(a)redhat.com>
---
M lib/vdsm/netconfpersistence.py
1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/38/43238/1
diff --git a/lib/vdsm/netconfpersistence.py b/lib/vdsm/netconfpersistence.py
index b8de93d..b5efff8 100644
--- a/lib/vdsm/netconfpersistence.py
+++ b/lib/vdsm/netconfpersistence.py
@@ -363,6 +363,11 @@
bond_attr.get('options'))
bond_attr['options'] = self._netinfo.bondOptsForIfcfg(
normalized_opts)
+ # before d18e2f10 bondingOptions were also part of networks, so in case
+ # we are upgrading from an older version, they should be ignored if
+ # they exist.
+ for net_attr in config_copy.networks.itervalues():
+ net_attr.pop('bondingOptions', None)
def _normalize_bonding_nics(self, config_copy):
for bond_attr in config_copy.bonds.itervalues():
--
To view, visit https://gerrit.ovirt.org/43238
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c29d4e6c109d4fa7b2d293133b8551e423ec139
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Ido Barkan <ibarkan(a)redhat.com>
Ido Barkan has uploaded a new change for review.
Change subject: net: if all devices are up, skip ifup during restoratrion.
......................................................................
net: if all devices are up, skip ifup during restoratrion.
The usual case since 3.5.4 is that all devices ahve ONBOOT=yes and
are expected to be up during restoration. But if only some of them
are down, it is better to be tedious and bring everything up with
the right order.
Change-Id: I9e97bea58be035e94ddd3a3a6c67132a2cbf1dc3
Signed-off-by: Ido Barkan <ibarkan(a)redhat.com>
---
M vdsm/vdsm-restore-net-config
1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/35/43435/1
diff --git a/vdsm/vdsm-restore-net-config b/vdsm/vdsm-restore-net-config
index 2863507..fc16e78 100755
--- a/vdsm/vdsm-restore-net-config
+++ b/vdsm/vdsm-restore-net-config
@@ -133,7 +133,11 @@
"""
for ifcfg_file in owned_ifcfg_files:
_upgrade_onboot(ifcfg_file)
- ifcfg.start_devices(owned_ifcfg_files)
+ down_links = _get_links_with_state_down(
+ [os.path.basename(name) for name in owned_ifcfg_files])
+ if down_links:
+ logging.debug("Some of the devices are down (%s).", down_links)
+ ifcfg.start_devices(owned_ifcfg_files)
def _upgrade_onboot(ifcfg_file):
--
To view, visit https://gerrit.ovirt.org/43435
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9e97bea58be035e94ddd3a3a6c67132a2cbf1dc3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan(a)redhat.com>
Hello Fred Rolland,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/42608
to review the following change.
Change subject: resize-lun: resizePV parameters
......................................................................
resize-lun: resizePV parameters
In lvm.resizePV, we need all devices names in the filters parameters.
Otherwise the operation will fail. It is needed in case we have more
than one LUN on the storage domain.
Change-Id: I4213f82a0307dc91202078b53c0b2c2f24433899
Bug-Url:https://bugzilla.redhat.com/609689
Signed-off-by: Fred Rolland <frolland(a)redhat.com>
---
M vdsm/storage/lvm.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/42608/1
diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py
index f93e5b9..090dc14 100644
--- a/vdsm/storage/lvm.py
+++ b/vdsm/storage/lvm.py
@@ -877,7 +877,7 @@
"""
pvName = _fqpvname(guid)
cmd = ["pvresize", pvName]
- rc, out, err = _lvminfo.cmd(cmd, (pvName,))
+ rc, out, err = _lvminfo.cmd(cmd)
if rc != 0:
raise se.CouldNotResizePhysicalVolume(pvName, err)
_lvminfo._invalidatepvs(pvName)
--
To view, visit https://gerrit.ovirt.org/42608
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4213f82a0307dc91202078b53c0b2c2f24433899
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com>
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Vinzenz Feenstra has uploaded a new change for review.
Change subject: virt: Rate limit epoll errors on untracked FDs
......................................................................
virt: Rate limit epoll errors on untracked FDs
There seem to be moments, when epoll starts returning errors on
file descriptors, which aren't any more tracked, so often, that
we're running on 100% CPU in the vmchannels listener thread.
To rate limit such conditions, this patch introduces a forced
sleep of 5 seconds after every 10 of such errors and it will
take down the whole process after 100 of such errors have
occurred.
Change-Id: I654e5b828e2c05e8a86fa0e945be67cec014c110
Bug-Url: https://bugzilla.redhat.com/1226911
Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com>
---
M vdsm/virt/vmchannels.py
1 file changed, 21 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/69/42569/1
diff --git a/vdsm/virt/vmchannels.py b/vdsm/virt/vmchannels.py
index 61ffcd5..9939546 100644
--- a/vdsm/virt/vmchannels.py
+++ b/vdsm/virt/vmchannels.py
@@ -18,16 +18,26 @@
# Refer to the README and COPYING files for full details of the license
#
+import logging
+import os
+import select
+import signal
import threading
import time
-import select
-import logging
from storage.misc import NoIntrPoll
# How many times a reconnect should be performed before a cooldown will be
# applied
COOLDOWN_RECONNECT_THRESHOLD = 5
+
+# How long the thread should sleep, when the cool down limit for untracked
+# errors has been reached
+ERROR_RATE_LIMIT_SLEEP_TIME = 5
+# After how many errors the cooldown should be applied
+UNTRACKED_ERROR_COOLDOWN_LIMIT = 10
+# After how many errors we should take down VDSM
+UNTRACKED_ERROR_RESTART_LIMIT = 100
class Listener(threading.Thread):
@@ -46,6 +56,7 @@
self._add_channels = {}
self._del_channels = []
self._timeout = None
+ self._untrackedErrors = 0
def _handle_event(self, fileno, event):
""" Handle an epoll event occurred on a specific file descriptor. """
@@ -55,6 +66,7 @@
if fileno in self._channels:
reconnect = True
else:
+ self._untrackedErrors += 1
self.log.debug("Received %.08X. On fd removed by epoll.",
event)
elif (event & select.EPOLLIN):
@@ -167,6 +179,13 @@
events = NoIntrPoll(self._epoll.poll, 1)
for (fileno, event) in events:
self._handle_event(fileno, event)
+ if self._untrackedErrors > UNTRACKED_ERROR_RESTART_LIMIT:
+ self.log.critical("Too many epoll errors happened."
+ "Taking down process")
+ os.kill(os.getpid(), signal.SIGTERM)
+ elif (self._untrackedErrors > 0 and
+ self._untrackedErrors % UNTRACKED_ERROR_COOLDOWN_LIMIT == 0):
+ time.sleep(ERROR_RATE_LIMIT_SLEEP_TIME)
else:
self._update_channels()
if (self._timeout is not None) and (self._timeout > 0):
--
To view, visit https://gerrit.ovirt.org/42569
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I654e5b828e2c05e8a86fa0e945be67cec014c110
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com>
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.updateVmData()
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/42930
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id79e7d6375e60216d1eb9a58e5a5c087db98625f
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Liron Aravot has uploaded a new change for review.
Change subject: adding StorageDomain.getBackedUpVmsInfo
......................................................................
adding StorageDomain.getBackedUpVmsInfo
This patch adds the StorageDomain.getBackedUpVmsInfo() verb which is
used to get the OVF information from an export domain in a pool that
operates without Spm.
The old verb that was used in pool with Spm was
StoragePool.getBackedUpVmsInfo() which was limited to the spm only
because it supported to get OVF information from the master domain as
well (the master_fs was mounted on the Spm).
Change-Id: Ie2896897bccc493457695181436a55f688bb596f
Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com>
---
M client/vdsClient.py
M vdsm/API.py
M vdsm/rpc/BindingXMLRPC.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/storage/hsm.py
M vdsm/storage/sp.py
6 files changed, 73 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/29/42129/1
diff --git a/client/vdsClient.py b/client/vdsClient.py
index 4b11181..c299811 100755
--- a/client/vdsClient.py
+++ b/client/vdsClient.py
@@ -623,6 +623,15 @@
return dom['status']['code'], dom['status']['message']
return 0, ''
+ def getSDBackedUpVmsInfo(self, args):
+ spUUID = args[0]
+ sdUUID = args[1]
+ vmList = args[2]
+ dom = self.s.getBackedUpVmsInfo(spUUID, sdUUID, vmList)
+ if dom['status']['code']:
+ return dom['status']['code'], dom['status']['message']
+ return 0, ''
+
def validateStorageDomain(self, args):
sdUUID = args[0]
dom = self.s.validateStorageDomain(sdUUID)
@@ -2467,6 +2476,12 @@
('<domain UUID> <descr>',
'Set storage domain description'
)),
+ 'getStorageDomainBackedUpVmsInfo': (serv.getSDBackedUpVmsInfo,
+ ('<spUUID> <sdUUID>',
+ '[vmList](vmId1,vmId2,...)]'
+ 'Get list of VMs from the'
+ 'given backup domain'
+ )),
'validateStorageDomain': (serv.validateStorageDomain,
('<domain UUID>',
'Validate storage domain'
diff --git a/vdsm/API.py b/vdsm/API.py
index 20e5b30..f771f89 100644
--- a/vdsm/API.py
+++ b/vdsm/API.py
@@ -1048,6 +1048,9 @@
def setDescription(self, description):
return self._irs.setStorageDomainDescription(self._UUID, description)
+ def getBackedUpVmsInfo(self, storagepoolID, vmList):
+ return self._irs.getVmsInfo(storagepoolID, self._UUID, vmList)
+
def validate(self):
return self._irs.validateStorageDomain(self._UUID)
diff --git a/vdsm/rpc/BindingXMLRPC.py b/vdsm/rpc/BindingXMLRPC.py
index 1735dba..cf12664 100644
--- a/vdsm/rpc/BindingXMLRPC.py
+++ b/vdsm/rpc/BindingXMLRPC.py
@@ -662,6 +662,10 @@
domain = API.StorageDomain(sdUUID)
return domain.getImages()
+ def domainGetBackedUpVmsInfo(self, spUUID, sdUUID, vmList=None):
+ domain = API.StorageDomain(sdUUID)
+ return domain.getBackedUpVmsInfo(spUUID, vmList)
+
def domainGetInfo(self, sdUUID, options=None):
domain = API.StorageDomain(sdUUID)
return domain.getInfo()
@@ -1084,7 +1088,8 @@
(self.domainAttachHsm, 'attachStorageDomainHsm'),
(self.domainDetachHsm, 'detachStorageDomainHsm'),
(self.domainDetachAnyStoragePoolHsm,
- 'detachAnyStoragePoolHsm'))
+ 'detachAnyStoragePoolHsm'),
+ (self.domainGetBackedUpVmsInfo, 'getBackedUpVmsInfo'))
def getIrsMethods(self):
return ((self.domainActivate, 'activateStorageDomain'),
diff --git a/vdsm/rpc/vdsmapi-schema.json b/vdsm/rpc/vdsmapi-schema.json
index 2eeefb4..221e0b5 100644
--- a/vdsm/rpc/vdsmapi-schema.json
+++ b/vdsm/rpc/vdsmapi-schema.json
@@ -4996,6 +4996,27 @@
'data': {'storagedomainID': 'UUID', 'storagepoolID': 'UUID'}}
##
+# @StorageDomain.getBackedUpVmsInfo:
+#
+# Get information about backed-up virtual machines from a backup domain.
+#
+# @storagepoolID: The UUID of the Storage Pool
+#
+# @storagedomainID: The UUID of the backup Storage Domain
+#
+# @vmList: Limit results to a list of VM UUIDs
+#
+# Returns:
+# A dictionary of OVF information indexed by VM UUID
+#
+# Since: 4.18.0
+##
+{'command': {'class': 'StorageDomain', 'name': 'getBackedUpVmsInfo'},
+ 'data': {'storagepoolID': 'UUID', 'storagedomainID': 'UUID',
+ 'vmList': ['UUID']},
+ 'returns': 'OVFMap'}
+
+##
# @StorageDomainCreateArgumentsBlock:
#
# Creation argument for Block-based Storage Domains.
diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index 5876646..621ea89 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -1513,6 +1513,30 @@
return dict(vmlist=vms)
@public
+ def getBackedUpVmsInfo(self, spUUID, sdUUID, vmList=None):
+ """
+ Gets a list of VMs with their info from the pool.
+
+ * If 'sdUUID' is given and it's a backup domain then get the list of
+ VMs from it.
+ * If 'vmList' is given get info for these VMs only.
+
+ :param spUUID: The UUID of the storage pool that you want to query.
+ :type spUUID: UUID
+ :param sdUUID: The UUID of the backup domain that the you want to
+ query or :keyword:`None`.
+ :type sdUUID: UUID
+ :param vmList: A UUID list of the VMs you want info on or
+ :keyword:`None` for all VMs in pool or backup domain.
+ :param options: ?
+ """
+ pool = self.getPool(spUUID)
+ self.validateBackupDom(sdUUID)
+ vars.task.getSharedLock(STORAGE, sdUUID)
+ vms = pool.getVmsInfo(sdUUID, vmList)
+ return dict(vmlist=vms)
+
+ @public
def createVolume(self, sdUUID, spUUID, imgUUID, size, volFormat,
preallocate, diskType, volUUID, desc,
srcImgUUID=volume.BLANK_UUID,
diff --git a/vdsm/storage/sp.py b/vdsm/storage/sp.py
index 3dcf343..eb79c3b 100644
--- a/vdsm/storage/sp.py
+++ b/vdsm/storage/sp.py
@@ -1996,6 +1996,10 @@
return sdCache.produce(sdUUID).getVMsList()
def getVmsInfo(self, sdUUID, vmList=None):
+ return self._getVmsInfo(sdUUID, vmList)
+
+ @unsecured
+ def _getVmsInfo(self, sdUUID, vmList=None):
self.validatePoolSD(sdUUID)
return sdCache.produce(sdUUID).getVMsInfo(vmList=vmList)
--
To view, visit https://gerrit.ovirt.org/42129
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie2896897bccc493457695181436a55f688bb596f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.activateHsm()
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43153
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I68eccd300b8ba0cc45b2485f978eb3ae367c16e2
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: adding the StorageDomain.upgradeVersion verb
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/42478
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id8fb87c6e04e48072ec90d07a0f4c7a61411a808
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: adding StorageDomain.removeVmData()
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43154
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic53b19a43850341ee488b50fbb339fdf68c1a048
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Nir Soffer has posted comments on this change.
Change subject: freeze: Avoid unneeded freeze during snapshot
......................................................................
Patch Set 6: Verified+1
Tested with older engine that does not send frozen flag.
--
To view, visit https://gerrit.ovirt.org/43293
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I328470f700ab74daeed5375c0ade83628d16c387
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: freeze: Avoid unneeded freeze during snapshot
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43293
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I328470f700ab74daeed5375c0ade83628d16c387
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 9: Code-Review-1
see inline comments from Michal
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm-reg: remove the sub-project
......................................................................
Patch Set 1: Code-Review-1
(3 comments)
https://gerrit.ovirt.org/#/c/43254/1/configure.ac
File configure.ac:
Line 331: AC_PATH_PROG([TAR_PATH], [tar], [/bin/tar])
Line 332: AC_PATH_PROG([TC_PATH], [tc], [/sbin/tc])
Line 333: AC_PATH_PROG([TEE_PATH], [tee], [/usr/bin/tee])
Line 334: AC_PATH_PROG([TOUCH_PATH], [touch], [/bin/touch])
Line 335: AC_PATH_PROG([TUNE2FS_PATH], [tune2fs], [/sbin/tune2fs])
please check which of
-EX_CAT = '@CAT_PATH@'
-EX_CHKCONFIG = '@CHKCONFIG_PATH@'
-EX_DMIDECODE = '@DMIDECODE_PATH@'
-EX_ECHO = '@ECHO_PATH@'
-EX_GRUBBY = '@GRUBBY_PATH@'
-EX_HWCLOCK = '@HWCLOCK_PATH@'
-EX_NTPQ = '@NTPQ_PATH@'
-EX_OPENSSL = '@OPENSSL_PATH@'
-EX_REBOOT = '@REBOOT_PATH@'
-EX_RPM = '@RPM_PATH@'
-EX_SED = '@SED_PATH@'
-EX_SERVICE = '@SERVICE_PATH@'
-EX_SYSTEMCTL = '@SYSTEMCTL_PATH@'
-EX_YUM = '@YUM_PATH@'
can now be dropped from configure.ac
Line 336: AC_PATH_PROG([UDEVADM_PATH], [udevadm], [/sbin/udevadm])
Line 337: AC_PATH_PROG([UMOUNT_PATH], [umount], [/bin/umount])
Line 338: AC_PATH_PROG([WGET_PATH], [wget], [/usr/bin/wget])
Line 339: AC_PATH_PROG([YUM_PATH], [yum], [/usr/bin/yum])
https://gerrit.ovirt.org/#/c/43254/1/vdsm_reg/Makefile.am
File vdsm_reg/Makefile.am:
Line 54
Line 55
Line 56
Line 57
Line 58
all dropped *.in files should be cleaned from .gitignore, too.
https://gerrit.ovirt.org/#/c/43254/1/vdsm_reg/deployUtil.py.in
File vdsm_reg/deployUtil.py.in:
Line 85
Line 86
Line 87
Line 88
Line 89
vdsm/{add,del}Network can (and should) be dropped now.
--
To view, visit https://gerrit.ovirt.org/43254
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5d19ddb0e65657515e2dc0eb9eeae320d8d6bf71
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Petr Horáček has uploaded a new change for review.
Change subject: net: add TODO to pyroute2 configurator
......................................................................
net: add TODO to pyroute2 configurator
Track needed changes in TODO instead of untouched patches on gerrit.
Change-Id: Idbfe663b6180654dba76d69ccc11ecbf14abff8f
Signed-off-by: Petr Horáček <phoracek(a)redhat.com>
---
M vdsm/network/configurators/pyroute_two.py
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/10/43410/1
diff --git a/vdsm/network/configurators/pyroute_two.py b/vdsm/network/configurators/pyroute_two.py
index c7730d3..6751c16 100644
--- a/vdsm/network/configurators/pyroute_two.py
+++ b/vdsm/network/configurators/pyroute_two.py
@@ -16,6 +16,10 @@
#
# Refer to the README and COPYING files for full details of the license
#
+""" TODO
+- use reuse=True when calling create()
+- release() is deprecated, use close() instead
+"""
from __future__ import absolute_import
import logging
--
To view, visit https://gerrit.ovirt.org/43410
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idbfe663b6180654dba76d69ccc11ecbf14abff8f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek(a)redhat.com>
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 8: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/43289/8/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 331: self._guestSocketFile = self._makeChannelPath(_VMCHANNEL_DEVICE_NAME)
Line 332: self._qemuguestSocketFile = self._makeChannelPath(_QEMU_GA_DEVICE_NAME)
Line 333: self.guestAgent = guestagent.GuestAgent(
Line 334: self._guestSocketFile, self.cif.channelListener, self.log,
Line 335: self._onGuestStatusChange)
Malicious guest can (try to) DoS VDSM or Engine. The current Vm._evaluatedStatus offers some protection. Do we need to re-introduce it in later patches? see https://gerrit.ovirt.org/#/c/43414
Line 336: self._domain = DomainDescriptor.from_id(self.id)
Line 337: self._released = False
Line 338: self._releaseLock = threading.Lock()
Line 339: self.saveState()
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 9:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 6:
(1 comment)
https://gerrit.ovirt.org/#/c/43312/6/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 3615: if not self.extendDrivesIfNeeded():
Line 3616: self.log.info("No VM drives were extended")
Line 3617:
Line 3618: # we need new disk stats
Line 3619: self.send_status_event(**self._getRunningVmStats())
> where/when do we need them?
good point(s). My reasoning was that after the resize, it is worth to send back immediately updated disk stats. But indeed need to check how much it takes.
Line 3620:
Line 3621: elif action == libvirt.VIR_DOMAIN_EVENT_IO_ERROR_REPORT:
Line 3622: self.log.info('I/O error %s device %s reported to guest OS',
Line 3623: err, blockDevAlias)
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 8:
We can try sending all the stats.
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Dan Kenigsberg has uploaded a new change for review.
Change subject: tcTest: skip tests if somehow same bridge name is reused
......................................................................
tcTest: skip tests if somehow same bridge name is reused
My commit 61fff86c did not solve the problem of tests magically
re-drawing the same bridge name as a formerly-started test. There is no
need to mark these jobs as failure, as the problem is not in our code
but in our broken test.
Change-Id: Id678882d68976618360bcce76f16837d67e274ea
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
---
M tests/tcTests.py
1 file changed, 8 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/98/41898/1
diff --git a/tests/tcTests.py b/tests/tcTests.py
index cbc7faa..49ad6f2 100644
--- a/tests/tcTests.py
+++ b/tests/tcTests.py
@@ -78,7 +78,14 @@
class _Bridge(_Interface):
def addDevice(self):
- check_call([EXT_BRCTL, 'addbr', self.devName])
+ try:
+ check_call([EXT_BRCTL, 'addbr', self.devName])
+ except RuntimeError as e:
+ if 'can\'t create bridge with the same name' in e.message:
+ raise SkipTest(e.message)
+ else:
+ raise
+
# learning interval is different on different kernels, so set it
# explicit for 2.x kernels
if os.uname()[2].startswith("2"):
--
To view, visit https://gerrit.ovirt.org/41898
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id678882d68976618360bcce76f16837d67e274ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Ido Barkan has uploaded a new change for review.
Change subject: net: wait for restored devices to be up
......................................................................
net: wait for restored devices to be up
If restore-net-config does not wait for all devices to be up, it
might observe a transient kernel state where, for example, bonding
devices might report no slaves. This makes restoration re-restore a
bond because it is reported by the kernel as different from the
persisted device.
Change-Id: I3cd3de577e5d0bcf5e87c4894e94e03c209ce76a
Signed-off-by: Ido Barkan <ibarkan(a)redhat.com>
---
M vdsm/vdsm-restore-net-config
1 file changed, 24 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/09/43209/1
diff --git a/vdsm/vdsm-restore-net-config b/vdsm/vdsm-restore-net-config
index bec5c87..4dc7012 100755
--- a/vdsm/vdsm-restore-net-config
+++ b/vdsm/vdsm-restore-net-config
@@ -26,10 +26,12 @@
import threading
from vdsm.config import config
+from vdsm import ipwrapper
from vdsm import netinfo
from vdsm.constants import P_VDSM_RUN
from vdsm.netconfpersistence import RunningConfig, PersistentConfig, \
KernelConfig, CONF_PERSIST_DIR, BaseConfig
+from vdsm.netlink.monitor import Monitor
import hostdev
# Ifcfg persistence restoration
@@ -40,6 +42,8 @@
from network import configurators
import pkgutil
+
+_ALL_DEVICES_UP_TIMEOUT = 30
_NETS_RESTORED_MARK = os.path.join(P_VDSM_RUN, 'nets_restored')
_VIRTUAL_FUNCTIONS_PATH = os.path.join(CONF_PERSIST_DIR, 'virtual_functions')
@@ -109,6 +113,7 @@
nets = changed_config.networks
bonds = changed_config.bonds
if nets or bonds:
+ _wait_for_for_all_devices_up(nets.keys() + bonds.keys())
_convert_to_blocking_dhcp(nets)
logging.debug('Calling setupNetworks with networks (%s) '
'and bond (%s).', nets, bonds)
@@ -248,6 +253,25 @@
return available_bonds
+def _wait_for_for_all_devices_up(links):
+ down_links = set([l for l in ipwrapper.getLinks() if
+ l.name in links and not l.oper_up])
+
+ if not down_links:
+ logging.debug("all devices are up.")
+ return
+
+ logging.debug("waiting for %s to be up.", down_links)
+ monitor = Monitor(('link',), timeout=_ALL_DEVICES_UP_TIMEOUT,
+ silent_timeout=True)
+ monitor.start()
+ for event in monitor:
+ if event['state'] == netinfo.OPERSTATE_UP:
+ down_links.discard(event['name'])
+ if not down_links:
+ break
+
+
def _get_all_configurators():
"""Returns the class objects of all the configurators in the netconf pkg"""
prefix = configurators.__name__ + '.'
--
To view, visit https://gerrit.ovirt.org/43209
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3cd3de577e5d0bcf5e87c4894e94e03c209ce76a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan(a)redhat.com>
Michal Skrivanek has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 6: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 8:
looks ok
running stats is the best bet I suppose.
In future we should send everything so engine can drop the subsequent poll call
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 6:
(1 comment)
not sure, should be tested.
no harm if we dont' send it (we can skip some quick pause/extend/unpause cases, but that shouldn't matter)
https://gerrit.ovirt.org/#/c/43312/6/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 3615: if not self.extendDrivesIfNeeded():
Line 3616: self.log.info("No VM drives were extended")
Line 3617:
Line 3618: # we need new disk stats
Line 3619: self.send_status_event(**self._getRunningVmStats())
where/when do we need them?
does "the extend function take a long time? if so perhaps better to send event beforehand?
and then on resume it is already handled?
Line 3620:
Line 3621: elif action == libvirt.VIR_DOMAIN_EVENT_IO_ERROR_REPORT:
Line 3622: self.log.info('I/O error %s device %s reported to guest OS',
Line 3623: err, blockDevAlias)
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 8: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has abandoned this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Abandoned
abandoned as planned. We don't really need this patch.
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: events: extract function to make event info
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/43268
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If60e8422d06fb1ff45a846858856ad68d52fd6ce
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has abandoned this change.
Change subject: virt: events: extract function to make event info
......................................................................
Abandoned
abandoned as planned. We don't really need this patch.
--
To view, visit https://gerrit.ovirt.org/43268
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: If60e8422d06fb1ff45a846858856ad68d52fd6ce
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 8:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 5: Code-Review-1
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 5:
unneeded, same as 43268. drop one of them.
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: virt: events: extract function to make event info
......................................................................
Patch Set 5: Code-Review-1
unneeded flexibility^Wcomplexity. Just add arguments to send_status_event
--
To view, visit https://gerrit.ovirt.org/43268
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If60e8422d06fb1ff45a846858856ad68d52fd6ce
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 2:
(3 comments)
https://gerrit.ovirt.org/#/c/43270/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1308: exitMessage, exitReasonCode)
Line 1309: # DoubleDownError is protecting us here
Line 1310: event_data = self._getExitedVmStats()
Line 1311: except DoubleDownError:
Line 1312: event_data = {}
> I don't think so. it raises when another lastStatus = DOWN happens...which
yep, will remove before the verification
Line 1313: pass
Line 1314: try:
Line 1315: self.guestAgent.stop()
Line 1316: except Exception:
Line 1316: except Exception:
Line 1317: pass
Line 1318: self.saveState()
Line 1319: if event_data:
Line 1320: self._send_event(
> too long for a single line?
it should fit and probably end up better looking
Line 1321: vmstatus.DOWN,
Line 1322: **event_data
Line 1323: )
Line 1324:
Line 1367: def _getExitedVmStats(self):
Line 1368: stats = {
Line 1369: 'exitCode': self.conf['exitCode'],
Line 1370: 'exitMessage': self.conf['exitMessage'],
Line 1371: 'exitReason': self.conf['exitReason']
> whitespace or I just don't understand gerrit diff?
I changed the positioning of closing brace '}'. Unneeded change indeed.
Line 1372: }
Line 1373: if 'timeOffset' in self.conf:
Line 1374: stats['timeOffset'] = self.conf['timeOffset']
Line 1375: return stats
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
Francesco Romani has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/43269/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 775: self.recovering = False
Line 776: self.saveState()
Line 777:
Line 778: self._send_event(
Line 779: self._getVmStatus(),
> I wonder...
Sorry, this was lost in rebase fest.
I had the same doubt. I choosed the state we see because of the guest agent delay (see _getVmStauts and _getVmStatusFromGuest).
I was concerned of sending 'Up' too early, and this seemed a bit safer, and helped to keep the patch(set) minimal.
It is open to discussion, of course.
Line 780: **self._getRunningVmStats()
Line 781: )
Line 782:
Line 783: except MissingLibvirtDomainError as e:
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: add minimal running stats to status
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/43290
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I894a1b10d3cc3dd33664ea9f0842cbc7f1eb07b3
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has abandoned this change.
Change subject: vm: event: add minimal running stats to status
......................................................................
Abandoned
better not send data we're not 100% sure Engine could benefit from.
--
To view, visit https://gerrit.ovirt.org/43290
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I894a1b10d3cc3dd33664ea9f0842cbc7f1eb07b3
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Francesco Romani has uploaded a new change for review.
Change subject: vm: extract cheap subset of running vm stats
......................................................................
vm: extract cheap subset of running vm stats
The getRunningVmStats() method does a lot of work an fetches
all the stats related to a running VMs. This involves
fetching data from periodic tasks, sanitizing all the
data and so forth.
There is an subset of this data which it is easy and cheap
to gather, and yet it contains useful information from
Engine.
This patch extracts this subset and moves it in a separate
function, to be used both by old code and from a new event.
Change-Id: Ia3e2c19d0b69e78f463976478ded12f306e2f2c7
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 14 insertions(+), 8 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/85/43285/2
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index bed6b7e..770435b 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -1364,21 +1364,29 @@
stats['boot'] = self.conf['boot']
return stats
- def _getRunningVmStats(self):
- """
- gathers all the stats which can change while a VM is running.
- """
+ def _getUpVmStats(self):
stats = {
'elapsedTime': str(int(time.time() - self._startTime)),
'monitorResponse': str(self._monitorResponse),
'timeOffset': self.conf.get('timeOffset', '0'),
'clientIp': self.conf.get('clientIp', ''),
- 'network': {},
- 'disks': {}}
+ }
if 'pauseCode' in self.conf:
stats['pauseCode'] = self.conf['pauseCode']
+ if self._watchdogEvent:
+ stats['watchdogEvent'] = self._watchdogEvent
if self.isMigrating():
stats['migrationProgress'] = self.migrateStatus()['progress']
+ return stats
+
+ def _getRunningVmStats(self):
+ """
+ gathers all the stats which can change while a VM is running.
+ """
+ stats = self._getUpVmStats()
+
+ stats['network'] = {}
+ stats['disks'] = {}
decStats = {}
try:
@@ -1414,8 +1422,6 @@
stats.update(self._getGraphicsStats())
stats['hash'] = self._getVmStatusHash()
- if self._watchdogEvent:
- stats['watchdogEvent'] = self._watchdogEvent
if self._numaInfo:
stats['vNodeRuntimeInfo'] = self._numaInfo
if self._vcpuLimit:
--
To view, visit https://gerrit.ovirt.org/43285
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia3e2c19d0b69e78f463976478ded12f306e2f2c7
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Francesco Romani has uploaded a new change for review.
Change subject: vm: factor out the vm hashes
......................................................................
vm: factor out the vm hashes
Extracts the code which reports the devices and guest
hashes in a single, shared and simple helper.
Change-Id: Id05143021d505e815abbd227f58f460cf9b45210
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 7 insertions(+), 4 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/86/43286/1
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 5266b71..7cc4d0b 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -386,8 +386,7 @@
self._evaluatedStatus = vm_status
current_status = self.lastStatus
stats['status'] = vm_status,
- stats['hash'] = str(hash((self._domain.devices_hash,
- self.guestAgent.diskMappingHash)))
+ stats['hash'] = self._getVmStatusHash()
# TODO: DOWN and exitCode must be set atomically. Once this is
# done we can remove the multiple conditions from this code.
@@ -403,6 +402,10 @@
self.log.debug('Last status %s and evaluated status %s',
current_status, vm_status)
self._notify('VM_status', stats)
+
+ def _getVmStatusHash(self):
+ return str(hash((self._domain.devices_hash,
+ self.guestAgent.diskMappingHash)))
def _notify(self, operation, params):
sub_id = '|virt|%s|%s' % (operation, self.id)
@@ -1415,8 +1418,8 @@
self.log.exception("Error setting vm disk stats")
stats.update(self._getGraphicsStats())
- stats['hash'] = str(hash((self._domain.devices_hash,
- self.guestAgent.diskMappingHash)))
+ stats['hash'] = self._getVmStatusHash()
+
if self._watchdogEvent:
stats['watchdogEvent'] = self._watchdogEvent
if self._numaInfo:
--
To view, visit https://gerrit.ovirt.org/43286
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id05143021d505e815abbd227f58f460cf9b45210
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: events: extract function to make event info
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43268
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If60e8422d06fb1ff45a846858856ad68d52fd6ce
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 5: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 5: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on RTC update
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/43310
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2e17262812547ea64365aa4fe77b64d27d579de5
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has abandoned this change.
Change subject: vm: event: send status event on RTC update
......................................................................
Abandoned
abandoned as requested. No need now.
--
To view, visit https://gerrit.ovirt.org/43310
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I2e17262812547ea64365aa4fe77b64d27d579de5
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on watchdog trigger
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/43311
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I390664a5295a51d475902b2ddeaba77d65ad718d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has abandoned this change.
Change subject: vm: event: send status event on watchdog trigger
......................................................................
Abandoned
abandoned as requested. No need now.
--
To view, visit https://gerrit.ovirt.org/43311
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I390664a5295a51d475902b2ddeaba77d65ad718d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: send status event on lifecycle event
......................................................................
Patch Set 3:
(1 comment)
setDownStatus should be enough.
other events we don't really care about
https://gerrit.ovirt.org/#/c/43313/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 4487: elif (detail == libvirt.VIR_DOMAIN_EVENT_RESUMED_MIGRATED and
Line 4488: self.lastStatus == vmstatus.MIGRATION_DESTINATION):
Line 4489: self._incomingMigrationFinished.set()
Line 4490:
Line 4491: self.send_status_event()
u
Line 4492:
Line 4493: def _updateDevicesDomxmlCache(self, xml):
Line 4494: """
Line 4495: Devices cache their device's XML, which is used for per-device
--
To view, visit https://gerrit.ovirt.org/43313
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I46c3ca382a1055ba45618ed9e3d364d971add145
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on lifecycle event
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit https://gerrit.ovirt.org/43313
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I46c3ca382a1055ba45618ed9e3d364d971add145
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has abandoned this change.
Change subject: vm: event: send status event on lifecycle event
......................................................................
Abandoned
no need for safety net. We either want precise events or no events at all.
--
To view, visit https://gerrit.ovirt.org/43313
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I46c3ca382a1055ba45618ed9e3d364d971add145
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: send status event on watchdog trigger
......................................................................
Patch Set 3: Code-Review-1
I wouldn't bother right now. Bigger problem is that we don't update lastStatus nor guestCPURunning
That should be fixed first, but out of scope right now
--
To view, visit https://gerrit.ovirt.org/43311
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I390664a5295a51d475902b2ddeaba77d65ad718d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Michal Skrivanek has posted comments on this change.
Change subject: vm: event: send status event on RTC update
......................................................................
Patch Set 3: Code-Review-1
no need for events on RTC change, we don't use it anymore eanyway
--
To view, visit https://gerrit.ovirt.org/43310
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2e17262812547ea64365aa4fe77b64d27d579de5
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has uploaded a new change for review.
Change subject: sampling: make PidCpuSample a function
......................................................................
sampling: make PidCpuSample a function
make the PidCpuSample class a function, which
returns a CpuSample namedtuple. The surrounding
code doesn't need to change, and the module is
now a tiny bit shorter and simpler.
Change-Id: Iaffb10fa48ee6876156a27c2ad132298fc9d72ae
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/sampling.py
1 file changed, 5 insertions(+), 6 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/50/41250/1
diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index fb99441..6b13913 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -179,16 +179,15 @@
self.nodesMemSample[nodeIndex] = nodeMemSample
-class PidCpuSample(object):
+def _get_pid_cpu_sample(pid):
"""
A sample of the CPU consumption of a process.
The sample is taken at initialization time and can't be updated.
"""
- def __init__(self, pid):
- with open('/proc/%s/stat' % pid) as stat:
- self.user, self.sys = \
- map(int, stat.read().split()[13:15])
+ with open('/proc/%s/stat' % pid) as stat:
+ user, sys = map(int, stat.read().split()[13:15])
+ return CpuSample(user, 0, sys, 0)
class TimedSample(object):
@@ -260,7 +259,7 @@
"""
super(HostSample, self).__init__()
self.interfaces = _get_interfaces_and_samples()
- self.pidcpu = PidCpuSample(pid)
+ self.pidcpu = _get_pid_cpu_sample(pid)
self.totcpu = _get_total_cpu_sample()
meminfo = utils.readMemInfo()
freeOrCached = (meminfo['MemFree'] +
--
To view, visit https://gerrit.ovirt.org/41250
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaffb10fa48ee6876156a27c2ad132298fc9d72ae
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Francesco Romani has uploaded a new change for review.
Change subject: sampling: make TotalCpuSample a function
......................................................................
sampling: make TotalCpuSample a function
make the TotalCpuSample class a function, which
returns a CpuSample namedtuple. The surrounding
code doesn't need to change, and the module is
now a tiny bit shorter and simpler.
Change-Id: I27a4248516811bb6e341e438228b828fa661a767
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/sampling.py
1 file changed, 5 insertions(+), 7 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/49/41249/1
diff --git a/vdsm/virt/sampling.py b/vdsm/virt/sampling.py
index db10656..fb99441 100644
--- a/vdsm/virt/sampling.py
+++ b/vdsm/virt/sampling.py
@@ -122,17 +122,15 @@
['user', 'userNice', 'sys', 'idle'])
-class TotalCpuSample(object):
+def _get_total_cpu_sample():
"""
A sample of total CPU consumption.
The sample is taken at initialization time and can't be updated.
"""
- def __init__(self):
- with open('/proc/stat') as f:
- self.user, userNice, self.sys, self.idle = \
- map(int, f.readline().split()[1:5])
- self.user += userNice
+ with open('/proc/stat') as f:
+ user, userNice, sys, idle = map(int, f.readline().split()[1:5])
+ return CpuSample(user + userNice, userNice, sys, idle)
class CpuCoreSample(object):
@@ -263,7 +261,7 @@
super(HostSample, self).__init__()
self.interfaces = _get_interfaces_and_samples()
self.pidcpu = PidCpuSample(pid)
- self.totcpu = TotalCpuSample()
+ self.totcpu = _get_total_cpu_sample()
meminfo = utils.readMemInfo()
freeOrCached = (meminfo['MemFree'] +
meminfo['Cached'] + meminfo['Buffers'])
--
To view, visit https://gerrit.ovirt.org/41249
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I27a4248516811bb6e341e438228b828fa661a767
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: events: extract function to make event info
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43268
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If60e8422d06fb1ff45a846858856ad68d52fd6ce
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on lifecycle event
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43313
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I46c3ca382a1055ba45618ed9e3d364d971add145
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on watchdog trigger
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43311
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I390664a5295a51d475902b2ddeaba77d65ad718d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on RTC update
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43310
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2e17262812547ea64365aa4fe77b64d27d579de5
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: add minimal running stats to status
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43290
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I894a1b10d3cc3dd33664ea9f0842cbc7f1eb07b3
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has uploaded a new change for review.
Change subject: vm: simplify Vm._getVmStatus method
......................................................................
vm: simplify Vm._getVmStatus method
Vm._getVmStatus returns a dict with just one key/value pair,
being the computed vm status (unsurprisingly).
This was done without a good reason, just to make the caller
look uniform. But more and more we need just the status value,
so the temporary dictionary is wasteful and redundant.
So, this patch simplifies the code to return indeed just the value.
Change-Id: I3f02efa1001aff5fbf01d127b333250500de9d54
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 7 insertions(+), 7 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/67/43267/2
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 826e4e9..4dd8153 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -379,7 +379,7 @@
self._lastStatus = value
def send_status_event(self):
- vm_status = self._getVmStatus()['status']
+ vm_status = self._getVmStatus()
stats = {}
with self._eventLock:
if vm_status != self._evaluatedStatus:
@@ -1327,7 +1327,7 @@
else:
stats.update(self._getConfigVmStats())
stats.update(self._getRunningVmStats())
- stats.update(self._getVmStatus())
+ stats['status'] = self._getVmStatus()
stats.update(self._getGuestStats())
return stats
@@ -1474,16 +1474,16 @@
vmstatus.MIGRATION_SOURCE, vmstatus.MIGRATION_DESTINATION,
vmstatus.PAUSED)
if self.lastStatus in statuses:
- return {'status': self.lastStatus}
+ return self.lastStatus
elif self.isMigrating():
if self._migrationSourceThread.hibernating:
- return {'status': vmstatus.SAVING_STATE}
+ return vmstatus.SAVING_STATE
else:
- return {'status': vmstatus.MIGRATION_SOURCE}
+ return vmstatus.MIGRATION_SOURCE
elif self.lastStatus == vmstatus.UP:
- return {'status': _getVmStatusFromGuest()}
+ return _getVmStatusFromGuest()
else:
- return {'status': self.lastStatus}
+ return self.lastStatus
def _getGraphicsStats(self):
def getInfo(dev):
--
To view, visit https://gerrit.ovirt.org/43267
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f02efa1001aff5fbf01d127b333250500de9d54
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Nir Soffer has posted comments on this change.
Change subject: freeze: Avoid unneeded freeze during snapshot
......................................................................
Patch Set 5:
This version makes creating the return value little nicer.
--
To view, visit https://gerrit.ovirt.org/43293
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I328470f700ab74daeed5375c0ade83628d16c387
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: freeze: Avoid unneeded freeze during snapshot
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43293
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I328470f700ab74daeed5375c0ade83628d16c387
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on setDownStatus
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43270
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a99d074461bcc2b61d15dd46f7d020abbe48afe
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on domDependentInit
......................................................................
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43269
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie379911a374a9ba3e82658592c295f7a9a0ac8a0
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on lifecycle event
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43313
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I46c3ca382a1055ba45618ed9e3d364d971add145
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on reboot
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43309
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I748b187228a39e55982c1b7d05ab47681481801c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on watchdog trigger
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43311
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I390664a5295a51d475902b2ddeaba77d65ad718d
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on RTC update
......................................................................
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43310
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2e17262812547ea64365aa4fe77b64d27d579de5
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: add minimal running stats to status
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43290
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I894a1b10d3cc3dd33664ea9f0842cbc7f1eb07b3
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: emit event on guest status change
......................................................................
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43289
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib98d64a250a6d2de70f7fe5206e5a0ad648c151d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
Francesco Romani has uploaded a new change for review.
Change subject: vm: event: do not send status event on startup
......................................................................
vm: event: do not send status event on startup
We send an explicit event, so no need to send implicitely
when we receive something from the libvirt event loop.
Change-Id: I2d2c1b1d1132e1d3792b1a8c8f9241b6e6eb3164
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/88/43288/1
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 6f9a2d7..d453ff9 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -799,7 +799,6 @@
else:
self.log.exception("The vm start process failed")
self.setDownStatus(ERROR, vmexitreason.GENERIC_ERROR, str(e))
- self.send_status_event()
def _incomingMigrationPending(self):
return 'migrationDest' in self.conf or 'restoreState' in self.conf
--
To view, visit https://gerrit.ovirt.org/43288
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2d2c1b1d1132e1d3792b1a8c8f9241b6e6eb3164
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Francesco Romani has posted comments on this change.
Change subject: vm: event: introduce _send_event helper
......................................................................
Patch Set 2: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/43287/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 382: self._send_event(
Line 383: self._getVmStatus()
Line 384: )
Line 385:
Line 386: def _send_event(self, vm_status, **kwargs);
typo!
Line 387: event_data = {}
Line 388: with self._eventLock:
Line 389: if vm_status != self._evaluatedStatus:
Line 390: self._evaluatedStatus = vm_status
--
To view, visit https://gerrit.ovirt.org/43287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c25ba9f520e6409cc06e99a2450d2c0d98c26fb
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
Martin Polednik has uploaded a new change for review.
Change subject: virt: fix VM creation logic condition
......................................................................
virt: fix VM creation logic condition
We want to look up the CPU in tuple of CPUs, but we currently supply
string instead of tuple.
Change-Id: Ie1eb15f9865dadab791be97b7d009d5bae025265
Signed-off-by: Martin Polednik <mpolednik(a)redhat.com>
---
M vdsm/virt/vmxml.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/70/43170/1
diff --git a/vdsm/virt/vmxml.py b/vdsm/virt/vmxml.py
index 7b9ab9c..b9ff3be 100644
--- a/vdsm/virt/vmxml.py
+++ b/vdsm/virt/vmxml.py
@@ -337,7 +337,7 @@
cpu = Element('cpu')
- if self.arch in (caps.Architecture.X86_64):
+ if self.arch in (caps.Architecture.X86_64,):
cpu.setAttrs(match='exact')
features = self.conf.get('cpuType', 'qemu64').split(',')
--
To view, visit https://gerrit.ovirt.org/43170
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie1eb15f9865dadab791be97b7d009d5bae025265
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Nir Soffer has posted comments on this change.
Change subject: ceph: Support OVF_STORE upload for ceph.
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/43296/2/vdsm/rpc/bindingxmlrpc.py
File vdsm/rpc/bindingxmlrpc.py:
Line 189: self.HEADER_RBD_IMAGE)
Line 190: methodArgs['rbdHosts'] = self.headers.getheader(
Line 191: self.HEADER_RBD_HOSTS)
Line 192: methodArgs['rbdKey'] = self.headers.getheader(
Line 193: self.HEADER_RBD_KEY)
We need same code for do_GET, lets extract it to _get_rbd_headers().
Line 194:
Line 195: uploadFinishedEvent, operationEndCallback = \
Line 196: self._createEventWithCallback()
Line 197:
--
To view, visit https://gerrit.ovirt.org/43296
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iffcc1af9861a5580786340c2e5a325d1663b3a5e
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
Nir Soffer has posted comments on this change.
Change subject: freeze: Avoid unneeded freeze during snapshot
......................................................................
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/43293/4/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:
Line 7881: # @snapMemory: #optional str representation of the volumes in which
Line 7882: # the memory dump and vm configuration should be saved
Line 7883: #
Line 7884: # @frozen: #optional Boolean value set to true if the vm is frozen
Line 7885: # and does not need to be freezed during the snapshot.
> please add
Sure, good catch.
Line 7886: #
Line 7887: # Since: 4.10.0
Line 7888: ##
Line 7889: {'command': {'class': 'VM', 'name': 'snapshot'},
--
To view, visit https://gerrit.ovirt.org/43293
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I328470f700ab74daeed5375c0ade83628d16c387
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes
automation(a)ovirt.org has posted comments on this change.
Change subject: virt: clientif: remove generic vm status event
......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43314
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I392857f2e4f02c14f88aca85af17861e84b2f6fa
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on lifecycle event
......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43313
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I46c3ca382a1055ba45618ed9e3d364d971add145
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on IO Error
......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43312
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4be90f1267e82b8af2abb36c06cfd9a70fb558
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
automation(a)ovirt.org has posted comments on this change.
Change subject: vm: event: send status event on watchdog trigger
......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/43311
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I390664a5295a51d475902b2ddeaba77d65ad718d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No