Change in vdsm[ovirt-4.0]: vdscli: Provide py3 compatible xmlrpclib (using six)

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: vdscli: Provide py3 compatible xmlrpclib (using six) .. vdscli: Provide py3 compatible xmlrpclib (using six) Change-Id:

Change in vdsm[ovirt-4.0]: net virt: Support connecting VM/s to an ovs bridge

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net virt: Support connecting VM/s to an ovs bridge .. net virt: Support connecting VM/s to an ovs bridge Change-Id:

Change in vdsm[ovirt-4.0]: net: move ip configuration to ip.address module

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: move ip configuration to ip.address module .. net: move ip configuration to ip.address module We need to share IP configuration functions with

Change in vdsm[ovirt-4.0]: netswitch: split setup actions on netswitch level

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: netswitch: split setup actions on netswitch level .. netswitch: split setup actions on netswitch level This patch relocates split of nets and bonds

Change in vdsm[ovirt-4.0]: py3: Require libselinux-python3

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: py3: Require libselinux-python3 .. py3: Require libselinux-python3 The selinux package/module is not required in the spec and CI automation

Change in vdsm[ovirt-4.0]: net: Setup ipv6autoconf with OVS switch

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: Setup ipv6autoconf with OVS switch .. net: Setup ipv6autoconf with OVS switch When handling an OVS switch setup, set ipv6autoconf for its

Change in vdsm[ovirt-4.0]: ovs: don't count bond slave duplicates

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: don't count bond slave duplicates .. ovs: don't count bond slave duplicates We should check that set(slaves) contains 2 or more slaves to not

Change in vdsm[ovirt-4.0]: ovs: edit bond mode

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: edit bond mode .. ovs: edit bond mode With this patch we allow setup of OVS bond with one of 'fallback' modes 1 and 4 and miimon. In another

Change in vdsm[ovirt-4.0]: ovs: add/remove OVS network based on a bond

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: add/remove OVS network based on a bond .. ovs: add/remove OVS network based on a bond Bond modes will be handled in a following patch.

Change in vdsm[ovirt-4.0]: ovs: use transaction class for devices setup

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: use transaction class for devices setup .. ovs: use transaction class for devices setup With this patch we use transaction class instead of

Change in vdsm[ovirt-4.0]: ovs: fix bond removal validation

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: fix bond removal validation .. ovs: fix bond removal validation We should allow removal of a bond which network-user will use nic instead.

Change in vdsm[ovirt-4.0]: ovs: validate that to-be-removed bond is not used

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: validate that to-be-removed bond is not used .. ovs: validate that to-be-removed bond is not used Change-Id:

Change in vdsm[ovirt-4.0]: ovs: add/remove OVS network based on a VLAN

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: add/remove OVS network based on a VLAN .. ovs: add/remove OVS network based on a VLAN Change-Id: I98be71c300f2413ea8a8f6c71ba7daac0cd16d5d

Change in vdsm[ovirt-4.0]: ovs: add/remove OVS network that is based on a nic

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: add/remove OVS network that is based on a nic .. ovs: add/remove OVS network that is based on a nic Change-Id:

Change in vdsm[ovirt-4.0]: ovs: don't check if changes are needed in net editation

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: don't check if changes are needed in net editation .. ovs: don't check if changes are needed in net editation We cannot compare query

Change in vdsm[ovirt-4.0]: ovs: use unfaked ovs_netinfo under ovs package

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: use unfaked ovs_netinfo under ovs package .. ovs: use unfaked ovs_netinfo under ovs package Change-Id:

Change in vdsm[ovirt-4.0]: ovs: add bridges_by_sb property to OvsInfo

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: add bridges_by_sb property to OvsInfo .. ovs: add bridges_by_sb property to OvsInfo This will be used in a following ovs setupNetworks patch.

Change in vdsm[ovirt-4.0]: ovs: bridge is optional for del-port

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: ovs: bridge is optional for del-port .. ovs: bridge is optional for del-port ovs-vsctl does not require bridge name for port removal. We will use

Change in vdsm[ovirt-4.0]: net: use netinfo to split OVS nets and bonds

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: use netinfo to split OVS nets and bonds .. net: use netinfo to split OVS nets and bonds Our decision of what should be done with a query (add,

Change in vdsm[ovirt-4.0]: net: use netinfo to split OVS/legacy nets and bonds

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: use netinfo to split OVS/legacy nets and bonds .. net: use netinfo to split OVS/legacy nets and bonds Netinfo now contains 'switch' attribute

Change in vdsm[ovirt-4.0]: net: use netinfo for OVS validation

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: use netinfo for OVS validation .. net: use netinfo for OVS validation Use real running netinfo configuration for OVS validation instead of

Change in vdsm[ovirt-4.0]: net: ovs validator should accept multiple untagged nets

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: ovs validator should accept multiple untagged nets .. net: ovs validator should accept multiple untagged nets Change-Id:

Change in vdsm[ovirt-4.0]: net: fix validation bug: ovs_bonds should be used

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: fix validation bug: ovs_bonds should be used .. net: fix validation bug: ovs_bonds should be used Change-Id:

Change in vdsm[ovirt-4.0]: net: common and legacy CachingNetInfo

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: common and legacy CachingNetInfo .. net: common and legacy CachingNetInfo CachingNetInfo is not suitable for mixed OVS and Legacy networks.

Change in vdsm[ovirt-4.0]: net: common and legacy CachingNetInfo

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: net: common and legacy CachingNetInfo .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/60866 To unsubscribe, visit

Change in vdsm[ovirt-4.0]: net: report OVS netinfo in caps

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: net: report OVS netinfo in caps .. net: report OVS netinfo in caps Even non-root caps (legacy networks+nics) are now obtained from supervdsm. The

Change in vdsm[ovirt-4.0]: net: report OVS netinfo in caps

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: net: report OVS netinfo in caps .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/60865 To unsubscribe, visit

Change in vdsm[ovirt-4.0]: netinfo: report 'vlanid' only if vlan is set

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: netinfo: report 'vlanid' only if vlan is set .. netinfo: report 'vlanid' only if vlan is set Change-Id: I6a0c30177f238154b34513f83b7914d1783fd422

Change in vdsm[master]: sampling: refactor tests

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: sampling: refactor tests .. Patch Set 1: Code-Review-1 (2 comments) https://gerrit.ovirt.org/#/c/61309/1//COMMIT_MSG Commit Message: PS1, Line 7: sampling:

Change in vdsm[ovirt-4.0]: virt net: Change graphics type from network to address

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: virt net: Change graphics type from network to address .. virt net: Change graphics type from network to address This patch changes the graphics

Change in vdsm[master]: vm: periodic: fix stats age reporting

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: periodic: fix stats age reporting .. Patch Set 1: Verified+1 verified 1. using the tests 2. as described in BZ: running one VM, SIGSTOP'ing QEMU, waited

Change in vdsm[master]: sampling: refactor tests

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: sampling: refactor tests .. Patch Set 1: Verified+1 (1 comment) https://gerrit.ovirt.org/#/c/61309/1/tests/samplingTests.py File tests/samplingTests.py:

Change in vdsm[master]: sampling: micro-optimization

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: sampling: micro-optimization .. Patch Set 1: Verified-1 not yet ready. -- To view, visit https://gerrit.ovirt.org/61311 To unsubscribe, visit

Change in vdsm[master]: sampling: refactor tests

2016-07-25 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: sampling: refactor tests .. sampling: refactor tests Refactor the tests to make them clearer and to make room for further improvements. Change-Id:

Change in vdsm[master]: vm: periodic: fix stats age reporting

2016-07-25 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: periodic: fix stats age reporting .. vm: periodic: fix stats age reporting The Vm class reports the monitor responsiveness in two ways: 1. if a libvirt

Change in vdsm[master]: sampling: micro-optimization

2016-07-25 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: sampling: micro-optimization .. sampling: micro-optimization make timestamp update faster. Change-Id: Id4361bf4ccae67be3a9e42a0d3368e1e1c246393 TODO:

Change in vdsm[master]: v2v: Do not give Xen a special treatment.

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: v2v: Do not give Xen a special treatment. .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/57430/1/lib/vdsm/v2v.py File lib/vdsm/v2v.py: Line 995:

Change in vdsm[master]: logging: Improve logging when receiving signals

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: logging: Improve logging when receiving signals .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/61289 To unsubscribe, visit

Change in vdsm[master]: logging: Improve logging when receiving signals

2016-07-25 Thread fromani
Francesco Romani has posted comments on this change. Change subject: logging: Improve logging when receiving signals .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/61289 To unsubscribe, visit

Change in vdsm[master]: refmt: Use more standard main() function

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: refmt: Use more standard main() function .. refmt: Use more standard main() function There is no need to make main() private. Change-Id:

Change in vdsm[master]: remft: Add online help

2016-07-25 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: remft: Add online help .. remft: Add online help Example: $ contrib/refmt -h usage: refmt [-h] [file [file ...]] Reformat python

Change in vdsm[master]: vmxml: move namespace declaration

2016-07-22 Thread fromani
Francesco Romani has abandoned this change. Change subject: vmxml: move namespace declaration .. Abandoned made obsolete by updated https://gerrit.ovirt.org/#/c/60481/15 -- To view, visit https://gerrit.ovirt.org/57381 To

Change in vdsm[master]: vm: reorder operations in _domDependentInit

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: reorder operations in _domDependentInit .. Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/59817/5/vdsm/virt/vm.py File vdsm/virt/vm.py: PS5,

Change in vdsm[master]: vm: split guest agent init from domDependentInit

2016-07-22 Thread fromani
Francesco Romani has abandoned this change. Change subject: vm: split guest agent init from domDependentInit .. Abandoned -- To view, visit https://gerrit.ovirt.org/59816 To unsubscribe, visit

Change in vdsm[master]: vm: split device init from domDependentInit

2016-07-22 Thread fromani
Francesco Romani has abandoned this change. Change subject: vm: split device init from domDependentInit .. Abandoned -- To view, visit https://gerrit.ovirt.org/59814 To unsubscribe, visit https://gerrit.ovirt.org/settings

Change in vdsm[master]: vm: reorder operations in _domDependentInit

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: reorder operations in _domDependentInit .. Patch Set 18: (1 comment) https://gerrit.ovirt.org/#/c/59817/18/vdsm/virt/vm.py File vdsm/virt/vm.py: Line

Change in vdsm[master]: vm: split guest agent init from domDependentInit

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: split guest agent init from domDependentInit .. Patch Set 18: Code-Review-2 Let's do like this: 1. improve Vm.domDependentInit() separately:

Change in vdsm[master]: vm: split device init from domDependentInit

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: split device init from domDependentInit .. Patch Set 18: Code-Review-2 the plan changed as follows: 1. improve Vm.domDependentInit in a separate series

Change in vdsm[master]: vm: split device init from domDependentInit

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: split device init from domDependentInit .. Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/59814/5/vdsm/virt/vm.py File vdsm/virt/vm.py: PS5,

Change in vdsm[master]: vm: rename Vm._guestEvent

2016-07-22 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: rename Vm._guestEvent .. vm: rename Vm._guestEvent The _guestEvent field of Vm class is misleading. It does not track a "guest event", but rather sets a

Change in vdsm[master]: vm: legacy rng devices fix not needed anymore

2016-07-22 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: legacy rng devices fix not needed anymore .. vm: legacy rng devices fix not needed anymore We introduced this fix to deal with upgrades from Vdsm

Change in vdsm[master]: vm: move device initialization together

2016-07-22 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: move device initialization together .. vm: move device initialization together It was scatthered around in Vm._domDependentInit for no good reason No

Change in vdsm[master]: docs: add README about logging configuration

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: docs: add README about logging configuration .. Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/60533/2/README.logging File README.logging: PS2, Line

Change in vdsm[master]: docs: add README about logging configuration

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: docs: add README about logging configuration .. Patch Set 3: Verified+1 copied score. -- To view, visit https://gerrit.ovirt.org/60533 To unsubscribe,

Change in vdsm[master]: docs: add README about logging configuration

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: docs: add README about logging configuration .. Patch Set 2: (2 comments) https://gerrit.ovirt.org/#/c/60533/2/README.logging File README.logging: Line 29:

Change in vdsm[ovirt-4.0]: build: Remove python 3 test run

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: build: Remove python 3 test run .. Patch Set 1: We can make an exception to the flow for this patch. -- To view, visit https://gerrit.ovirt.org/61189 To

Change in vdsm[ovirt-4.0]: virt net: Change graphics type from network to address

2016-07-22 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: virt net: Change graphics type from network to address .. virt net: Change graphics type from network to address This patch changes the graphics

Change in vdsm[master]: remft: Add online help

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: remft: Add online help .. Patch Set 2: need ACK from Dan before merge into master. Nir, feel free to go forward if you wish. -- To view, visit

Change in vdsm[master]: refmt: Use more standard main() function

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: refmt: Use more standard main() function .. Patch Set 2: need ACK from Dan before merge into master. Nir, feel free to go forward if you wish. -- To view,

Change in vdsm[master]: hostdev: add back setup log message

2016-07-22 Thread fromani
Francesco Romani has posted comments on this change. Change subject: hostdev: add back setup log message .. Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/61181 To unsubscribe, visit

Change in vdsm[master]: hostdev: add back setup log message

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: hostdev: add back setup log message .. Patch Set 2: Code-Review+1 (1 comment) one question inside

Change in vdsm[ovirt-3.6]: json: use forzenset for filtered methods

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: json: use forzenset for filtered methods .. Patch Set 3: Code-Review-2 we are using this scapegoat patch to verify if a previously reported failure is real

Change in vdsm[ovirt-3.6]: json: use forzenset for filtered methods

2016-07-21 Thread fromani
Francesco Romani has restored this change. Change subject: json: use forzenset for filtered methods .. Restored -- To view, visit https://gerrit.ovirt.org/58796 To unsubscribe, visit https://gerrit.ovirt.org/settings

Change in vdsm[master]: docs: add README about logging configuration

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: docs: add README about logging configuration .. Patch Set 2: Code-Review-1 (3 comments) https://gerrit.ovirt.org/#/c/60533/2/README.logging File

Change in vdsm[master]: utils: add utlity to generate positive cpuset notation

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: utils: add utlity to generate positive cpuset notation .. Patch Set 4: (1 comment) https://gerrit.ovirt.org/#/c/60639/4/lib/vdsm/utils.py File

Change in vdsm[master]: utils: add utlity to generate positive cpuset notation

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: utils: add utlity to generate positive cpuset notation .. Patch Set 4: Code-Review-1 (1 comment) -1 for visibility

Change in vdsm[master]: docs: add README about logging configuration

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: docs: add README about logging configuration .. Patch Set 2: Verified+1 we can't really break anything here. "make rpm" still works as expected. -- To

Change in vdsm[master]: docs: add README about logging configuration

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: docs: add README about logging configuration .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/60533/1/README.logging File README.logging: Line 36:

Change in vdsm[master]: hostdev: prefer numa nodes close to device MMIO

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: hostdev: prefer numa nodes close to device MMIO .. Patch Set 4: (1 comment) https://gerrit.ovirt.org/#/c/60640/4/vdsm/virt/vmxml.py File vdsm/virt/vmxml.py:

Change in vdsm[ovirt-4.0]: automation: Use ovirt-4.0 repositories

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: automation: Use ovirt-4.0 repositories .. Patch Set 1: no changes to code whatsoever -> we can make one exception to the flow. -- To view, visit

Change in vdsm[ovirt-4.0]: automation: Use ovirt-4.0 repositories

2016-07-21 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: automation: Use ovirt-4.0 repositories .. automation: Use ovirt-4.0 repositories The master repositories do not provide the fc23 packages any more,

Change in vdsm[ovirt-4.0]: automation: Use ovirt-4.0 repositories

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: automation: Use ovirt-4.0 repositories .. Patch Set 1: Code-Review+2 thanks, I missed this and I was wondering what was broken :\ -- To view, visit

Change in vdsm[master]: WIP: vm: skip domDependentInit for containers

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: WIP: vm: skip domDependentInit for containers .. Patch Set 24: (1 comment) https://gerrit.ovirt.org/#/c/54179/24/vdsm/virt/vm.py File vdsm/virt/vm.py:

Change in vdsm[master]: utils: add utlity to generate positive cpuset notation

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: utils: add utlity to generate positive cpuset notation .. Patch Set 3: If you used a recipe, a link/reference/whatever to that recipe is an awesome first

Change in vdsm[master]: vm: reorder operations in _domDependentInit

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: reorder operations in _domDependentInit .. Patch Set 16: rationale for this patch is to pave the road for https://gerrit.ovirt.org/#/c/53820 AND make

Change in vdsm[master]: vm: split device init from domDependentInit

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: split device init from domDependentInit .. Patch Set 16: rationale for this patch is to pave the road for https://gerrit.ovirt.org/#/c/53820 AND make

Change in vdsm[master]: vm: split guest agent init from domDependentInit

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: split guest agent init from domDependentInit .. Patch Set 16: rationale for this patch is to pave the road for https://gerrit.ovirt.org/#/c/53820 AND

Change in vdsm[master]: vdsm: virt: add optional container support

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vdsm: virt: add optional container support .. Patch Set 39: (1 comment) https://gerrit.ovirt.org/#/c/53820/39/vdsm/virt/vm.py File vdsm/virt/vm.py: Line

Change in vdsm[master]: hostdev: prefer numa nodes close to device MMIO

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: hostdev: prefer numa nodes close to device MMIO .. Patch Set 3: Code-Review-1 (1 comment) only one minor style comment

Change in vdsm[master]: vmxml: add appendNumaTune return value

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmxml: add appendNumaTune return value .. Patch Set 1: Code-Review+2 why not? -- To view, visit https://gerrit.ovirt.org/61106 To unsubscribe, visit

Change in vdsm[master]: virt: Use DOM helpers in Vm._findGraphicsDeviceXMLByType etc.

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Use DOM helpers in Vm._findGraphicsDeviceXMLByType etc. .. Patch Set 7: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/56107 To unsubscribe,

Change in vdsm[master]: virt: Minor formatting fix in vmdevices/storage.py

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Minor formatting fix in vmdevices/storage.py .. Patch Set 1: Code-Review-1 (1 comment) -1 for visibility, no really strong feelings here

Change in vdsm[master]: virt: Use vmxml DOM helpers in storage devices

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Use vmxml DOM helpers in storage devices .. Patch Set 7: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/56102 To unsubscribe, visit

Change in vdsm[master]: osutils: Move NoIntrCall to osutils.uninterruptible

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: osutils: Move NoIntrCall to osutils.uninterruptible .. Patch Set 2: Code-Review+1 much nicer this way -- To view, visit https://gerrit.ovirt.org/61153 To

Change in vdsm[master]: osutils: Start the osutils module

2016-07-21 Thread fromani
Francesco Romani has posted comments on this change. Change subject: osutils: Start the osutils module .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/61142 To unsubscribe, visit

Change in vdsm[master]: utils: add utlity to generate positive cpuset notation

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: utils: add utlity to generate positive cpuset notation .. Patch Set 3: Code-Review-1 (1 comment) -1 for visibility. The code is clever, but maybe too packed

Change in vdsm[master]: hostdev: report device MMIO numa node

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: hostdev: report device MMIO numa node .. Patch Set 3: Code-Review+2 could also be useful for Engine's scheduler, so looks like a good change on its own. --

Change in vdsm[master]: hostdev: expose numa_node in host device

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: hostdev: expose numa_node in host device .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/60638 To unsubscribe, visit

Change in vdsm[ovirt-4.0]: virt net: Change graphics type from network to address

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt net: Change graphics type from network to address .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/61120 To unsubscribe, visit

Change in vdsm[master]: vmdevices: New method Base._update_device_attributes

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vmdevices: New method Base._update_device_attributes .. Patch Set 2: Code-Review-1 (1 comment) this method is a bit messy, but overall I think that

Change in vdsm[master]: virt: Make DomainDescriptor use XML helpers

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Make DomainDescriptor use XML helpers .. Patch Set 7: (2 comments) https://gerrit.ovirt.org/#/c/55769/7/tests/vmXmlTests.py File tests/vmXmlTests.py:

Change in vdsm[master]: virt: Use DOM helpers in Vm.setLinkAndNetwork

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Use DOM helpers in Vm.setLinkAndNetwork .. Patch Set 6: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/58190 To unsubscribe, visit

Change in vdsm[master]: virt: Return first child from vmxml.xml_to_dom

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Return first child from vmxml.xml_to_dom .. Patch Set 6: Code-Review+1 good enough for now. -- To view, visit https://gerrit.ovirt.org/58194 To

Change in vdsm[master]: virt: Switch to etree in vmxml.py

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Switch to etree in vmxml.py .. Patch Set 6: Code-Review+1 my previous comment was addressed partially. I'm fine to have it as proof of concept and

Change in vdsm[master]: virt: Use DOM helpers in Vm disk device handling

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Use DOM helpers in Vm disk device handling .. Patch Set 7: Code-Review+1 thanks, for the updates, looks better now -- To view, visit

Change in vdsm[master]: virt: Introduce MutableDomainDescriptor

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Introduce MutableDomainDescriptor .. Patch Set 1: Code-Review+1 I tend to like it, still not sure this is the best approach (perhaps changing the

Change in vdsm[master]: virt: Use DOM helpers in vmxml.device_address

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Use DOM helpers in vmxml.device_address .. Patch Set 7: Milan answered to this comment in a previous patch, I'm fine with his plan. -- To view, visit

Change in vdsm[master]: virt: Make DomainDescriptor use XML helpers

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Make DomainDescriptor use XML helpers .. Patch Set 8: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/55769 To unsubscribe, visit

Change in vdsm[master]: Send vm info report each sampling interval from VMBulkSampler

2016-07-20 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Send vm info report each sampling interval from VMBulkSampler .. Patch Set 5: Here is a *proof of concept* of the periodic uploader, the alternative approach

Change in vdsm[ovirt-4.0]: oop: Explicitly close running ioprocesses

2016-07-20 Thread fromani
Francesco Romani has submitted this change and it was merged. Change subject: oop: Explicitly close running ioprocesses .. oop: Explicitly close running ioprocesses IOProcess client is using __del__ for automatic closing when

<    5   6   7   8   9   10   11   12   13   14   >