Yaniv Bronhaim has posted comments on this change.
Change subject: spec: bump policycoreutils-python version to 2.0.83-19.47.
..
Patch Set 7:
(1 comment)
http://gerrit.ovirt.org/#/c/33771/7/vdsm.spec.in
File vdsm.spec.in:
Lin
Yaniv Bronhaim has posted comments on this change.
Change subject: tool: persist selinux booleans on ovirt node.
..
Patch Set 2:
Fabian, I don't understand your comment.. please elaborate. Mooli, we just need
any node installa
Amador Pahim has posted comments on this change.
Change subject: caps: Do not memoize CPU topology
..
Patch Set 3: Code-Review-1
(5 comments)
I don't like the idea of getting rid from libvirt. We changed to libvirt some
time
Nir Soffer has posted comments on this change.
Change subject: Support ioTune values >2^31 in getStats over xml-rpc
..
Patch Set 4: Code-Review+1
But this is a backport to 3.5, and the rest of the code use the same thing.
--
oVirt Jenkins CI Server has posted comments on this change.
Change subject: fix broken TestConfigNetwork.testAddNetworkValidation
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13439/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: tool: bugfix function called as attribute.
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/163/
Dan Kenigsberg has submitted this change and it was merged.
Change subject: fix broken TestConfigNetwork.testAddNetworkValidation
..
fix broken TestConfigNetwork.testAddNetworkValidation
this was broken after http://gerrit.ovei
Dan Kenigsberg has posted comments on this change.
Change subject: fix broken TestConfigNetwork.testAddNetworkValidation
..
Patch Set 2: Verified+1 Code-Review+2
Fixing commit message and shedding doubtedly unrelated change to
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm: add functionality to detach host devices
..
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/32389
To unsubscribe, visit http://gerri
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: return displayIp as a string in default case
..
vm: return displayIp as a string in default case
Previously, default value of displayInfo.displayIp
Dan Kenigsberg has posted comments on this change.
Change subject: vm: return displayIp as a string in default case
..
Patch Set 1: Code-Review+2
Let it be.
--
To view, visit http://gerrit.ovirt.org/34675
To unsubscribe, visi
Dan Kenigsberg has posted comments on this change.
Change subject: libvirtconnection: add function to create a new connection
..
Patch Set 10: Code-Review+2
(1 comment)
http://gerrit.ovirt.org/#/c/34293/10/tests/libvirtconnect
Dan Kenigsberg has submitted this change and it was merged.
Change subject: libvirtconnection: add function to create a new connection
..
libvirtconnection: add function to create a new connection
Added a new function open_conn
oVirt Jenkins CI Server has posted comments on this change.
Change subject: storageServer: remove dead code from getSessionInfo
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_mer
Dan Kenigsberg has posted comments on this change.
Change subject: vmdevices: vm.${NAME}_DEVICES -> vmdevices.${NAME}
..
Patch Set 2: Code-Review-1
deepcopy hacks sneaked in. Please rebase manually.
--
To view, visit http://g
Dan Kenigsberg has posted comments on this change.
Change subject: virt: move vm.GraphicsDevice -> vmdevices.Graphics
..
Patch Set 2: Code-Review-1
deepcopy hacks sneaked in, please rebase manually.
--
To view, visit http://g
Dan Kenigsberg has posted comments on this change.
Change subject: vmdevices: vm.${NAME}_DEVICES -> vmdevices.${NAME}
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34556
To unsubscribe, visit http://g
Dan Kenigsberg has posted comments on this change.
Change subject: guestagent: Simpler and safer disk mapping hash
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34625
To unsubscribe, visit http://gerr
Dan Kenigsberg has submitted this change and it was merged.
Change subject: guestagent: Simpler and safer disk mapping hash
..
guestagent: Simpler and safer disk mapping hash
Previously diskMapingHash was initialized to the mag
Dan Kenigsberg has submitted this change and it was merged.
Change subject: guestagent: Use new style class
..
guestagent: Use new style class
Using old style class is a trap for innocent developers. They do not
support propert
Dan Kenigsberg has posted comments on this change.
Change subject: guestagent: Use new style class
..
Patch Set 1: Code-Review+2
(1 comment)
http://gerrit.ovirt.org/#/c/34624/1//COMMIT_MSG
Commit Message:
Line 6:
Line 7: gue
Dan Kenigsberg has submitted this change and it was merged.
Change subject: domain_descriptor: More careful falsy check
..
domain_descriptor: More careful falsy check
Using "if self._devices" works with xml.minidom, but if we s
Dan Kenigsberg has posted comments on this change.
Change subject: domain_descriptor: More careful falsy check
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34623
To unsubscribe, visit http://gerrit.o
Dan Kenigsberg has submitted this change and it was merged.
Change subject: domain_descriptor: Eliminate magic hash init value
..
domain_descriptor: Eliminate magic hash init value
We used to initialize devices hash to 0 when t
Dan Kenigsberg has posted comments on this change.
Change subject: domain_descriptor: Eliminate magic hash init value
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34622
To unsubscribe, visit http://g
Dan Kenigsberg has submitted this change and it was merged.
Change subject: tool: bugfix function called as attribute.
..
tool: bugfix function called as attribute.
to fix bug, introduced in commit 8f87f21, instead of using
sel
Dan Kenigsberg has posted comments on this change.
Change subject: tool: bugfix function called as attribute.
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34487
To unsubscribe, visit http://gerrit.ov
Dan Kenigsberg has submitted this change and it was merged.
Change subject: storageServer: remove dead code from getSessionInfo
..
storageServer: remove dead code from getSessionInfo
storageServer -> getSessionInfo:
cleanup of
Dan Kenigsberg has abandoned this change.
Change subject: oop: vdsm.config is not used anymore
..
Abandoned
So sorry - I've missed this fix, and taken http://gerrit.ovirt.org/34896
already.
--
To view, visit http://gerrit.ov
Nir Soffer has posted comments on this change.
Change subject: External hypervisor VMs integration
..
Patch Set 18:
(2 comments)
http://gerrit.ovirt.org/#/c/33309/18/vdsm/v2v.py
File vdsm/v2v.py:
Line 121: i['dev'
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: return full copy of the conf data on status()
..
Patch Set 6:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merge
Dan Kenigsberg 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
Dan Kenigsberg has posted comments on this change.
Change subject: vm: return full copy of the conf data on status()
..
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34147
To unsubscribe, visit http://ge
oVirt Jenkins CI Server has posted comments on this change.
Change subject: tests: make pyflakes happy with outOfProcess tests
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merg
oVirt Jenkins CI Server has posted comments on this change.
Change subject: spec: remove redundant check.
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/159/ :
FAILURE
h
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: safe(r) Vm.conf update in creation thread
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/15
Fabian Deutsch has posted comments on this change.
Change subject: tool: persist selinux booleans on ovirt node.
..
Patch Set 2: Code-Review-1
I believe this will not work.
The selinux policy will be mounted in place way to la
Nir Soffer has posted comments on this change.
Change subject: Support ioTune values >2^31 in getStats over xml-rpc
..
Patch Set 4:
Iotune values are integers - you can use str()
--
To view, visit http://gerrit.ovirt.org/3023
Douglas Schilling Landgraf has posted comments on this change.
Change subject: tool: persist selinux booleans on ovirt node.
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/34518
To unsubscribe, visit h
Dan Kenigsberg has submitted this change and it was merged.
Change subject: tests: make pyflakes happy with outOfProcess tests
..
tests: make pyflakes happy with outOfProcess tests
This patch fixes
python -c 'import pyflakes;
Dan Kenigsberg has posted comments on this change.
Change subject: tests: make pyflakes happy with outOfProcess tests
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/34896
To unsubscribe, visit http://g
Dan Kenigsberg has submitted this change and it was merged.
Change subject: spec: remove redundant check.
..
spec: remove redundant check.
if not fedora 0%{?fedora} evaluates to 0.
Change-Id: I50e1706e7f8c9b07b6788fbbabf61b403
Hello Piotr Kliczewski, Nir Soffer, Dan Kenigsberg, Michal Skrivanek,
I'd like you to do a code review. Please visit
http://gerrit.ovirt.org/34938
to review the following change.
Change subject: vm: safe(r) Vm.conf update in creation thread
.
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: safe(r) Vm.conf update in creation thread
..
vm: safe(r) Vm.conf update in creation thread
A VM is created asynchronously for performance's sake.
I
Dan Kenigsberg has posted comments on this change.
Change subject: vm: safe(r) Vm.conf update in creation thread
..
Patch Set 3: Code-Review+2
Accepting this temporary patch to a much deeper problem
--
To view, visit http://g
Francesco Romani has posted comments on this change.
Change subject: vm: safe(r) Vm.conf update in creation thread
..
Patch Set 3:
(1 comment)
http://gerrit.ovirt.org/#/c/34813/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 139
Martin Polednik has posted comments on this change.
Change subject: vm: return full copy of the conf data on status()
..
Patch Set 5: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/34147
To unsubscribe, visit http://g
Martin Polednik has posted comments on this change.
Change subject: vm: safe(r) Vm.conf update in creation thread
..
Patch Set 3: Code-Review-1
(1 comment)
The patch is fine, only suggesting/starting discussion whether we need
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: replace old ipwrapper event monitor
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13438/ : FAILUR
Petr Horáček has uploaded a new change for review.
Change subject: netlink: replace old ipwrapper event monitor
..
netlink: replace old ipwrapper event monitor
Change-Id: Ie7a9765e59a8392fff9b3fd556133159aeb60b1c
Signed-off-by:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: ignore_enodev parameter in *_info
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13437/ : FAILURE
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: ignore_enodev parameter in *_info
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13436/ : FAILURE
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: event monitor
..
Patch Set 26:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13435/ : FAILURE
http://jenkins.ovi
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: ignore_enodev parameter in *_info
..
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13434/ : FAILURE
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: event monitor
..
Patch Set 25: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13433/ : FA
Petr Horáček has posted comments on this change.
Change subject: netlink: ignore_enodev parameter in *_info
..
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/34925/1/lib/vdsm/netlink/link.py
File lib/vdsm/netlink/link.py
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: ignore_enodev parameter in *_info
..
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13432/ : FAILURE
Petr Horáček has uploaded a new change for review.
Change subject: netlink: ignore_enodev parameter in *_info
..
netlink: ignore_enodev parameter in *_info
When we try to get information about the removed device without the
cach
Francesco Romani has posted comments on this change.
Change subject: sampling: extract SampleWindow class
..
Patch Set 6: Verified+1
Verified again (just in case)
by installing fedora 20 on a VM runned by a patched VDSM and obs
Petr Horáček has posted comments on this change.
Change subject: libnl: make link_cache optional for route_info
..
Patch Set 9: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/32950
To unsubscribe, visit http://gerrit.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: netlink: event monitor
..
Patch Set 24: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13430/ : FA
oVirt Jenkins CI Server has posted comments on this change.
Change subject: libnl: make link_cache optional for route_info
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13431/ : FAIL
oVirt Jenkins CI Server has posted comments on this change.
Change subject: libnl: make link_cache optional for route_info
..
Patch Set 8: Code-Review-1 Verified-1
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-te
oVirt Jenkins CI Server has posted comments on this change.
Change subject: gluster: geo replication status and status detail
..
Patch Set 10:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13429/ :
oVirt Jenkins CI Server has posted comments on this change.
Change subject: libnl: non-existing link exception
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13427/ : FAILURE
http://
Petr Horáček has posted comments on this change.
Change subject: libnl: make link cache optional for link_info and addr_info
..
Patch Set 7: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/32949/7/lib/vdsm/netlink/link.p
Francesco Romani has abandoned this change.
Change subject: sampling: rename generic window constant
..
Abandoned
no longer needed
--
To view, visit http://gerrit.ovirt.org/31607
To unsubscribe, visit http://gerrit.ovirt.org/
Petr Horáček has posted comments on this change.
Change subject: libnl: non-existing link exception
..
Patch Set 3: Code-Review-1
have to think more about it...
--
To view, visit http://gerrit.ovirt.org/34690
To unsubscribe,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: libnl: non-existing link exception
..
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/13426/ : FAILURE
http://
Francesco Romani has posted comments on this change.
Change subject: caps: Do not use lscpu on ppc64
..
Patch Set 3:
Thanks for the updating the commit message.
There is no way to discover which kernel versions provide the corr
Francesco Romani has posted comments on this change.
Change subject: caps: Do not memoize CPU topology
..
Patch Set 3: Code-Review+1
really nice work with tests.
--
To view, visit http://gerrit.ovirt.org/34189
To unsubscribe,
71 matches
Mail list logo