Arik Hadas has submitted this change and it was merged.
Change subject: engine: Remove VM Pool even if VMs are running
......................................................................
engine: Remove VM Pool even if VMs are running
Support VM Pool removal even in the case when some VMs from it are
running.
If some of the VMs in the pool are running, RemoveVmPoolCommand stops
them and uses CommandCoordinator to wait until all VMs in the pool will
be down. After that the pool with all VMs is removed as usually.
Change-Id: I778362023e9f03b17e7306425e84245c0b9fcec0
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1246886
Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmPoolCommand.java
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmPoolCommandCallback.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
4 files changed, 134 insertions(+), 24 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Shmuel Leib Melamud: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/47654
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I778362023e9f03b17e7306425e84245c0b9fcec0
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org