oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: Get size information of a gluster volume.
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7004/ : FAILUR
Darshan N has uploaded a new change for review.
Change subject: gluster: Get size information of a gluster volume.
..
gluster: Get size information of a gluster volume.
New vdsm gluster verb to get free, used and total size of
g
Ondřej Svoboda has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 20:
(2 comments)
The functional tests are being solved by making sure only the expected
Dan Kenigsberg has posted comments on this change.
Change subject: virt: migration: move threads in a separate module
..
Patch Set 10: Code-Review-1
my pep8 cleanups must be standing in your way. please rebase carefully.
--
T
Dan Kenigsberg has posted comments on this change.
Change subject: virt: migration: move threads in a separate module
..
Patch Set 10: Code-Review+2
Anything that breaks vm.py must be good.
--
To view, visit http://gerrit.ovi
Dan Kenigsberg has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 20:
(obviously, the most important thing is to understand why
testSetupNetworksAddNetw
Dan Kenigsberg has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 20: Code-Review-1
(2 comments)
http://gerrit.ovirt.org/#/c/24456/20/lib/vdsm/netinfo.py
Dan Kenigsberg has posted comments on this change.
Change subject: vds-tool: keep env on sysV service
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/26255
To unsubscribe, visit http://gerrit.ovirt.org/
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vds-tool: keep env on sysV service
..
vds-tool: keep env on sysV service
Update env when SYSTEMCTL_SKIP_REDIRECT is set instead of dropping
it.
Change
Dan Kenigsberg has submitted this change and it was merged.
Change subject: pep8: make pep8-1.5 happy with E265
..
pep8: make pep8-1.5 happy with E265
Change-Id: I91c49e1c0f36590b8052462b9e066c110eacd67b
Signed-off-by: Dan Keni
Dan Kenigsberg has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E241
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/26249
To unsubscribe, visit http://gerrit.ovirt.org
Dan Kenigsberg has submitted this change and it was merged.
Change subject: pep8: make pep8-1.5 happy with E241
..
pep8: make pep8-1.5 happy with E241
Change-Id: Ieaa71ec0adc2fdeff4f5530220d0f55c619bd726
Signed-off-by: Dan Keni
Dan Kenigsberg has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/26248
To unsubscribe, visit http://gerrit.ovirt.org
On Tue, Apr 01, 2014 at 10:03:44PM +0100, Jenkins ci oVirt Server wrote:
> Project: http://jenkins.ovirt.org/job/vdsm_unit_tests/
> Build: http://jenkins.ovirt.org/job/vdsm_unit_tests/2873/
> Build Number: 2873
> Build Status: Failure
> Triggered By: Started by an SCM change
>
>
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Initial commit for vdsm-tool testing infrastructure
..
Initial commit for vdsm-tool testing infrastructure
Running in make check, flexible in modifying
Yaniv Bronhaim has posted comments on this change.
Change subject: Initial commit for vdsm-tool testing infrastructure
..
Patch Set 21: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/25263
To unsubscribe, visit http:/
mooli tayer has posted comments on this change.
Change subject: vds-tool: keep env on sysV service
..
Patch Set 2: Verified+1
Verification(el 6.4):
vdsm-tool service-start vdsmd & vdsm-tool service-stop work well.
env dumped
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Do not allow empty network names (bridged nor bridgeless)
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/700
Antoni Segura Puimedon has posted comments on this change.
Change subject: Do not allow empty network names (bridged nor bridgeless)
..
Patch Set 2: Verified+1
Fixed it for networks as well.
Verification process:
- Tried to cr
Antoni Segura Puimedon has posted comments on this change.
Change subject: Do not allow empty bridge names
..
Patch Set 1: Verified-1
This is not enough because objectivize, having bridge='' and checking for
bridgeness with:
Douglas Schilling Landgraf has abandoned this change.
Change subject: autogen.sh: deprecate --system
..
Abandoned
--
To view, visit http://gerrit.ovirt.org/25804
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-M
Francesco Romani has posted comments on this change.
Change subject: caps: Collect numa information
..
Patch Set 8:
(3 comments)
It can be already good enough but I think we can make it even better.
http://gerrit.ovirt.org/#/
Francesco Romani has posted comments on this change.
Change subject: vnic QoS: Fix specParams to XML definition conversion
..
Patch Set 1: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26254
To unsubscribe, visit htt
Francesco Romani has posted comments on this change.
Change subject: virt: Move all guest agents related cleanup together
..
Patch Set 1: Code-Review+1
(1 comment)
http://gerrit.ovirt.org/#/c/26280/1/vdsm/virt/vm.py
File vdsm/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 9:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7002/ : SUCCESS
h
Dima Kuznetsov has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 9:
I decided to remove the memoize, yet still handle error outside.
HostStatsThread however will not tr
oVirt Jenkins CI Server has posted comments on this change.
Change subject: [wip] json-rpc: AMQP test reactor
..
Patch Set 2: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7001/
Dima Kuznetsov has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E241
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26249
To unsubscribe, visit http://gerrit.ovirt.org
Dima Kuznetsov has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26248
To unsubscribe, visit http://gerrit.ovirt.org
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 20:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7000/
mooli tayer has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E241
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26249
To unsubscribe, visit http://gerrit.ovirt.org/se
mooli tayer has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26248
To unsubscribe, visit http://gerrit.ovirt.org/se
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 19:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6999/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 18:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6998/
mooli tayer has posted comments on this change.
Change subject: vds-tool: keep env on sysV service
..
Patch Set 2:
Not sure how to verify this one...
--
To view, visit http://gerrit.ovirt.org/26255
To unsubscribe, visit http:
mooli tayer has posted comments on this change.
Change subject: Initial commit for vdsm-tool testing infrastructure
..
Patch Set 21: Verified+1
--
To view, visit http://gerrit.ovirt.org/25263
To unsubscribe, visit http://gerri
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 17:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6997/
Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: Remove all guest sockets on cleanup
..
virt: Remove all guest sockets on cleanup
Previously vdsm was cleaning up the guest socket files, but when
Dan Kenigsberg has posted comments on this change.
Change subject: virt: Remove all guest sockets on cleanup
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/26288
To unsubscribe, visit http://gerrit.ovi
oVirt Jenkins CI Server has posted comments on this change.
Change subject: [wip] json-rpc: AMQP test reactor
..
Patch Set 1: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6996/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsClient: file based mechanism to provide password
..
Patch Set 11:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6995/ :
Michal Skrivanek has posted comments on this change.
Change subject: virt: Remove all guest sockets on cleanup
..
Patch Set 1: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26288
To unsubscribe, visit http://gerrit.o
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
(1 comment)
http://gerrit.ovirt.org/#/c/26113/6/vdsm/vdsm
File vdsm/vdsm:
Line 84: try:
Line 85:
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
(2 comments)
Thanks, will clean up.
http://gerrit.ovirt.org/#/c/26113/6/vdsm/vdsm
File vdsm/vdsm:
Line
oVirt Jenkins CI Server has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6994/ : SUCCESS
http://j
oVirt Jenkins CI Server has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E241
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6993/ : SUCCESS
http://j
Saggi Mizrahi has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6: Code-Review-1
(2 comments)
http://gerrit.ovirt.org/#/c/26113/6/vdsm/vdsm
File vdsm/vdsm:
Line 88: ',
Dan Kenigsberg has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E241
..
Patch Set 2: Verified+1
No syntactical changes
--
To view, visit http://gerrit.ovirt.org/26249
To unsubscribe, visit ht
Dan Kenigsberg has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 2: Verified+1
had to rebase. No syntactical changes.
--
To view, visit http://gerrit.ovirt.org/26248
To unsub
Dan Kenigsberg has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 1: Code-Review+2
Ohad, this is only partial remedy. If you're using pep8-1.5, you'd have to set
PEP8="pep8 -
Dima Kuznetsov has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 8:
Verified with included tests, all pass with the fix in utils.py, roughly half
fail without.
--
To view, vi
mooli tayer has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 1: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26248
To unsubscribe, visit http://gerrit.ovirt.org/se
Nir Soffer has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 8:
(1 comment)
http://gerrit.ovirt.org/#/c/25877/8/vdsm/sampling.py
File vdsm/sampling.py:
Line 466:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: util: fix execCmd to accept tuples
..
util: fix execCmd to accept tuples
There was an issue occasionally when flags were given along with a
command in
Dan Kenigsberg has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 7:
Passing http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/7891/ is good
enough in this case.
--
To view,
Ohad Basan has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E265
..
Patch Set 1:
can this be merged please?
it is causing problems on ci
--
To view, visit http://gerrit.ovirt.org/26248
To uns
Ohad Basan has posted comments on this change.
Change subject: pep8: make pep8-1.5 happy with E241
..
Patch Set 1:
can this be merged please?
it is causing problems on ci
--
To view, visit http://gerrit.ovirt.org/26249
To uns
Dan Kenigsberg has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 7: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/26070
To unsubscribe, visit http://gerrit.ovirt.org/
Dima Kuznetsov has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 7: Verified+1
--
To view, visit http://gerrit.ovirt.org/26070
To unsubscribe, visit http://gerrit.ovirt.org/set
Nir Soffer has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 8:
(2 comments)
The tests are very nice now.
Error message must be improved, docstring can be improved.
h
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 16:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6992/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 8:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6991/ : SUCCESS
h
Dima Kuznetsov has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 8:
* Broke apart the one line if.
* Added couple of tests.
* Made fixtures bit shorter.
* Created a getF
Nir Soffer has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 7: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26070
To unsubscribe, visit http://gerrit.ovirt.org/sett
Nir Soffer has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 7:
Dima, can you comment about new version you upload? This will save time to your
reviewers.
--
To view, visit h
oVirt Jenkins CI Server has posted comments on this change.
Change subject: network: Introduce the network package
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_install_rpm_sanity_gerrit/443/ :
SU
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 15:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6990/
Dan Kenigsberg has posted comments on this change.
Change subject: network: Introduce the network package
..
Patch Set 5: Verified+1 Code-Review+2
Copying V+1 due to the minuscule change.
--
To view, visit http://gerrit.ovirt
Dan Kenigsberg has submitted this change and it was merged.
Change subject: network: Introduce the network package
..
network: Introduce the network package
This is the first patch of the topic branch that wants to end up with
Dan Kenigsberg has submitted this change and it was merged.
Change subject: tests: make virtTests run on RHEL/CentOS
..
tests: make virtTests run on RHEL/CentOS
This patch includes fixes to let the virt functional test run
on R
oVirt Jenkins CI Server has posted comments on this change.
Change subject: util: fix execCmd to accept tuples
..
Patch Set 7:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6989/ : SUCCESS
http://je
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo: Retrieve bonding options differing from defaults
..
Patch Set 14:
Build Failed
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6988/
Francesco Romani has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 3:
Will do for future updates.
--
To view, visit http://gerrit.ovirt.org/26260
To unsubscribe, visit
Nir Soffer has posted comments on this change.
Change subject: core: Add boot time to the getVdsStats API
..
Patch Set 7: Code-Review-1
(7 comments)
Much better, but few issues should be improved.
http://gerrit.ovirt.org/#/c/
Francesco Romani has posted comments on this change.
Change subject: tests: make virtTests run on RHEL/CentOS
..
Patch Set 2: Verified+1
verification:
* runned the tests on RHEL6 an F19, seems to work as expected
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: move threads in a separate module
..
Patch Set 10:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6986/ : S
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: always run migration monitor
..
Patch Set 11:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6987/ : SUCCES
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: decouple monitoring from thread
..
Patch Set 3:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6984/ : SUCC
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 3:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6985/ : SUCCESS
ht
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: add monitor thread control loop
..
Patch Set 12:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6983/ : SUC
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: merge monitor and downtime thread
..
Patch Set 13:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6982/ : S
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: always run migration monitor
..
Patch Set 10:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6980/ : SUCCES
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: use contextmanager for monitor
..
Patch Set 14:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6981/ : SUCC
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: move threads in a separate module
..
Patch Set 9:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6979/ : SU
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6978/ : SUCCESS
ht
Federico Simoncelli has posted comments on this change.
Change subject: autogen.sh: deprecate --system
..
Patch Set 5:
Same thing, src rpm is just a tarball with a spec.
--
To view, visit http://gerrit.ovirt.org/25804
To unsu
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: decouple monitoring from thread
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6977/ : SUCC
Nir Soffer has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 3:
Francesco, please add a comment for each new patch set, to save time for your
reviewers. We don't want to
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: add monitor thread control loop
..
Patch Set 11:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6976/ : SUC
Francesco Romani has posted comments on this change.
Change subject: virt: migration: always run migration monitor
..
Patch Set 11: Verified+1
--
To view, visit http://gerrit.ovirt.org/25975
To unsubscribe, visit http://gerrit
Francesco Romani has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 3: Verified+1
--
To view, visit http://gerrit.ovirt.org/26260
To unsubscribe, visit http://gerrit.ovir
Vinzenz Feenstra has posted comments on this change.
Change subject: virt: migration: decouple monitoring from thread
..
Patch Set 3: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26279
To unsubscribe, visit http://g
Francesco Romani has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 1: Verified+1
--
To view, visit http://gerrit.ovirt.org/26260
To unsubscribe, visit http://gerrit.ovir
Francesco Romani has posted comments on this change.
Change subject: virt: migration: always run migration monitor
..
Patch Set 9: Verified+1
--
To view, visit http://gerrit.ovirt.org/25975
To unsubscribe, visit http://gerrit.
Francesco Romani has posted comments on this change.
Change subject: virt: migration: always run migration monitor
..
Patch Set 9:
Verification (with 26260 and 25970):
* runned patched VDSM (and spotted the clientIF issue)
* mi
Francesco Romani has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 1:
Verification (with 25970 and 25975):
* runned patched VDSM
* migrated a VM back and forth a few time
Vinzenz Feenstra has posted comments on this change.
Change subject: virt: migration: rename migration threads
..
Patch Set 3: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26260
To unsubscribe, visit http://gerrit.o
Francesco Romani has posted comments on this change.
Change subject: virt: migration: move threads in a separate module
..
Patch Set 10: Verified+1
Unfortunately I had to burn a few +1s :(
Changes:
* updated clientIF (forgotte
Vinzenz Feenstra has posted comments on this change.
Change subject: virt: migration: move threads in a separate module
..
Patch Set 10: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/25970
To unsubscribe, visit http:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: merge monitor and downtime thread
..
Patch Set 12:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6975/ : S
1 - 100 of 129 matches
Mail list logo