Roy Golan has submitted this change and it was merged.
Change subject: core: Introduce a sequential multiple action runner
......................................................................
core: Introduce a sequential multiple action runner
Add a sequential multiple action runner which execute each validation
and execution after the validation and execution of the previous
command.
Use it to avoid the deletion of the last disk and cpu profile.
Bug-Url:
https://bugzilla.redhat.com/1142880
Change-Id: I76f03a5a87469e7df9ec6f6d08f932e16a1e574f
Signed-off-by: Roman Mohr <rmohr(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GlusterMultipleActionsRunner.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MultipleActionsRunner.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MultipleActionsRunnersFactory.java
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/NestedCommandFactory.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ParallelMultipleActionsRunner.java
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/PrevalidatingMultipleActionsRunner.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmFromPoolRunner.java
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SequentialMultipleActionsRunner.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SortedMultipleActionsRunnerBase.java
A
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MultipleActionsRunnerBaseTest.java
A
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/PrevalidatingMultipleActionsRunnerTest.java
A
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/SequentialMultipleActionsRunnerTest.java
12 files changed, 586 insertions(+), 198 deletions(-)
Approvals:
Martin Peřina: Looks good to me, but someone else must approve
Roman Mohr: Verified
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/52753
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I76f03a5a87469e7df9ec6f6d08f932e16a1e574f
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>