Hello Dan Kenigsberg,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/43499
to review the following change.
Change subject: net: tests: only test kernel config if unified persistence
..
net:
automat...@ovirt.org has posted comments on this change.
Change subject: net: tests: only test kernel config if unified persistence
..
Patch Set 1:
* Update tracker::#1203422::OK
* Check Bug-Url::OK
* Check Public
Ido Barkan has posted comments on this change.
Change subject: net: tests: only test kernel config if unified persistence
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/43499
To unsubscribe, visit
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: no need to call setupNetworks with empty argument during
network restoration
..
no need to call setupNetworks with empty argument during network
Yaniv Bronhaim has submitted this change and it was merged.
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
automat...@ovirt.org has posted comments on this change.
Change subject: net: restore-nets should support ONBOOT=no
..
Patch Set 4:
* Update tracker::#1203422::OK
* Check TR::#1203422::OK
* Set MODIFIED::bug
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: net: ignore bondingOptions in persistent networks.
..
net: ignore bondingOptions in persistent networks.
Up until https://gerrit.ovirt.org/#/c/42146/
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: net: support strings for 'bridged' in kernelConfig
..
net: support strings for 'bridged' in kernelConfig
Because kernelConfig did not know how to
automat...@ovirt.org has posted comments on this change.
Change subject: net: support strings for 'bridged' in kernelConfig
..
Patch Set 3:
* Update tracker::#1237032::OK
* Check TR::#1237032::ERROR, 3.6.0 should match
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: net: flatten ifcfg.ConfigWriter
..
net: flatten ifcfg.ConfigWriter
Make startDevices and stopDevices public for further use.
Also, promotoe them to be
Francesco Romani has posted comments on this change.
Change subject: vm: events: do not send hashes in status messages
..
Patch Set 7: Verified+1
(trivially) verified that an events do not have the hash anymore.
--
To view,
automat...@ovirt.org has posted comments on this change.
Change subject: vm: run before_vm_create hook only in the VM boot
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
--
To view, visit
Ido Barkan has posted comments on this change.
Change subject: net: if all devices are up, skip ifup during restoratrion.
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/43435
To unsubscribe, visit
Ido Barkan has posted comments on this change.
Change subject: net: if all devices are up, skip ifup during restoratrion.
..
Patch Set 1:
yes. on my minidell every restoration now take about 3 seconds.
--
To view, visit
Dan Kenigsberg has posted comments on this change.
Change subject: net: wait for restored devices to be up
..
Patch Set 6: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43222
To unsubscribe, visit
Dan Kenigsberg has posted comments on this change.
Change subject: net: flatten ifcfg.ConfigWriter
..
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43381
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: net: tests: skip testRestoreToBlockingDHCP if ifcfg persistence
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug
Ido Barkan has posted comments on this change.
Change subject: net: tests: skip testRestoreToBlockingDHCP if ifcfg persistence
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/43441/1/tests/functional/networkTests.py
Dan Kenigsberg has posted comments on this change.
Change subject: net: tests: skip testRestoreToBlockingDHCP if ifcfg persistence
..
Patch Set 2: Code-Review-1
(1 comment)
Freddy Rolland has posted comments on this change.
Change subject: resize-lun: Rescan fiber channel devices
..
Patch Set 3:
(1 comment)
https://gerrit.ovirt.org/#/c/43388/3/vdsm/storage/fc-scan
File vdsm/storage/fc-scan:
Freddy Rolland has posted comments on this change.
Change subject: resize-lun: Rescan fiber channel devices
..
Patch Set 4: Verified+1
--
To view, visit https://gerrit.ovirt.org/43388
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
Patch Set 1: Verified-1
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::ERROR, At least one
Ido Barkan has uploaded a new change for review.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
net: _upgrade_onboot failed on python 2.6.6
The flags keyword argument was added only on python 2.7.
Change-Id:
automat...@ovirt.org has posted comments on this change.
Change subject: resize-lun: Fix resizing when vg has multiple pvs
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
Dan Kenigsberg has submitted this change and it was merged.
Change subject: resize-lun: Fix resizing when vg has multiple pvs
..
resize-lun: Fix resizing when vg has multiple pvs
When resizing a PV, LVM needs the other PVs
Francesco Romani has posted comments on this change.
Change subject: concurrent: Add Barrier class
..
Patch Set 6: Code-Review+1
seems OK and nicely tested
--
To view, visit https://gerrit.ovirt.org/42927
To unsubscribe,
automat...@ovirt.org has posted comments on this change.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
Patch Set 3:
* Update tracker::#1203422::OK
* Check Bug-Url::OK
* Check Public Bug::#1203422::OK, public
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: fix hostdev addressing
..
Patch Set 10:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
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
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
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
Francesco Romani has posted comments on this change.
Change subject: tests: Add RWLock tests
..
Patch Set 9: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/42907
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: net: tests: only test kernel config if unified persistence
..
Patch Set 2:
* Update tracker::#1203422::OK
* Set MODIFIED::bug
Dan Kenigsberg has posted comments on this change.
Change subject: net: tests: only test kernel config if unified persistence
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43499
To unsubscribe,
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: tests: only test kernel config if unified persistence
..
net: tests: only test kernel config if unified persistence
Testing KernelConfig does not
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: net: restore-nets should support ONBOOT=no
..
net: restore-nets should support ONBOOT=no
on 3.5.x devices were persisted with ONBOOT=no. Hence, the
automat...@ovirt.org has posted comments on this change.
Change subject: no need to call setupNetworks with empty argument during
network restoration
..
Patch Set 4:
* Update tracker::#1203422::OK
* Set MODIFIED::bug
automat...@ovirt.org has posted comments on this change.
Change subject: net: wait for restored devices to be up
..
Patch Set 7:
* Update tracker::#1203422::OK
* Set MODIFIED::bug 1203422#1203422IGNORE, not all related
automat...@ovirt.org has posted comments on this change.
Change subject: net: flatten ifcfg.ConfigWriter
..
Patch Set 4:
* Update tracker::#1203422::OK
* Set MODIFIED::bug 1203422#1203422IGNORE, not all related patches
Francesco Romani has posted comments on this change.
Change subject: vdsm hooks: report hook stderr to Engine if it fails an action
..
Patch Set 18: Code-Review+1
(2 comments)
seems OK. Please state in the comments how you
Francesco Romani has posted comments on this change.
Change subject: vm: event: add arguments to send_status_event
..
Patch Set 4: Verified+1
verified with 43269 and 43270: the notification events has the extra fields.
--
To
Martin Polednik has posted comments on this change.
Change subject: hostdev: fix hostdev addressing
..
Patch Set 10: Verified+1
Addresses are reported correctly - meaning the VM can be freely restarted and
it's start won't
Dan Kenigsberg has submitted this change and it was merged.
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
automat...@ovirt.org has posted comments on this change.
Change subject: net: if all devices are up, skip ifup during restoratrion.
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url
automat...@ovirt.org has posted comments on this change.
Change subject: resize-lun: Rescan fiber channel devices
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Dan Kenigsberg has posted comments on this change.
Change subject: net: tests: skip testRestoreToBlockingDHCP if ifcfg persistence
..
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43441
To unsubscribe,
Francesco Romani has posted comments on this change.
Change subject: hostdev: inline source address function
..
Patch Set 2: Code-Review+1
thanks!
--
To view, visit https://gerrit.ovirt.org/43379
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: hostdev: fix hostdev addressing
..
Patch Set 10: Code-Review+1
Thanks for the updates, I think it's good now.
--
To view, visit
Petr Horáček has posted comments on this change.
Change subject: net: tests: only test kernel config if unified persistence
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43499
To unsubscribe, visit
Ido Barkan has posted comments on this change.
Change subject: net: support strings for 'bridged' in kernelConfig
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/43446
To unsubscribe, visit
Ido Barkan has posted comments on this change.
Change subject: net: restore-nets should support ONBOOT=no
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/43382
To unsubscribe, visit
Dan Kenigsberg has posted comments on this change.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/43507
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: virt: migration: create full status in migration.py
..
Patch Set 2: Code-Review-1
(1 comment)
-1 for visibility
v2 seems just a rebase, please check my
Dan Kenigsberg has posted comments on this change.
Change subject: net: restore-nets should support ONBOOT=no
..
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43382
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: net: tests: skip testRestoreToBlockingDHCP if ifcfg persistence
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug
Francesco Romani has abandoned this change.
Change subject: vm: run before_vm_create hook only in the VM boot
..
Abandoned
I guess this pet peeve of mine should wait better times.
--
To view, visit
Francesco Romani has posted comments on this change.
Change subject: virt: make vm events callback public
..
Patch Set 5:
Reason for Jenkins CI Failure:
DEBUG util.py:378: Traceback (most recent call last):
DEBUG
automat...@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
automat...@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
automat...@ovirt.org has posted comments on this change.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
Patch Set 2: -Verified
* Update tracker::#1203422::OK
* Check Bug-Url::OK
* Check Public
Francesco Romani has posted comments on this change.
Change subject: virt: migration: create full status in migration.py
..
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/43357/1/vdsm/API.py
File vdsm/API.py:
Line
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: add tests for guest addressing
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Freddy Rolland has posted comments on this change.
Change subject: resize-lun: Rescan fiber channel devices
..
Patch Set 5: Verified+1
--
To view, visit https://gerrit.ovirt.org/43388
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: inline source address function
..
Patch Set 6:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Dan Kenigsberg has posted comments on this change.
Change subject: testlib: Use nose.case.Test.getDescription()
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43431
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: testlib: Show fully qualified class name
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To
automat...@ovirt.org has posted comments on this change.
Change subject: virt: add logic for POWER cpu xml element
..
Patch Set 7:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Dan Kenigsberg has posted comments on this change.
Change subject: net: tests: clean legacy bonds from kernelConfig
..
Patch Set 1:
(2 comments)
nits about comments
https://gerrit.ovirt.org/#/c/43513/1//COMMIT_MSG
Commit
automat...@ovirt.org has posted comments on this change.
Change subject: vdsm hooks: report hook stderr to Engine if it fails an action
..
Patch Set 19:
* Update tracker::#1219630::OK
* Check Bug-Url::OK
* Check Public
Francesco Romani has posted comments on this change.
Change subject: hostdev: add basic unit tests for XML
..
Patch Set 4: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/43088/4/tests/hostdevTests.py
File
Francesco Romani has posted comments on this change.
Change subject: hostdev: add tests for guest addressing
..
Patch Set 3: Code-Review+1
ok, I see your point. I can't think of an wasy way to inject address, so let's
move
automat...@ovirt.org has posted comments on this change.
Change subject: net: fix python 2.7 specific code
..
Patch Set 1:
* Update tracker::#1203422::OK
* Check Bug-Url::OK
* Check Public Bug::#1203422::OK, public bug
* Check
automat...@ovirt.org has posted comments on this change.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
Patch Set 4:
* Update tracker::#1203422::OK
* Check Bug-Url::OK
* Check Public Bug::#1203422::OK, public
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
Ido Barkan has uploaded a new change for review.
Change subject: net: fix python 2.7 specific code
..
net: fix python 2.7 specific code
This syntax is only supported in python 2.7 hence should be removed
from 3.5 branch which
Dan Kenigsberg has submitted this change and it was merged.
Change subject: testlib: Remove unneeded methods
..
testlib: Remove unneeded methods
VdsmTestRunner was implementing __init__() and run() for no reason.
Maybe we used
Dan Kenigsberg has posted comments on this change.
Change subject: testlib: Remove unneeded methods
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43430
To unsubscribe, visit
Francesco Romani has posted comments on this change.
Change subject: virt: add logic for POWER cpu xml element
..
Patch Set 6: Code-Review+1
looks OK to me
--
To view, visit https://gerrit.ovirt.org/42736
To unsubscribe,
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: add basic unit tests for XML
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Martin Polednik has posted comments on this change.
Change subject: hostdev: add basic unit tests for XML
..
Patch Set 4: Verified+1
--
To view, visit https://gerrit.ovirt.org/43088
To unsubscribe, visit
Dan Kenigsberg has posted comments on this change.
Change subject: net: _upgrade_onboot failed on python 2.6.6
..
Patch Set 4:
but any idea what's
14:13:15 ERROR: test_parseVolumeStatus (gluster_cli_tests.GlusterCliTests)
automat...@ovirt.org has posted comments on this change.
Change subject: virt: migration: create full status in migration.py
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found,
Shmuel Leib Melamud has posted comments on this change.
Change subject: api: 'device' property for RNG device
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/43166
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: api: 'device' property for RNG device
..
Patch Set 2:
* Update tracker::#1233825::OK
* Check Bug-Url::OK
* Check Public Bug::#1233825::OK, public bug
*
automat...@ovirt.org has posted comments on this change.
Change subject: resize-lun: Rescan fiber channel devices
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: inline source address function
..
Patch Set 4:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
automat...@ovirt.org has posted comments on this change.
Change subject: net: tests: clean legacy bonds from kernelConfig
..
Patch Set 1:
* Update tracker::#1203422::OK
* Check Bug-Url::OK
* Check Public Bug::#1203422::OK,
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: fix hostdev addressing
..
Patch Set 11:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: inline source address function
..
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: add basic unit tests for XML
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Dan Kenigsberg has posted comments on this change.
Change subject: net: test_setupNetworks_bond_with_custom_option with ifcfg
persistence
..
Patch Set 1:
(1 comment)
Nir Soffer has posted comments on this change.
Change subject: resize-lun: Rescan fiber channel devices
..
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/43388/5/vdsm/storage/fc-scan
File vdsm/storage/fc-scan:
Line
automat...@ovirt.org has posted comments on this change.
Change subject: testlib: Use nose.case.Test.getDescription()
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make
automat...@ovirt.org has posted comments on this change.
Change subject: testlib: Remove unneeded methods
..
Patch Set 2:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: inline source address function
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Dan Kenigsberg has posted comments on this change.
Change subject: net: fix python 2.7 specific code
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43512
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: fix hostdev addressing
..
Patch Set 12:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Dan Kenigsberg has posted comments on this change.
Change subject: testlib: Show fully qualified class name
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/43432
To unsubscribe, visit
Martin Polednik has posted comments on this change.
Change subject: hostdev: inline source address function
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/43379
To unsubscribe, visit
automat...@ovirt.org has posted comments on this change.
Change subject: hostdev: add tests for guest addressing
..
Patch Set 3:
* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
1 - 100 of 282 matches
Mail list logo