Change in vdsm[master]: imagetickets: add tests

2016-02-29 Thread aaviram
Amit Aviram has posted comments on this change. Change subject: imagetickets: add tests .. Patch Set 10: (2 comments) https://gerrit.ovirt.org/#/c/52900/10/tests/imagetickets_test.py File tests/imagetickets_test.py: Line 121:

Change in vdsm[master]: bridge: obtain method

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: bridge: obtain method .. Patch Set 7: (1 comment) https://gerrit.ovirt.org/#/c/53472/7/tests/stompTests.py File tests/stompTests.py: Line 55: def unregi

Change in vdsm[master]: migration: added support for convergence schedule

2016-02-29 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: migration: added support for convergence schedule .. Patch Set 18: Verified+1 -- To view, visit https://gerrit.ovirt.org/46940 To unsubscribe, visit https://ger

Change in vdsm[master]: move logUtils to lib

2016-02-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: move logUtils to lib .. move logUtils to lib Change-Id: I39b833ff5746db7a764ab9d9120e73a083d14365 Signed-off-by: Yaniv Bronhaim Reviewed-on: https://g

Change in vdsm[master]: move logUtils to lib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: move logUtils to lib .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt

Change in vdsm[ovirt-3.6]: lib: executor: log pool status on discard

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: lib: executor: log pool status on discard .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/48417 To unsubscribe, visit https://gerrit.ovirt

Change in vdsm[master]: imagetickets: add tests

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: imagetickets: add tests .. Patch Set 10: (2 comments) https://gerrit.ovirt.org/#/c/52900/10/tests/imagetickets_test.py File tests/imagetickets_test.py: Line 121:

Change in vdsm[master]: move logUtils to lib

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: move logUtils to lib .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/53889 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit

Change in vdsm[master]: bridge: obtain method

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: bridge: obtain method .. Patch Set 7: (1 comment) https://gerrit.ovirt.org/#/c/53472/7/tests/stompTests.py File tests/stompTests.py: Line 55: def unregister_s

Change in vdsm[master]: fileUtils: Add atomic_symlink utility

2016-02-29 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: fileUtils: Add atomic_symlink utility .. fileUtils: Add atomic_symlink utility This helper function creates or updates a symlink atomically. If the symlink exists

Change in vdsm[master]: fileUtils: Add atomic_symlink utility

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: fileUtils: Add atomic_symlink utility .. Patch Set 1: Verified+1 Verified by the new tests. -- To view, visit https://gerrit.ovirt.org/53680 To unsubscribe, visit

Change in vdsm[master]: fileUtils: Add atomic_symlink utility

2016-02-29 Thread alitke
Adam Litke has posted comments on this change. Change subject: fileUtils: Add atomic_symlink utility .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/53680 To unsubscribe, visit https://gerrit.ovirt.org

Change in vdsm[ovirt-3.6]: virt: stats: make _disk_iops_bytes more robust

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: stats: make _disk_iops_bytes more robust .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/53997 To unsubscribe, visit https://gerrit.

Change in vdsm[ovirt-3.6]: virt: stats: make compute_latency more robust

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: stats: make compute_latency more robust .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/53996 To unsubscribe, visit https://gerrit.o

Change in vdsm[ovirt-3.6]: virt: stats: make disk_rate more robust

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: virt: stats: make disk_rate more robust .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/53995 To unsubscribe, visit https://gerrit.ovirt.o

Change in vdsm[master]: jsonrpc: provide more info when method not found

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: jsonrpc: provide more info when method not found .. Patch Set 4: (2 comments) https://gerrit.ovirt.org/#/c/53714/4/lib/yajsonrpc/__init__.py File lib/yajsonr

Change in vdsm[master]: spec: Add a dependency to vdsm on vdsm-cli

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: spec: Add a dependency to vdsm on vdsm-cli .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/54006/1/vdsm.spec.in File vdsm.spec.in: Line 255: Requir

Change in vdsm[master]: json: use forzenset for filtered methods

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: json: use forzenset for filtered methods .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/54160/1/lib/yajsonrpc/__init__.py File lib/yajsonrpc/__init

Change in vdsm[master]: json: use forzenset for filtered methods

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: json: use forzenset for filtered methods .. Patch Set 1: Code-Review+1 (1 comment) https://gerrit.ovirt.org/#/c/54160/1/lib/yajsonrpc/__init__.py File lib/yajsonrp

Change in vdsm[master]: move alignmentScan.py to lib

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: move alignmentScan.py to lib .. Patch Set 6: Code-Review-1 (1 comment) https://gerrit.ovirt.org/#/c/53758/6/tests/alignmentScanTests.py File tests/alignmentScanTes

Change in vdsm[master]: move blkid to lib

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: move blkid to lib .. Patch Set 2: Code-Review+1 Looks safe, but we need to verify the flow when preparing a UUID type drive. I don't know how to trigger this flow,

Change in vdsm[master]: gluster: Don't fail connect server when getting volume info

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: gluster: Don't fail connect server when getting volume info .. Patch Set 5: (3 comments) https://gerrit.ovirt.org/#/c/53785/5/vdsm/storage/storageServer.py File vd

Change in vdsm[master]: jsonrpc: provide more info when method not found

2016-02-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jsonrpc: provide more info when method not found .. Patch Set 4: (2 comments) https://gerrit.ovirt.org/#/c/53714/4/lib/yajsonrpc/__init__.py File lib/yajsonrpc/__i

Change in vdsm[ovirt-3.6]: net: use the correct dict with all the networks for disable_...

2016-02-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: use the correct dict with all the networks for disable_ipv6 restoration .. Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/53942/2/vdsm/vdsm-restor

Change in vdsm[master]: numa: add docstrings for public methods

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: add docstrings for public methods .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54171 To unsubscribe, visit https://gerrit.o

Change in vdsm[master]: numa: reorganize module

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: reorganize module .. Patch Set 3: Code-Review-1 (1 comment) -1 for visibility: looks fine but I'd like more context https://gerrit.ovirt.org/#/c/54167

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: caps/numa: move autonuma status to sitelib .. Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54166 To unsubscribe, visit https://gerri

Change in vdsm[master]: numa: refactor to single pass caps parse

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: refactor to single pass caps parse .. Patch Set 8: (1 comment) partial review, I need to do another pass later https://gerrit.ovirt.org/#/c/53288/8/li

Change in vdsm[master]: numa: move cpu topology (numa) related functionality to sitelib

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: move cpu topology (numa) related functionality to sitelib .. Patch Set 8: Code-Review+2 (1 comment) looks fine, but pretty please read inside comments

Change in vdsm[master]: numa: move numa related functionality to sitelib

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: move numa related functionality to sitelib .. Patch Set 7: Code-Review-1 (2 comments) looks fine, but we should take some (very cheap) preventive actio

Change in vdsm[master]: numa: get distance information from libvirt

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: get distance information from libvirt .. Patch Set 5: +2 because is virt/sla flow -- To view, visit https://gerrit.ovirt.org/53285 To unsubscribe, vis

Change in vdsm[master]: numa: get distance information from libvirt

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: numa: get distance information from libvirt .. Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/53285 To unsubscribe, visit https://gerr

Change in vdsm[master]: migration: Add retry on full capacity

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: migration: Add retry on full capacity .. Patch Set 18: Code-Review-1 -- To view, visit https://gerrit.ovirt.org/52799 To unsubscribe, visit https://gerrit.ov

Change in vdsm[master]: migration: Add retry on full capacity

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: migration: Add retry on full capacity .. Patch Set 18: Code-Review+1 (3 comments) Looks good, but a couple of questions inside. Overall, -1 for visibility h

Change in vdsm[master]: vm: Move saveState() out of constructor

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: Move saveState() out of constructor .. Patch Set 3: Code-Review-1 -1 for visibility, please read. This change looks correct, but devil's in the details. C

Change in vdsm[master]: response: Add support for specific error checking in is_error

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: response: Add support for specific error checking in is_error .. Patch Set 2: Code-Review+1 Looks fine, +2'd on hold: please don't forget to add a comment on

Change in vdsm[master]: response: Add support for specific error checking in is_error

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: response: Add support for specific error checking in is_error .. Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54036 To unsubscribe,

Change in vdsm[master]: Apply storage QoS on running VM

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: Apply storage QoS on running VM .. Patch Set 14: Code-Review-1 (3 comments) Mostly OK, but please check iniline comments, especially about IDs. -1 for visibl

Change in vdsm[master]: virt: Add `vm' argument to underlying_device_info methods

2016-02-29 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: Add `vm' argument to underlying_device_info methods .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/53676/1//COMMIT_MSG Commit Message: Line

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

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: WIP: vm: skip domDependentInit for containers .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: WIP: add sudo permissions to run containers

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: WIP: add sudo permissions to run containers .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header m

Change in vdsm[master]: containers: always add emulator to XML

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: containers: always add emulator to XML .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matche

Change in vdsm[master]: virt: recovery: refactor entry points

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: recovery: refactor entry points .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: containers: detect and report container support

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: containers: detect and report container support .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure head

Change in vdsm[master]: containers: make sure directory are ready

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: containers: make sure directory are ready .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header mat

Change in vdsm[master]: containers: initial support

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: containers: initial support .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: numa: add docstrings for public methods

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: add docstrings for public methods .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header match

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: caps/numa: move autonuma status to sitelib .. Patch Set 5: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header ma

Change in vdsm[master]: numa: reorganize module

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: reorganize module .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: numa: add docstrings for public methods

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: add docstrings for public methods .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header match

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: caps/numa: move autonuma status to sitelib .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header ma

Change in vdsm[master]: numa: reorganize module

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: reorganize module .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: sync debian folder with supervdsm_api dir

2016-02-29 Thread stirabos
Simone Tiraboschi has posted comments on this change. Change subject: sync debian folder with supervdsm_api dir .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/54153 To unsubscribe, visit https://gerri

Change in vdsm[master]: numa: add docstrings for public methods

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: add docstrings for public methods .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header match

Change in vdsm[master]: numa: add docstrings for public methods

2016-02-29 Thread mpolednik
Martin Polednik has uploaded a new change for review. Change subject: numa: add docstrings for public methods .. numa: add docstrings for public methods Change-Id: Id84872556a3f0699a738071ca7d3f9809bea305b Signed-off-by: Martin

Change in vdsm[master]: createImageLinks: recreate if the link exists but it's broken

2016-02-29 Thread stirabos
Simone Tiraboschi has posted comments on this change. Change subject: createImageLinks: recreate if the link exists but it's broken .. Patch Set 7: (1 comment) https://gerrit.ovirt.org/#/c/52937/7/vdsm/storage/fileSD.py File v

Change in vdsm[master]: numa: reorganize module

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: reorganize module .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: caps/numa: move autonuma status to sitelib .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header ma

Change in vdsm[master]: numa: reorganize module

2016-02-29 Thread mpolednik
Martin Polednik has uploaded a new change for review. Change subject: numa: reorganize module .. numa: reorganize module Change-Id: I7e8ca3092d4c14827d0965a535ef3d0ac7ba4b0a Signed-off-by: Martin Polednik --- M lib/vdsm/numa.py

Change in vdsm[master]: numa: move numa related functionality to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: move numa related functionality to sitelib .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure hea

Change in vdsm[master]: numa: move cpu topology (numa) related functionality to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: move cpu topology (numa) related functionality to sitelib .. Patch Set 8: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found

Change in vdsm[master]: numa: refactor to single pass caps parse

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: refactor to single pass caps parse .. Patch Set 8: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matc

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: caps/numa: move autonuma status to sitelib .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header ma

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: caps/numa: move autonuma status to sitelib .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header ma

Change in vdsm[master]: caps/numa: move autonuma status to sitelib

2016-02-29 Thread mpolednik
Martin Polednik has uploaded a new change for review. Change subject: caps/numa: move autonuma status to sitelib .. caps/numa: move autonuma status to sitelib Change-Id: I42e7eae743d319b0afd9653a9d6f1f2701856ca9 Signed-off-by: M

Change in vdsm[master]: gluster: Don't fail connect server when getting volume info

2016-02-29 Thread ahino
Ala Hino has posted comments on this change. Change subject: gluster: Don't fail connect server when getting volume info .. Patch Set 5: Tried the following: Provided this dummy path: test:/vol1 And provide these mount options:

Change in vdsm[master]: api: remove unused methods

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: api: remove unused methods .. Patch Set 3: Verified+1 Verified by running local build. -- To view, visit https://gerrit.ovirt.org/53673 To unsubscribe, visi

Change in vdsm[master]: api: remove unused methods

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: api: remove unused methods .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: numa: get distance information from libvirt

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: get distance information from libvirt .. Patch Set 5: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header m

Change in vdsm[master]: numa: move cpu topology (numa) related functionality to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: move cpu topology (numa) related functionality to sitelib .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found

Change in vdsm[master]: numa: move numa related functionality to sitelib

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: move numa related functionality to sitelib .. Patch Set 6: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure hea

Change in vdsm[master]: numa: refactor to single pass caps parse

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: numa: refactor to single pass caps parse .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matc

Change in vdsm[master]: [wip] bridge: usage of yaml schema

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: [wip] bridge: usage of yaml schema .. Patch Set 4: Verified+1 Verified by host deploying the host and seeing no issues. Before I assume this patch to be read

Change in vdsm[master]: contrib: schema converter

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: contrib: schema converter .. Patch Set 16: Verified+1 Verified by host deploying the host and seeing no issues. -- To view, visit https://gerrit.ovirt.org/5

Change in vdsm[master]: bridge: obtain method

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: bridge: obtain method .. Patch Set 7: Verified+1 Verified by host deploying the host and seeing no issues. -- To view, visit https://gerrit.ovirt.org/53472

Change in vdsm[master]: json: use forzenset for filtered methods

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: json: use forzenset for filtered methods .. Patch Set 1: Verified+1 Verified by host deploying the host and seeing no issues. -- To view, visit https://gerr

Change in vdsm[master]: bridge: remove dead code

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: bridge: remove dead code .. Patch Set 3: Verified+1 Verified by host deploying the host and seeing no issues. -- To view, visit https://gerrit.ovirt.org/539

Change in vdsm[master]: jsonrpc: provide more info when method not found

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: jsonrpc: provide more info when method not found .. Patch Set 4: Verified+1 Verified by host deploying the host and seeing no issues. -- To view, visit http

Change in vdsm[master]: net: api: move networks removal to a separate function

2016-02-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: api: move networks removal to a separate function .. net: api: move networks removal to a separate function As a part of preparation of network/ap

Change in vdsm[master]: net: api: move networks removal to a separate function

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: api: move networks removal to a separate function .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- T

Change in vdsm[master]: net: api: move networks removal to a separate function

2016-02-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: api: move networks removal to a separate function .. Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54086 To unsubscribe, visit htt

Change in vdsm[master]: bridge: remove dead code

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: bridge: remove dead code .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '

Change in vdsm[master]: schema: remove unused types

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: schema: remove unused types .. Patch Set 6: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: contrib: schema converter

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: contrib: schema converter .. Patch Set 16: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: json: use forzenset for filtered methods

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: json: use forzenset for filtered methods .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matc

Change in vdsm[master]: [wip] bridge: usage of yaml schema

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: [wip] bridge: usage of yaml schema .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'B

Change in vdsm[master]: gluster: fix schema

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: gluster: fix schema .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and

Change in vdsm[master]: json: use forzenset for filtered methods

2016-02-29 Thread piotr . kliczewski
Piotr Kliczewski has uploaded a new change for review. Change subject: json: use forzenset for filtered methods .. json: use forzenset for filtered methods Change-Id: I17becc319fb8e226f0badc883007900c8862a520 Signed-off-by: pkl

Change in vdsm[master]: jsonrpc: provide more info when method not found

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jsonrpc: provide more info when method not found .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure hea

Change in vdsm[master]: bridge: obtain method

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: bridge: obtain method .. Patch Set 7: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' an

Change in vdsm[master]: net: tests: cleanup net after test_rollback

2016-02-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: tests: cleanup net after test_rollback .. net: tests: cleanup net after test_rollback Without @cleanupNet decorator, failed test can leave network

Change in vdsm[master]: net: tests: cleanup net after test_rollback

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: tests: cleanup net after test_rollback .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, vis

Change in vdsm[master]: net: tests: cleanup net after test_rollback

2016-02-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: tests: cleanup net after test_rollback .. Patch Set 2: Code-Review+2 Edy's comment has been acted upon -- To view, visit https://gerrit.ovirt.org/54010 T

Change in vdsm[master]: net: tests: cleanup net after test_rollback

2016-02-29 Thread phoracek
Petr Horáček has posted comments on this change. Change subject: net: tests: cleanup net after test_rollback .. Patch Set 2: Verified+1 -- To view, visit https://gerrit.ovirt.org/54010 To unsubscribe, visit https://gerrit.ovir

Change in vdsm[master]: net: rename destroyOnMasterRemoval

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: rename destroyOnMasterRemoval .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https:

Change in vdsm[master]: net: rename destroyOnMasterRemoval

2016-02-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: rename destroyOnMasterRemoval .. net: rename destroyOnMasterRemoval destroyOnMasterRemoval parameter was needed to say whether bonding should be r

Change in vdsm[master]: net: rename destroyOnMasterRemoval

2016-02-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: rename destroyOnMasterRemoval .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/54082 To unsubscribe, visit https://gerrit.ovirt.or

Change in vdsm[master]: gluster: Don't fail connect server when getting volume info

2016-02-29 Thread ahino
Ala Hino has posted comments on this change. Change subject: gluster: Don't fail connect server when getting volume info .. Patch Set 4: (1 comment) https://gerrit.ovirt.org/#/c/53785/4/vdsm/storage/storageServer.py File vdsm/

Change in vdsm[master]: gluster: Don't fail connect server when getting volume info

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: gluster: Don't fail connect server when getting volume info .. Patch Set 5: * #1303977::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1303977::OK, p

Change in vdsm[master]: Apply storage QoS on running VM

2016-02-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Apply storage QoS on running VM .. Patch Set 14: * #1201482::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1201482::OK, public bug * Check Product::

  1   2   >