Oved Ourfali has posted comments on this change.
Change subject: jsonrpc: Wrong key name for getDevicesVisibility
..
Patch Set 1: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/35306
To unsubscribe, visit http://gerri
Darshan N has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 7:
(6 comments)
http://gerrit.ovirt.org/#/c/17766/7//COMMIT_MSG
Commit Message:
Line 3: AuthorDate
Darshan N has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 8: Verified+1
--
To view, visit http://gerrit.ovirt.org/17766
To unsubscribe, visit http://gerrit.o
Petr Horáček has abandoned this change.
Change subject: netlink: event monitor functional tests
..
Abandoned
--
To view, visit http://gerrit.ovirt.org/35288
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-Messag
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 8: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_un
Antoni Segura Puimedon has uploaded a new change for review.
Change subject: network: Make vdsmd delNetwork use supervdsmd setupNetworks
..
network: Make vdsmd delNetwork use supervdsmd setupNetworks
delNetwork is part of a very
Antoni Segura Puimedon has uploaded a new change for review.
Change subject: Make _addNetwork and _delNetwork private
..
Make _addNetwork and _delNetwork private
Change-Id: Ie043a26c795a8c8d71a6142d4df45f6584120338
Signed-off-by
Antoni Segura Puimedon has posted comments on this change.
Change subject: Make _addNetwork and _delNetwork private
..
Patch Set 1: Code-Review-1
Unfinished
--
To view, visit http://gerrit.ovirt.org/35311
To unsubscribe, visi
Sandro Bonazzola has uploaded a new change for review.
Change subject: sos: fix config import
..
sos: fix config import
config module is in not a platform-specific module so
it's in lib and not in lib64 tree.
Change-Id: Ic5bd8a
oVirt Jenkins CI Server has posted comments on this change.
Change subject: network: Make vdsmd delNetwork use supervdsmd setupNetworks
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_e
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Make _addNetwork and _delNetwork private
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12710/ : FAILURE
oVirt Jenkins CI Server has posted comments on this change.
Change subject: sos: remove vdsm pki private keys collection
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12711/ : FAIL
oVirt Jenkins CI Server has posted comments on this change.
Change subject: sos: fix config import
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12712/ : FAILURE
http://jenkins.ov
Bala.FA has posted comments on this change.
Change subject: Provide device details using python-blivet module
..
Patch Set 5: Code-Review-1
(4 comments)
http://gerrit.ovirt.org/#/c/35028/5//COMMIT_MSG
Commit Message:
Line 3:
Bala.FA has posted comments on this change.
Change subject: Provide device details using python-blivet module
..
Patch Set 5:
(1 comment)
http://gerrit.ovirt.org/#/c/35028/5/vdsm/gluster/storagedev.py
File vdsm/gluster/storage
Sandro Bonazzola has posted comments on this change.
Change subject: sos: fix config import
..
Patch Set 1: Verified+1
This avoid to crash while importing the vdsm plugin but it doesn't work with
sos-3 anyway because of wrong
Hello Ondřej Svoboda, Antoni Segura Puimedon,
I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/35317
to review the following change.
Change subject: netinfo.getIpInfo: handle slashless address
..
net
Dan Kenigsberg has posted comments on this change.
Change subject: netinfo.getIpInfo: handle slashless address
..
Patch Set 1: Verified+1
Verified by Ciprian Cosma
https://bugzilla.redhat.com/show_bug.cgi?id=1165210#c2
--
To
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netinfo.getIpInfo: handle slashless address
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_network_functional_tests_gerrit/29/ :
Yaniv Bronhaim has posted comments on this change.
Change subject: jsonrpc: switch to the simplejson module
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/35290
To unsubscribe, visit http://gerrit.ovir
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: virt: migration: simplify the remote port handling
..
virt: migration: simplify the remote port handling
When the destination port for migrations is no
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: jsonrpc: switch to the simplejson module
..
jsonrpc: switch to the simplejson module
During testing, we discovered that on RHEL 6.x
and Centos 6.x we u
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Send to cannonizeHostPort port parameter as string instead of
integer.
..
Send to cannonizeHostPort port parameter as string instead of integer.
Chang
oVirt Jenkins CI Server has posted comments on this change.
Change subject: virt: migration: simplify the remote port handling
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: network: unified configureLibvirtNetwork parameters
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12713/
Shahar Havivi has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 22:
(3 comments)
http://gerrit.ovirt.org/#/c/33309/22/vdsm/rpc/Bridge.py
File vdsm/rpc/Bridge.py:
Line 324:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Send to cannonizeHostPort port parameter as string instead of
integer.
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpm
Ido Barkan has posted comments on this change.
Change subject: network: Make vdsmd addNetwork use supervdsmd setupNetworks
..
Patch Set 1: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/35090/1/vdsm/API.py
File vdsm/API
oVirt Jenkins CI Server has posted comments on this change.
Change subject: jsonrpc: switch to the simplejson module
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/87/ :
FAI
oVirt Jenkins CI Server has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 23:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12715/ : FAILURE
htt
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Convert VM from external source to Data Domain
..
Patch Set 21:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12716/ : F
oVirt Jenkins CI Server has posted comments on this change.
Change subject: pyroute2: reuse created devices
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12714/ : FAILURE
http://j
Dan Kenigsberg has posted comments on this change.
Change subject: vm: return full copy of the conf data on status()
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34982
To unsubscribe, visit http://ge
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: vm: return full copy of the conf data on status()
..
vm: return full copy of the conf data on status()
It is unsafe for status() to return a shallow co
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Extract the DOM to Drive name, alias and path logic to reusable
method
..
Extract the DOM to Drive name, alias and path logic to reusable method
Bug-U
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: tests: make api check support falsey values
..
tests: make api check support falsey values
the current vmApiTests assume truthy values in the replies.
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Fix the API definition for cpu tune methods
..
Fix the API definition for cpu tune methods
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120246
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Improve the _validateIoTuneParams so the params are passed as
argument
..
Improve the _validateIoTuneParams so the params are passed as argument
This
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Refactor updateVmPolicy to use DOM manipulation
..
Refactor updateVmPolicy to use DOM manipulation
This allows updating the old values instead of just
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Collect current QoS settings for IO devices and report through
RunningVmStats
..
Collect current QoS settings for IO devices and report through Running
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: janitorial: move isVdsmImage into utils
..
janitorial: move isVdsmImage into utils
this patch moves vdsm/virt/vm.isVdsmImage into utils without code
ch
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Refactor XMLElement to virt.utils
..
Refactor XMLElement to virt.utils
This is needed to avoid cyclic dependencies when other files
need the xml helper
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: tests: fix pep8 errors
..
tests: fix pep8 errors
Commit: a2890d5d614e856a44102d61576a8591a8d2e52f
introduced pep8 and pyflakes errors in tests.
This pa
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Add IO tunables support to updateVmPolicy
..
Add IO tunables support to updateVmPolicy
This allows the engine to set the hard and soft limits for IO.
M
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Add setIoTune and getIoTunePolicy to the xml-rpc API
..
Add setIoTune and getIoTunePolicy to the xml-rpc API
Bug-Url: https://bugzilla.redhat.com/show_
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Add API.VM.setIoTune
..
Add API.VM.setIoTune
This allows MoM (and the engine if necessary) to set the iotune
limits for disk devices.
Bug-Url: https:/
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Add API.VM.getIoTunePolicy
..
Add API.VM.getIoTunePolicy
This method can be used by MoM to retrieve the current ioTune
boundaries from the domain xml m
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: vm: fix getVmPolicy return type.
..
vm: fix getVmPolicy return type.
in case of error, getVmPolicy will return a VDSM API return
value, which may confu
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Change method names in vmtune.py to the PEP8 style
..
Change method names in vmtune.py to the PEP8 style
Bug-Url: https://bugzilla.redhat.com/show_bug.
Darshan N has posted comments on this change.
Change subject: gluster: Added geo-replication status and status detail
..
Patch Set 12: Verified+1
--
To view, visit http://gerrit.ovirt.org/18414
To unsubscribe, visit http://ger
Darshan N has posted comments on this change.
Change subject: gluster: Added geo-replication status and status detail
..
Patch Set 11:
(6 comments)
http://gerrit.ovirt.org/#/c/18414/11//COMMIT_MSG
Commit Message:
Line 11: *gl
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: Added geo-replication status and status detail
..
Patch Set 12: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_mast
Shahar Havivi has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 23:
(1 comment)
http://gerrit.ovirt.org/#/c/33309/23/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.
Dan Kenigsberg has posted comments on this change.
Change subject: adding tx/rx statistic to getInterfacesStats
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/35061
To unsubscribe, visit http://gerrit.
Francesco Romani has uploaded a new change for review.
Change subject: vm: re-enable HPET clock for non-windows guests
..
vm: re-enable HPET clock for non-windows guests
In commit 6167a6c33f150be we adhered to the recommendatio
Douglas Schilling Landgraf has posted comments on this change.
Change subject: sos: remove vdsm pki private keys collection
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/35241
To unsubscribe, visit ht
oVirt Jenkins CI Server has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 24:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12718/ : FAILURE
htt
Ondřej Svoboda has posted comments on this change.
Change subject: netinfo.getIpInfo: handle slashless address
..
Patch Set 1: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/35317
To unsubscribe, visit http://gerrit.o
Roy Golan has uploaded a new change for review.
Change subject: mom.d: make CpuTuneEnabled True by default
..
mom.d: make CpuTuneEnabled True by default
we should ship vdsm with that policy enabled otherwise cpu tuning isn't
tak
Roy Golan has posted comments on this change.
Change subject: mom.d: make CpuTuneEnabled True by default
..
Patch Set 1: Verified+1
--
To view, visit http://gerrit.ovirt.org/35330
To unsubscribe, visit http://gerrit.ovirt.org/
Hello Dan Kenigsberg,
I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/35331
to review the following change.
Change subject: utils: faster albeit less general deepcopy
..
utils: faster albeit less ge
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Convert VM from external source to Data Domain
..
Patch Set 22:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12719/ : F
Petr Horáček has posted comments on this change.
Change subject: network: unified configureLibvirtNetwork parameters
..
Patch Set 2: Verified+1
Doesn't affect the results of ifcfg and iproute network functional tests.
Fixed py
Amador Pahim has posted comments on this change.
Change subject: Sysctl to allow iSCSI multipath with multiple NICs in the same
subnet
..
Patch Set 19: Verified+1
Setting rp_filter to loose mode only for devices distinct from
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: re-enable HPET clock for non-windows guests
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12720/ : F
Francesco Romani has posted comments on this change.
Change subject: utils: faster albeit less general deepcopy
..
Patch Set 2: Verified+1
verified on 3.5 branch on RHEL6.6 by doing the usual routine of booting a few
tenths of
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: volume snapshot create and delete verbs.
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12721/ :
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: Added geo-replication status and status detail
..
Patch Set 13:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/1
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12723/
Darshan N has posted comments on this change.
Change subject: gluster: Added geo-replication status and status detail
..
Patch Set 13: Verified+1
--
To view, visit http://gerrit.ovirt.org/18414
To unsubscribe, visit http://ger
Darshan N has posted comments on this change.
Change subject: gluster: volume snapshot create and delete verbs.
..
Patch Set 4: Verified+1
--
To view, visit http://gerrit.ovirt.org/34535
To unsubscribe, visit http://gerrit.ovi
Darshan N has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 9: Verified+1
--
To view, visit http://gerrit.ovirt.org/17766
To unsubscribe, visit http://gerrit.o
Petr Horáček has posted comments on this change.
Change subject: libnl: non-existing link exception
..
Patch Set 12: Verified+1
Fedora 20, libnl3-3.2.24
Compiled, installed, started OK.
Passed unit tests except test_source_ro
Bala.FA has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 7:
(1 comment)
http://gerrit.ovirt.org/#/c/17766/7/vdsm/gluster/exception.py
File vdsm/gluster/except
Bala.FA has posted comments on this change.
Change subject: gluster: Add geo-replication start and stop session
..
Patch Set 9: Code-Review-1
Please respond to comment in patch set#7
--
To view, visit http://gerrit.ovirt.org/
Bala.FA has posted comments on this change.
Change subject: gluster: volume snapshot create and delete verbs.
..
Patch Set 4: Code-Review-1
(4 comments)
http://gerrit.ovirt.org/#/c/34535/4/vdsm.spec.in
File vdsm.spec.in:
Line
oVirt Jenkins CI Server has posted comments on this change.
Change subject: mom.d: make CpuTuneEnabled True by default
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12724/ : SUCCES
Ondřej Svoboda has posted comments on this change.
Change subject: avoid reading ifcfg files in netinfo
..
Patch Set 2: Code-Review+1
Nice :-)
--
To view, visit http://gerrit.ovirt.org/35043
To unsubscribe, visit http://gerri
Francesco Romani has abandoned this change.
Change subject: vm: re-enable HPET clock for non-windows guests
..
Abandoned
wrong direction. Solution is not to go back to HPET, but to improve kvm-clock
in qemu. See bz entry for d
Ondřej Svoboda has posted comments on this change.
Change subject: make isVDSMInterface and it's helpers static
..
Patch Set 6: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/34139
To unsubscribe, visit http://gerrit.
Francesco Romani has uploaded a new change for review.
Change subject: client: add option to make quiet
..
client: add option to make quiet
When used for (quite brutal) benchmarking, the output
from vdsClient can add noise and c
Francesco Romani has posted comments on this change.
Change subject: profile: runtime support to enable/disable profile
..
Patch Set 5: Code-Review-1
it is unsafe to do this at runtime, because yappi handles internal python dat
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profile: add trivial benchmark runner
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12725/ : FAILURE
ht
Shahar Havivi has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 26:
getExternalVMs JSON, username and password are mandatory. (remove the optional
prefix)
--
To view, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: return full copy of the conf data on status()
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/8
oVirt Jenkins CI Server has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 25:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12729/ : FAILURE
htt
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profile: runtime support to enable/disable profile
..
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12727/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: client: add option to make quiet
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12728/ : FAILURE
http://
oVirt Jenkins CI Server has posted comments on this change.
Change subject: client: vm params from JSON
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12726/ : FAILURE
http://jenki
Francesco Romani has uploaded a new change for review.
Change subject: jsonrpc: revert switch to the simplejson module
..
jsonrpc: revert switch to the simplejson module
This reverts commit 58553451547fe70958c01585f58d50c8ed7549
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Convert VM from external source to Data Domain
..
Patch Set 23:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12730/ : F
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Extract the DOM to Drive name, alias and path logic to reusable
method
..
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpm
oVirt Jenkins CI Server has posted comments on this change.
Change subject: tests: make api check support falsey values
..
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/90/ :
Francesco Romani has posted comments on this change.
Change subject: jsonrpc: revert switch to the simplejson module
..
Patch Set 3:
rebased with no code changes.
--
To view, visit http://gerrit.ovirt.org/35347
To unsubscribe
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Fix the API definition for cpu tune methods
..
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/91/ :
Dan Kenigsberg has posted comments on this change.
Change subject: jsonrpc: revert switch to the simplejson module
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/35347
To unsubscribe, visit http://gerr
Francesco Romani has posted comments on this change.
Change subject: jsonrpc: revert switch to the simplejson module
..
Patch Set 3: Verified+1
Verification:
on a 3.5 branch with this patch applied, rebuilt, installed and resta
Dan Kenigsberg has submitted this change and it was merged.
Change subject: jsonrpc: revert switch to the simplejson module
..
jsonrpc: revert switch to the simplejson module
This reverts commit 58553451547fe70958c01585f58d50c8
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Improve the _validateIoTuneParams so the params are passed as
argument
..
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpm
Lior Vernia has posted comments on this change.
Change subject: host QoS: have service curve parameters be integers
..
Patch Set 2: Verified+1
Works fine when they're multiples of 8 :)
--
To view, visit http://gerrit.ovirt.or
1 - 100 of 176 matches
Mail list logo