Nir Soffer has posted comments on this change.
Change subject: tests: Rename lvmTests to new naming convention
..
Patch Set 5:
Ping
--
To view, visit https://gerrit.ovirt.org/64328
To unsubscribe, visit
Nir Soffer has posted comments on this change.
Change subject: guest-lvs: Add lvm bootstrap tests
..
Patch Set 8: Code-Review-1
(7 comments)
https://gerrit.ovirt.org/#/c/64367/8/tests/storage_lvm_test.py
File
Nir Soffer has posted comments on this change.
Change subject: tests: Add loop module
..
Patch Set 5: Code-Review-1
(6 comments)
Needs more work to be useful for more interesting lvm tests.
Nir Soffer has posted comments on this change.
Change subject: service: Add vdsm-tool-configure service
..
Patch Set 3: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/64904/3//COMMIT_MSG
Commit Message:
Line 9:
gerrit-hooks has posted comments on this change.
Change subject: guest-lvs: Deactivate guest lvs during deactivation
..
Patch Set 4:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public
gerrit-hooks has posted comments on this change.
Change subject: guest-lvs: Deactivate guest lvs during bootstrap
..
Patch Set 3:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public bug
gerrit-hooks has posted comments on this change.
Change subject: tests: Rename lvmTests to new naming convention
..
Patch Set 5:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public bug
*
gerrit-hooks has posted comments on this change.
Change subject: guest-lvs: Skip foreign vgs during bootstrap
..
Patch Set 4:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public bug
*
gerrit-hooks has posted comments on this change.
Change subject: guest-lvs: Add failing test for guest lvs
..
Patch Set 9:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public bug
* Check
gerrit-hooks has posted comments on this change.
Change subject: guest-lvs: Add lvm bootstrap tests
..
Patch Set 8:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public bug
* Check
gerrit-hooks has posted comments on this change.
Change subject: tests: Add loop module
..
Patch Set 5:
* #1374545::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1374545::OK, public bug
* Check
Adam Litke has posted comments on this change.
Change subject: yml: parameter type fixes for StoragePool.connectStorageServer
..
Patch Set 2:
(3 comments)
https://gerrit.ovirt.org/#/c/59701/2/lib/api/vdsm-api.yml
File
Nir Soffer has posted comments on this change.
Change subject: wip: jsonrpc: introduce new client
..
Patch Set 9:
(2 comments)
We also need tests for this.
https://gerrit.ovirt.org/#/c/64502/9/lib/vdsm/client.py
File
gerrit-hooks has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 5:
* #65004::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Nir Soffer has submitted this change and it was merged.
Change subject: tests: Use fresh ResourceManager for every test
..
tests: Use fresh ResourceManager for every test
Previously if unregistring a namespace failed during
gerrit-hooks has posted comments on this change.
Change subject: tests: Streamline resourceManager import
..
Patch Set 4:
* #65003::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 4:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Nir Soffer has submitted this change and it was merged.
Change subject: tests: Streamline resourceManager import
..
tests: Streamline resourceManager import
In the application we typically import resourceManager as rm. Do the
Nir Soffer has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 3: Continuous-Integration+1
Network again
--
To view, visit https://gerrit.ovirt.org/65004
To
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 4:
* #65002::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: py3: make gluster_cli_tests pass
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Nir Soffer has submitted this change and it was merged.
Change subject: resourceManager: Remove unused listNamespaces
..
resourceManager: Remove unused listNamespaces
The only usage was a test for this, so we don't really need
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 3: Continuous-Integration+1
Network tests
--
To view, visit https://gerrit.ovirt.org/65002
To
Dan Kenigsberg has posted comments on this change.
Change subject: py3: make gluster_cli_tests pass
..
Patch Set 1:
Ramesh, can you verify that the change does not harm vdsm nor its gluster
support?
--
To view, visit
Irit Goihman has posted comments on this change.
Change subject: wip: jsonrpc: introduce new client
..
Patch Set 9:
(10 comments)
https://gerrit.ovirt.org/#/c/64502/8/lib/vdsm/client.py
File lib/vdsm/client.py:
Line 17: #
gerrit-hooks has posted comments on this change.
Change subject: wip: jsonrpc: introduce new client
..
Patch Set 9:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: blockSD: Storage domain life cycle management
..
Patch Set 7:
* #1331978::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1331978::OK, public bug
*
Nir Soffer has posted comments on this change.
Change subject: volume: Add qcow2_compat on create
..
Patch Set 19: Continuous-Integration+1
Network tests again
--
To view, visit https://gerrit.ovirt.org/64375
To unsubscribe,
Michal Skrivanek has posted comments on this change.
Change subject: virt: Make boolean values from boolean migration options
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/65007
To unsubscribe,
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move ResourceInfo class to module
..
Patch Set 4:
* #65001::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Nir Soffer has submitted this change and it was merged.
Change subject: resourceManager: Move ResourceInfo class to module
..
resourceManager: Move ResourceInfo class to module
Simplify ResourceManger class by moving the
gerrit-hooks has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move ResourceInfo class to module
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: tests: Streamline resourceManager import
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move Namespace class to module
..
Patch Set 3:
* #65000::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Irit Goihman has posted comments on this change.
Change subject: infra tests: added functional test for upgrading vdsm
..
Patch Set 14:
(1 comment)
https://gerrit.ovirt.org/#/c/61186/14/tests/functional/upgrade_vdsm_test.py
Nir Soffer has submitted this change and it was merged.
Change subject: resourceManager: Move Namespace class to module
..
resourceManager: Move Namespace class to module
Simplify the ResourceManager class by moving the nested
Milan Zamazal has posted comments on this change.
Change subject: virt: Don't crash in migration progress on compression bytes
..
Patch Set 2: Verified+1
Bug number added.
Verified that with this change the crash (KeyError)
gerrit-hooks has posted comments on this change.
Change subject: py3: make gluster_cli_tests pass
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Dan Kenigsberg has uploaded a new change for review.
Change subject: py3: make gluster_cli_tests pass
..
py3: make gluster_cli_tests pass
Change-Id: I7e212e247f057f2988debaf3b2de923851b90b24
Signed-off-by: Dan Kenigsberg
gerrit-hooks has posted comments on this change.
Change subject: virt: Don't crash in migration progress on compression bytes
..
Patch Set 2:
* #64468::Update tracker: OK
* #1380822::Update tracker: OK
* Check Bug-Url::OK
*
gerrit-hooks has posted comments on this change.
Change subject: virt: Make boolean values from boolean migration options
..
Patch Set 1:
* #1380822::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1380822::OK,
Milan Zamazal has uploaded a new change for review.
Change subject: virt: Make boolean values from boolean migration options
..
virt: Make boolean values from boolean migration options
`compressed' and `autoConverge' migration
gerrit-hooks has posted comments on this change.
Change subject: py3: let bulk_sampling_test pass under py3
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Dan Kenigsberg has uploaded a new change for review.
Change subject: py3: let bulk_sampling_test pass under py3
..
py3: let bulk_sampling_test pass under py3
Change-Id: Ic8f148a7984fab0a092fca588b5de61bdeb4e899
Signed-off-by:
Adam Litke has posted comments on this change.
Change subject: tests: Streamline resourceManager import
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/65003
To unsubscribe, visit
Adam Litke has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/65002
To unsubscribe, visit
Adam Litke has posted comments on this change.
Change subject: resourceManager: Move ResourceInfo class to module
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/65001
To unsubscribe, visit
Adam Litke has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 2: Code-Review+2
Nice!
--
To view, visit https://gerrit.ovirt.org/65004
To unsubscribe, visit
Adam Litke has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 2:
CI failing on network tests again.
--
To view, visit https://gerrit.ovirt.org/65002
To unsubscribe,
Adam Litke has posted comments on this change.
Change subject: resourceManager: Move Namespace class to module
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/65000
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Flatten LockType constants
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Flatten LockType constants
..
Patch Set 3: Continuous-Integration+1
--
To view, visit https://gerrit.ovirt.org/63628
To unsubscribe, visit
Nir Soffer has submitted this change and it was merged.
Change subject: resourceManager: Flatten LockType constants
..
resourceManager: Flatten LockType constants
Replace the LockType class with flat constants, streamlining
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move ResourceInfo class to module
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move Namespace class to module
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: tests: Streamline resourceManager import
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Flatten LockType constants
..
Patch Set 4:
* #63628::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Inline LockType.validate
..
Patch Set 4:
* #63627::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Nir Soffer has submitted this change and it was merged.
Change subject: resourceManager: Inline LockType.validate
..
resourceManager: Inline LockType.validate
Inline the function in the single call site, allowing replacing
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Inline LockType.validate
..
Patch Set 3: Continuous-Integration+1
--
To view, visit https://gerrit.ovirt.org/63627
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Inline LockType.validate
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Remove unused method
..
Patch Set 3:
* #63626::Update tracker: OK
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Nir Soffer has submitted this change and it was merged.
Change subject: resourceManager: Remove unused method
..
resourceManager: Remove unused method
LockType.fromState is not used. Remove it so we can replace the LockType
Adam Litke has posted comments on this change.
Change subject: resourceManager: Flatten LockType constants
..
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/63628
To unsubscribe, visit
Arik Hadas has posted comments on this change.
Change subject: remove unused field in vm stats
..
Patch Set 2:
Francesco, I understand the aim to preserve backward compatibility - but I
think we should be realistic, I believe
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Remove unused method
..
Patch Set 2: Continuous-Integration+1
Network tests on python 3, cannot be more irrelevant.
--
To view, visit
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 1: Verified+1
Verified by the tests.
--
To view, visit https://gerrit.ovirt.org/65002
To unsubscribe,
Nir Soffer has uploaded a new change for review.
Change subject: resourceManager: Move Namespace class to module
..
resourceManager: Move Namespace class to module
Simplify the ResourceManager class by moving the nested
Nir Soffer has uploaded a new change for review.
Change subject: resourceManager: Move ResourceInfo class to module
..
resourceManager: Move ResourceInfo class to module
Simplify ResourceManger class by moving the nested
Nir Soffer has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 1: Verified+1
Verified by the tests.
--
To view, visit https://gerrit.ovirt.org/65004
To
gerrit-hooks has posted comments on this change.
Change subject: tests: Streamline resourceManager import
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Move Namespace class to module
..
Patch Set 1: Verified+1
Verfied by the tests.
--
To view, visit https://gerrit.ovirt.org/65000
To unsubscribe,
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Remove unused listNamespaces
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Nir Soffer has posted comments on this change.
Change subject: tests: Streamline resourceManager import
..
Patch Set 1: Verified+1
Verified by the tests.
--
To view, visit https://gerrit.ovirt.org/65003
To unsubscribe, visit
Nir Soffer has uploaded a new change for review.
Change subject: tests: Streamline resourceManager import
..
tests: Streamline resourceManager import
In the application we typically import resourceManager as rm. Do the
same in
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move ResourceInfo class to module
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
gerrit-hooks has posted comments on this change.
Change subject: tests: Use fresh ResourceManager for every test
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Nir Soffer has uploaded a new change for review.
Change subject: tests: Use fresh ResourceManager for every test
..
tests: Use fresh ResourceManager for every test
Previously if unregistring a namespace failed during shutdown
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Move Namespace class to module
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Move ResourceInfo class to module
..
Patch Set 1: Verified+1
Verified by the tests.
--
To view, visit https://gerrit.ovirt.org/65001
To
Nir Soffer has uploaded a new change for review.
Change subject: resourceManager: Remove unused listNamespaces
..
resourceManager: Remove unused listNamespaces
The only usage was a test for this, so we don't really need it,
gerrit-hooks has posted comments on this change.
Change subject: remove unused field in vm stats
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Nir Soffer has posted comments on this change.
Change subject: acceptor: stop to double close acceptor
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/63685/1/lib/vdsm/protocoldetector.py
File
Arik Hadas has posted comments on this change.
Change subject: remove unused fields in vm stats
..
Patch Set 1: -Verified
indeed, I didn't take MOM into account.
Martin, I guess MOM needs vmName only for logging, right? and
Piotr Kliczewski has posted comments on this change.
Change subject: acceptor: stop to double close acceptor
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/63685/1/lib/vdsm/protocoldetector.py
File
gerrit-hooks has posted comments on this change.
Change subject: virt: Try to detect non guest iniated shutdowns
..
Patch Set 5:
* #1341106::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1341106::OK, public bug
*
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Flatten LockType constants
..
Patch Set 2: Verified+1
This version fixes a new usage of resourceManager.LockType added in
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Inline LockType.validate
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/63627
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Inline LockType.validate
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Flatten LockType constants
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Nir Soffer has posted comments on this change.
Change subject: resourceManager: Remove unused method
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/63626
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: resourceManager: Remove unused method
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header
Nir Soffer has posted comments on this change.
Change subject: acceptor: stop to double close acceptor
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/63685/1/lib/vdsm/protocoldetector.py
File
Piotr Kliczewski has posted comments on this change.
Change subject: acceptor: stop to double close acceptor
..
Patch Set 1:
(1 comment)
https://gerrit.ovirt.org/#/c/63685/1/lib/vdsm/protocoldetector.py
File
gerrit-hooks has posted comments on this change.
Change subject: virt: Try to detect non guest iniated shutdowns
..
Patch Set 4:
* #1341106::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1341106::OK, public bug
*
Piotr Kliczewski has posted comments on this change.
Change subject: infra tests: added functional test for upgrading vdsm
..
Patch Set 14:
(1 comment)
1 - 100 of 145 matches
Mail list logo