Nir Soffer has posted comments on this change.
Change subject: tests: Add test for testing the timeout mechanism
..
Patch Set 6: Verified+1
--
To view, visit https://gerrit.ovirt.org/58423
To unsubscribe, visit https://gerrit.
Nir Soffer has posted comments on this change.
Change subject: tests: Terminate watcher when tests fail
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/60239
To unsubscribe, visit https://gerrit.ovirt.org
Nir Soffer has posted comments on this change.
Change subject: tests: Enable tests watcher only if TIMEOUT set
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/60240
To unsubscribe, visit https://gerrit.ov
Nir Soffer has uploaded a new change for review.
Change subject: tests: Enable tests watcher only if TIMEOUT set
..
tests: Enable tests watcher only if TIMEOUT set
When running tests locally we don't need the watcher; if tests g
gerrit-hooks has posted comments on this change.
Change subject: tests: Terminate watcher when tests fail
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matc
Nir Soffer has submitted this change and it was merged.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
spbackends: Remove StoragePoolDiskBackend
As VDSM no longer supports 3.4 clusters (see commit 23650fb), there'
Nir Soffer has posted comments on this change.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
Patch Set 9: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/55859
To unsubscribe, visit https://gerrit.ovirt
Allon Mureinik has posted comments on this change.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
Patch Set 9: Verified+1
Addressed Nir's comments on vdsm-api.yml
--
To view, visit https://gerrit.ovirt.org/55859
gerrit-hooks has posted comments on this change.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
Patch Set 9:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header mat
Allon Mureinik has posted comments on this change.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55859/8/lib/api/vdsm-api.yml
File lib/api/vdsm-api.yml:
Lin
gerrit-hooks has posted comments on this change.
Change subject: sp.py: Assume a hostId when reconstructing master
..
Patch Set 5:
* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To vie
Nir Soffer has submitted this change and it was merged.
Change subject: sp.py: Assume a hostId when reconstructing master
..
sp.py: Assume a hostId when reconstructing master
As we no longer support V1/V2 data domains, we can a
Nir Soffer has posted comments on this change.
Change subject: tox: fail make process if required tox version isn't installed.
..
Patch Set 20:
(7 comments)
https://gerrit.ovirt.org/#/c/59306/20/Makefile.am
File Makefile.am:
Adam Litke has posted comments on this change.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
Patch Set 8: Code-Review+1
Looks good after you address Nir's comment.
--
To view, visit https://gerrit.ovirt.org/558
Adam Litke has posted comments on this change.
Change subject: sp.py: Assume a hostId when reconstructing master
..
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/55857
To unsubscribe, visit https://gerr
Nir Soffer has submitted this change and it was merged.
Change subject: infra: moved infra package from lib/vdsm/infra to
lib/vdsm/common
..
infra: moved infra package from lib/vdsm/infra to lib/vdsm/common
infra package was u
Nir Soffer has posted comments on this change.
Change subject: spbackends: Remove StoragePoolDiskBackend
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55859/8/lib/api/vdsm-api.yml
File lib/api/vdsm-api.yml:
Line 82
Nir Soffer has posted comments on this change.
Change subject: utils: build cert paths in single place
..
Patch Set 10:
(4 comments)
Looks mostly good
https://gerrit.ovirt.org/#/c/52354/10//COMMIT_MSG
Commit Message:
Line 4:
Nir Soffer has posted comments on this change.
Change subject: Using startCmd func for running async execution
..
Patch Set 1:
(4 comments)
https://gerrit.ovirt.org/#/c/60237/1/lib/vdsm/commands.py
File lib/vdsm/commands.py:
gerrit-hooks has posted comments on this change.
Change subject: Remove deathSignal usages in sync execCmd calls
..
Patch Set 4:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure head
gerrit-hooks has posted comments on this change.
Change subject: Using startCmd func for running async execution
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure head
Yaniv Bronhaim has uploaded a new change for review.
Change subject: Using startCmd func for running async execution
..
Using startCmd func for running async execution
Splitting execCmd and startCmd. execCmd still uses deathSign
Sandro Bonazzola has posted comments on this change.
Change subject: readme: Simplify installation instructions
..
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/48713/1//COMMIT_MSG
Commit Message:
Line 6:
Line 7: re
gerrit-hooks has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 17:
* #1344289::Update tracker: OK
* Set MODIFIED::bug 1344289#1344289IGNORE, not all re
Nir Soffer has submitted this change and it was merged.
Change subject: verify_untrusted_volume: Check compatibility version
..
verify_untrusted_volume: Check compatibility version
When importing a volume using imaged, QCOW2 vo
Nir Soffer has posted comments on this change.
Change subject: verify_untrusted_volume: Only allow compat designated by config
..
Patch Set 4: Code-Review-1
--
To view, visit https://gerrit.ovirt.org/60113
To unsubscribe, visi
Nir Soffer has posted comments on this change.
Change subject: verify_untrusted_volume: Only allow compat designated by config
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/60113/1/vdsm/storage/hsm.py
File vdsm/stor
gerrit-hooks has posted comments on this change.
Change subject: verify_untrusted_volume: Only allow compat designated by config
..
Patch Set 4:
* #1344289::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1344289::O
Adam Litke has posted comments on this change.
Change subject: verify_untrusted_volume: Only allow compat designated by config
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/60113/1/vdsm/storage/hsm.py
File vdsm/stor
Nir Soffer has posted comments on this change.
Change subject: image: Add timing for copy volume operations
..
Patch Set 3:
Waiting for qa ack (they ask for this log)
--
To view, visit https://gerrit.ovirt.org/60164
To unsubs
Nir Soffer has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 16: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/59411
To unsubscribe, visit https://
gerrit-hooks has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 16:
* #1344289::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1344289::OK, public
gerrit-hooks has posted comments on this change.
Change subject: verify_untrusted_volume: Only allow compat designated by config
..
Patch Set 3:
* #1344289::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1344289::O
Adam Litke has posted comments on this change.
Change subject: Avoid creating unneeded images
..
Patch Set 1: Code-Review-2
We'll handle in the previous patch
--
To view, visit https://gerrit.ovirt.org/60231
To unsubscribe, v
Adam Litke has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 15:
(2 comments)
https://gerrit.ovirt.org/#/c/59411/15/tests/storage_hsm_test.py
File tests/stora
gerrit-hooks has posted comments on this change.
Change subject: qemuimg: Introduce supports_compat helper
..
Patch Set 14:
* #1344289::Update tracker: OK
* Set MODIFIED::bug 1344289#1344289IGNORE, not all related patch
Nir Soffer has submitted this change and it was merged.
Change subject: qemuimg: Introduce supports_compat helper
..
qemuimg: Introduce supports_compat helper
To enable checking if a qcow2 image has a supported compatibility le
gerrit-hooks has posted comments on this change.
Change subject: qemuimg: Unbreak qmemuimg.check
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit https://g
Piotr Kliczewski has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 14: Verified+1
verified by running jsonrpc Host.getVMList
--
To view, visit https://gerrit.ovirt.org/35181
To un
Nir Soffer has submitted this change and it was merged.
Change subject: qemuimg: Unbreak qmemuimg.check
..
qemuimg: Unbreak qmemuimg.check
Since commit 3e7a11954582 (qemuimg: Use --output json for 'check' and
'info') qemuimg.ch
Piotr Kliczewski has posted comments on this change.
Change subject: stomp: simplified client
..
Patch Set 7: Verified+1
verified by running jsonrpc Host.getVMList
--
To view, visit https://gerrit.ovirt.org/59151
To unsubscri
Piotr Kliczewski has posted comments on this change.
Change subject: utils: build cert paths in single place
..
Patch Set 10: Verified+1
Verified by updating vdsm and starting new vm.
--
To view, visit https://gerrit.ovirt.or
Nir Soffer has posted comments on this change.
Change subject: verify_untrusted_volume: Only allow compat designated by config
..
Patch Set 2: Code-Review-1
See my comment on version 1:
https://gerrit.ovirt.org/#/c/60113/1/vdsm
Nir Soffer has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 15:
(2 comments)
https://gerrit.ovirt.org/#/c/59411/15/tests/storage_hsm_test.py
File tests/stora
Adam Litke has posted comments on this change.
Change subject: image: Add timing for copy volume operations
..
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/60164
To unsubscribe, visit https://gerrit.ov
Adam Litke has posted comments on this change.
Change subject: qemuimg: Unbreak qmemuimg.check
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/60168
To unsubscribe, visit https://gerrit.ovirt.org/setti
gerrit-hooks has posted comments on this change.
Change subject: utils: build cert paths in single place
..
Patch Set 10:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matc
gerrit-hooks has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 14:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-
gerrit-hooks has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 15:
* #1344289::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1344289::OK, public
Adam Litke has posted comments on this change.
Change subject: verify_untrusted_volume: Check compatibility version
..
Patch Set 15: Verified+1
--
To view, visit https://gerrit.ovirt.org/59411
To unsubscribe, visit https://ger
Piotr Kliczewski has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/60206
To unsubscribe, visit ht
gerrit-hooks has posted comments on this change.
Change subject: stomp: simplified client
..
Patch Set 6:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
gerrit-hooks has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 13:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-
Piotr Kliczewski has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 12:
(1 comment)
https://gerrit.ovirt.org/#/c/35181/12//COMMIT_MSG
Commit Message:
Line 23:
Line 24: For invokin
gerrit-hooks has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 12:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-
gerrit-hooks has posted comments on this change.
Change subject: utils: build cert paths in single place
..
Patch Set 8:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header match
gerrit-hooks has posted comments on this change.
Change subject: stomp: simplified client
..
Patch Set 5:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
gerrit-hooks has posted comments on this change.
Change subject: wip: tests: change testing framework to pytest
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure heade
Piotr Kliczewski has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 11:
(2 comments)
https://gerrit.ovirt.org/#/c/35181/11/contrib/jsonrpc
File contrib/jsonrpc:
Line 38:
Line 39:
Dan Kenigsberg has posted comments on this change.
Change subject: ovs hook: fix dhclient imports
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/60154
To unsubscribe, visit https://gerrit.ovirt.org/se
gerrit-hooks has posted comments on this change.
Change subject: wip: tests: change testing framework to pytest
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure heade
Irit Goihman has uploaded a new change for review.
Change subject: wip: tests: change testing framework to pytest
..
wip: tests: change testing framework to pytest
Change-Id: Ife527b9274c31a72b57da9b958a1dd2f9f1e5ea0
Signed-off-
Irit Goihman has posted comments on this change.
Change subject: infra: moved infra package from lib/vdsm/infra to
lib/vdsm/common
..
Patch Set 10:
verified infra removal on fedora 23 and centos 7
--
To view, visit https://g
Ramesh N has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/60206
To unsubscribe, visit https://gerri
Piotr Kliczewski has posted comments on this change.
Change subject: utils: build cert paths in single place
..
Patch Set 7:
(2 comments)
https://gerrit.ovirt.org/#/c/52354/7/lib/vdsm/constants.py.in
File lib/vdsm/constants.py
gerrit-hooks has posted comments on this change.
Change subject: Remove deathSignal usages in sync execCmd calls
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure head
gerrit-hooks has posted comments on this change.
Change subject: draft patch - removing all deathsignal usages where calls are
async only to verify work with subprocess32
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url fo
Yaniv Bronhaim has uploaded a new change for review.
Change subject: draft patch - removing all deathsignal usages where calls are
async only to verify work with subprocess32
..
draft patch - removing all deathsignal usages wher
Dan Kenigsberg has posted comments on this change.
Change subject: utils: build cert paths in single place
..
Patch Set 7:
(2 comments)
(very partial review)
https://gerrit.ovirt.org/#/c/52354/7/lib/vdsm/constants.py.in
File
Dan Kenigsberg has posted comments on this change.
Change subject: contrib: Simple jsonrpc client
..
Patch Set 11: Code-Review-1
(2 comments)
https://gerrit.ovirt.org/#/c/35181/11/contrib/jsonrpc
File contrib/jsonrpc:
Line 38
gerrit-hooks has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 2:
* #1352656::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1352656::OK, public
gerrit-hooks has posted comments on this change.
Change subject: api: fixing default value for GlusterHost.createBrick
..
Patch Set 1:
* #1352656::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1352656::OK, public
Ramesh N has uploaded a new change for review.
Change subject: api: fixing default value for GlusterHost.createBrick
..
api: fixing default value for GlusterHost.createBrick
Change default value for raidParamas to an empty dict
Dan Kenigsberg has posted comments on this change.
Change subject: v2v: Log detailed output of virt-v2v
..
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/59834/4//COMMIT_MSG
Commit Message:
Line 13: It would be cleaner
gerrit-hooks has posted comments on this change.
Change subject: tox: fail make process if required tox version isn't installed.
..
Patch Set 20:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url foun
gerrit-hooks has posted comments on this change.
Change subject: Kill and report asyncProc if fails on first write to stdin
..
Patch Set 10:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, ma
76 matches
Mail list logo