Change in vdsm[ovirt-3.6]: storage: don't chmod local storage symlink

2015-08-17 Thread gpadgett
Hello Nir Soffer, Greg Padgett, I'd like you to do a code review. Please visit https://gerrit.ovirt.org/44990 to review the following change. Change subject: storage: don't chmod local storage symlink .. storage: don't

Change in vdsm[ovirt-3.6]: storage: don't chmod local storage symlink

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: storage: don't chmod local storage symlink .. Patch Set 1: * Update tracker::#1112171::OK * Check Bug-Url::OK * Check Public Bug::#1112171::OK, public

Change in vdsm[ovirt-3.6]: storage: don't chmod local storage symlink

2015-08-17 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: storage: don't chmod local storage symlink .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/44990 To unsubscribe, visit

Change in vdsm[master]: v2v: get VM information from OVA file

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: v2v: get VM information from OVA file .. Patch Set 12: * Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[ovirt-3.6]: v2v: Import VM from OVA file

2015-08-17 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: v2v: Import VM from OVA file .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/44914 To unsubscribe, visit

Change in vdsm[ovirt-3.6]: v2v: get VM information from OVA file

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: v2v: get VM information from OVA file .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[ovirt-3.6]: v2v: get VM information from OVA file

2015-08-17 Thread shavivi
Hello Francesco Romani, I'd like you to do a code review. Please visit https://gerrit.ovirt.org/44913 to review the following change. Change subject: v2v: get VM information from OVA file .. v2v: get VM information from

Change in vdsm[ovirt-3.6]: v2v: Import VM from OVA file

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: v2v: Import VM from OVA file .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[ovirt-3.6]: v2v: Import VM from OVA file

2015-08-17 Thread shavivi
Hello Francesco Romani, I'd like you to do a code review. Please visit https://gerrit.ovirt.org/44914 to review the following change. Change subject: v2v: Import VM from OVA file .. v2v: Import VM from OVA file OVA is a

Change in vdsm[master]: v2v: get VM information from OVA file

2015-08-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: v2v: get VM information from OVA file .. v2v: get VM information from OVA file In order to import a VM that exists in an OVA file the oVirt engine

Change in vdsm[master]: hooks: Open vSwitch configurator

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: hooks: Open vSwitch configurator .. Patch Set 63: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: multipath: Backup old configuration using hard link

2015-08-17 Thread nsoffer
Nir Soffer has abandoned this change. Change subject: multipath: Backup old configuration using hard link .. Abandoned Will not work on node, and there is no reason to invest time in this. -- To view, visit

Change in vdsm[master]: multipath: Backup old configuration using hard link

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: multipath: Backup old configuration using hard link .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found -- To view, visit

Change in vdsm[master]: net: tests: enable tests to run with OVS

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: tests: enable tests to run with OVS .. Patch Set 5: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: vm: graphics: fix console open after restore

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: graphics: fix console open after restore .. Patch Set 4: Martin: sure make sense! will polish in the next upload (if any, see previous comment) -- To

Change in vdsm[master]: vm: graphics: fix console open after restore

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: graphics: fix console open after restore .. Patch Set 4: Verified-1 failed verification. restoreFlags doesn't allow us to fix the bit we need, hence

Change in vdsm[master]: v2v: Import VM from OVA file

2015-08-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: v2v: Import VM from OVA file .. v2v: Import VM from OVA file OVA is a tar file which contain a VM with its Ovf file and its disks as well. Change-Id:

Change in vdsm[master]: v2v: Import VM from OVA file

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: v2v: Import VM from OVA file .. Patch Set 12: * Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[ovirt-3.6]: v2v: get VM information from OVA file

2015-08-17 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: v2v: get VM information from OVA file .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/44913 To unsubscribe, visit

Change in vdsm[master]: hooks: Open vSwitch configurator

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: hooks: Open vSwitch configurator .. Patch Set 62: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: concurrent: Introduce concurrent.thread() utility

2015-08-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: concurrent: Introduce concurrent.thread() utility .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/44894/1/lib/vdsm/concurrent.py File

Change in vdsm[master]: concurrent: Introduce concurrent.thread() utility

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: concurrent: Introduce concurrent.thread() utility .. Patch Set 1: Code-Review+1 (1 comment) https://gerrit.ovirt.org/#/c/44894/1/lib/vdsm/concurrent.py File

Change in vdsm[master]: concurrent: Use new concurrent.thread() utility

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: concurrent: Use new concurrent.thread() utility .. Patch Set 2: Code-Review+1 yep, code looks nicer. -- To view, visit https://gerrit.ovirt.org/44895 To

Change in vdsm[master]: StorageDomainManifest: deleteImage and deps

2015-08-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: StorageDomainManifest: deleteImage and deps .. Patch Set 21: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/42268 To unsubscribe, visit

Change in vdsm[master]: StorageDomainManifest: move extend and extendVolume

2015-08-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: StorageDomainManifest: move extend and extendVolume .. Patch Set 21: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/42266 To unsubscribe, visit

Change in vdsm[master]: StorageDomainManifest: move image and volume enumeration

2015-08-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: StorageDomainManifest: move image and volume enumeration .. Patch Set 21: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/42269 To unsubscribe, visit

Change in vdsm[ovirt-3.6]: storage: don't chmod local storage symlink

2015-08-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: storage: don't chmod local storage symlink .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/44990 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/44926 To unsubscribe, visit

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 2: Verified+1 -- To view, visit https://gerrit.ovirt.org/44929 To unsubscribe, visit

Change in vdsm[master]: multipath: Write multipath.conf atomically

2015-08-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: multipath: Write multipath.conf atomically .. Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/44869/3/lib/vdsm/tool/configurators/multipath.py File

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: net: tests: make monitor test_events_keys more readable

2015-08-17 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: net: tests: make monitor test_events_keys more readable .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/44927 To unsubscribe, visit

Change in vdsm[master]: StorageDomainManifest: move extend and extendVolume

2015-08-17 Thread alitke
Adam Litke has posted comments on this change. Change subject: StorageDomainManifest: move extend and extendVolume .. Patch Set 20: (1 comment) https://gerrit.ovirt.org/#/c/42266/20/tests/sdm_indirection_tests.py File

Change in vdsm[master]: WIP virt: add and use vm.acpiReboot() method

2015-08-17 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: WIP virt: add and use vm.acpiReboot() method .. WIP virt: add and use vm.acpiReboot() method WRITEME Change-Id: I5d6dd3a81cd4ad981a839849a4a2914e35a53fa6

Change in vdsm[master]: WIP virt: add and use vm.acpiReboot() method

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: WIP virt: add and use vm.acpiReboot() method .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: vm: make acpiShutdown more robust

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vm: make acpiShutdown more robust .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: multipath: Write multipath.conf atomically

2015-08-17 Thread fabiand
Fabian Deutsch has posted comments on this change. Change subject: multipath: Write multipath.conf atomically .. Patch Set 3: Code-Review+1 (1 comment)

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread ibarkan
Ido Barkan has uploaded a new change for review. Change subject: net: always persist ifcfg files. .. net: always persist ifcfg files. Currently the persistence of ifcfg files is only done during setSafeNetworkConfig. But, if

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 1: Verified-1 * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::ERROR, At least one bug-url is

Change in vdsm[master]: net: tests: make monitor test_events_keys more readable

2015-08-17 Thread phoracek
Petr Horáček has uploaded a new change for review. Change subject: net: tests: make monitor test_events_keys more readable .. net: tests: make monitor test_events_keys more readable When this tests fails, print lists of

Change in vdsm[master]: net: tests: make monitor test_events_keys more readable

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: tests: make monitor test_events_keys more readable .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url

Change in vdsm[master]: makefile: Run pep8/pyflakes inside virtualenv

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: makefile: Run pep8/pyflakes inside virtualenv .. Patch Set 1: * Update tracker::#1182088::OK * Check Bug-Url::OK * Check Public Bug::#1182088::OK, public

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread ibarkan
Ido Barkan has uploaded a new change for review. Change subject: net: always persist ifcfg files. .. net: always persist ifcfg files. Currently the persistence of ifcfg files is only done during setSafeNetworkConfig. But, if

Change in vdsm[master]: virt: rename {Notifying, Disconnected}VirDomain

2015-08-17 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: virt: rename {Notifying,Disconnected}VirDomain .. virt: rename {Notifying,Disconnected}VirDomain Now that these classes resides in virdomain module, we can

Change in vdsm[master]: virt: cosmetic fixes

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: cosmetic fixes .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: WIP: RFC: virt: add VmTracker

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: WIP: RFC: virt: add VmTracker .. Patch Set 7: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: periodic: factor out periodic operations

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: periodic: factor out periodic operations .. Patch Set 6: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: vm: periodic: NumaInfoMonitor doesn't need jobs

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vm: periodic: NumaInfoMonitor doesn't need jobs .. Patch Set 5: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: vmstats: network: avoid ZeroDivisionError

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: network: avoid ZeroDivisionError .. Patch Set 5: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: WIP: RFC: track domain availability

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: WIP: RFC: track domain availability .. Patch Set 8: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: make virdomain.Notifying inherit from object

2015-08-17 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: virt: make virdomain.Notifying inherit from object .. virt: make virdomain.Notifying inherit from object The only known reason virdomain.Notifying() is not

Change in vdsm[master]: v2v: use the response module

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: v2v: use the response module .. Patch Set 24: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: m2c: rename m2c modules

2015-08-17 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: m2c: rename m2c modules .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/44493 To unsubscribe, visit https://gerrit.ovirt.org/settings

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 2: Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/44929 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: always persist ifcfg files. .. net: always persist ifcfg files. Currently the persistence of ifcfg files is only done during

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: always persist ifcfg files. .. net: always persist ifcfg files. Currently the persistence of ifcfg files is only done during

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 3: * Update tracker::#1252268::OK * Set MODIFIED::bug 1252268#1252268IGNORE, not oVirt prod but Red

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 3: * Update tracker::#1252268::OK * Set MODIFIED::bug 1252268#1252268IGNORE, not oVirt prod but Red

Change in vdsm[master]: virt: make virdomain.Notifying inherit from object

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: make virdomain.Notifying inherit from object .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found,

Change in vdsm[master]: virt: periodic: factor out periodic operations

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: periodic: factor out periodic operations .. Patch Set 6: Verified+1 verified running patched vdsm and inspecting the output of vdsClient -s 0

Change in vdsm[master]: vm: do not log known missing alias

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vm: do not log known missing alias .. Patch Set 6: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vm: remove unused 'output' value in response

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vm: remove unused 'output' value in response .. Patch Set 15: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: periodic: make runnable a method

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: periodic: make runnable a method .. Patch Set 2: Verified+1 verified running patched VDSM and checking the logs for errors (none in sight) -- To view,

Change in vdsm[master]: schema: add v2vJobs to HostStats

2015-08-17 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: schema: add v2vJobs to HostStats .. schema: add v2vJobs to HostStats v2vJobs reports via HostStats and needs to be declared in schema Change-Id:

Change in vdsm[master]: schema: add v2vJobs to HostStats

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: schema: add v2vJobs to HostStats .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: v2v: add test for getOvaInfo verb

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: v2v: add test for getOvaInfo verb .. Patch Set 5: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: WIP virt: add and use vm.acpiReboot() method

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: WIP virt: add and use vm.acpiReboot() method .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: vm: make acpiShutdown more robust

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vm: make acpiShutdown more robust .. Patch Set 4: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: net: always persist ifcfg files.

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: net: always persist ifcfg files.

2015-08-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: always persist ifcfg files. .. net: always persist ifcfg files. Currently the persistence of ifcfg files is only done during

Change in vdsm[master]: periodic: make runnable a method

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: periodic: make runnable a method .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: rename {Notifying, Disconnected}VirDomain

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: rename {Notifying,Disconnected}VirDomain .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[ovirt-3.6]: net: always persist ifcfg files.

2015-08-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/44929 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 2: Code-Review+2 Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/44926 To unsubscribe, visit

Change in vdsm[master]: virt: periodic: avoid NotConnectedError noise

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: periodic: avoid NotConnectedError noise .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: virt: make virdomain.Notifying inherit from object

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: make virdomain.Notifying inherit from object .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found,

Change in vdsm[master]: vmstats: make nic_traffic private

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: make nic_traffic private .. Patch Set 5: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vmstats: handle known-missing stats

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: handle known-missing stats .. Patch Set 10: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: introduce virdomain module

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: introduce virdomain module .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vmstats: handle known-missing stats

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmstats: handle known-missing stats .. Patch Set 10: -Code-Review with (artificially slowed down) migration in progress: domstats on dest side: Domain:

Change in vdsm[master]: WIP: RFC: virt: add VmTracker

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: WIP: RFC: virt: add VmTracker .. Patch Set 8: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: cosmetic fixes

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: cosmetic fixes .. Patch Set 4: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: periodic: make VmDispatcher ignore TooManyTasks

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: periodic: make VmDispatcher ignore TooManyTasks .. Patch Set 29: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: vmstats: handle known-missing stats

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vmstats: handle known-missing stats .. Patch Set 11: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: periodic: make runnable a method

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: periodic: make runnable a method .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: introduce virdomain module

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: introduce virdomain module .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: virt: periodic: factor out periodic operations

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: periodic: factor out periodic operations .. Patch Set 7: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: virt: periodic: avoid NotConnectedError noise

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: periodic: avoid NotConnectedError noise .. Patch Set 3: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: virt: rename {Notifying, Disconnected}VirDomain

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: rename {Notifying,Disconnected}VirDomain .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: WIP: RFC: track domain availability

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: WIP: RFC: track domain availability .. Patch Set 9: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: net: tests: make monitor test_events_keys more readable

2015-08-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: tests: make monitor test_events_keys more readable .. Patch Set 2: 14:29:24 + git cherry-pick FETCH_HEAD 14:29:24 error: could not apply 7b15e79... Adding

Change in vdsm[ovirt-3.5]: net: always persist ifcfg files.

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: net: always persist ifcfg files. .. Patch Set 2: -Verified * Update tracker::#1252268::OK * Check Bug-Url::OK * Check Public Bug::#1252268::OK, public

Change in vdsm[master]: tests: add tests for disk hotplug/hotunplug

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: tests: add tests for disk hotplug/hotunplug .. Patch Set 19: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make

Change in vdsm[master]: vm: remove _reportError

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: vm: remove _reportError .. Patch Set 30: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: clientIF: add logs during the recovery

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: clientIF: add logs during the recovery .. Patch Set 14: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: virt: fully convert to response module

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: virt: fully convert to response module .. Patch Set 14: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: numa: make use of DisconnectedVirDomain

2015-08-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: numa: make use of DisconnectedVirDomain .. Patch Set 4: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: StorageDomainManifest: deleteImage and deps

2015-08-17 Thread alitke
Adam Litke has posted comments on this change. Change subject: StorageDomainManifest: deleteImage and deps .. Patch Set 20: (3 comments) https://gerrit.ovirt.org/#/c/42268/20/tests/sdm_indirection_tests.py File

Change in vdsm[master]: vmstats: handle known-missing stats

2015-08-17 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmstats: handle known-missing stats .. Patch Set 10: Verified+1 (4 comments) verified running a slowd down migration (migration_max_bandwidth=1) and

Change in vdsm[master]: netinfo: Replace misused asserts with InvalidConfiguration

2015-08-17 Thread ibarkan
Ido Barkan has posted comments on this change. Change subject: netinfo: Replace misused asserts with InvalidConfiguration .. Patch Set 3: Gladly! But begin with writing the tests please ☺ -- To view, visit

  1   2   >