Change in ovirt-engine[master]: added support for isRunOnlyIfAllValidation flag

ahadas at redhat.com ahadas at redhat.com
Thu Aug 25 06:25:43 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: added support for isRunOnlyIfAllValidation flag
......................................................................


added support for isRunOnlyIfAllValidation flag

Added support for running the runMultipleAction() method with the
isRunOnlyIfAllValidation flag.

The isRunOnlyIfAllValidation flag existed in code but was not supported
(i.e. was always treated as set to false)

It was used by Gluster with flag set to true and therefore changed to
false to avoid regression and should be
tested by them after this fix.

Change-Id: Ic3ae6bfd5929d0fd804dae4f55d18e9dd0b1eb8e
Bug-Url: https://bugzilla.redhat.com/1364171
Signed-off-by: Sharon Gratch <sgratch at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/PrevalidatingMultipleActionsRunner.java
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/communication/GWTRPCCommunicationProvider.java
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/communication/VdcOperation.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGeneralModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
6 files changed, 42 insertions(+), 17 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Sharon Gratch: Verified
  Arik Hadas: Looks good to me, but someone else must approve
  Moti Asayag: Looks good to me, approved
  Ramesh N: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/62684
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic3ae6bfd5929d0fd804dae4f55d18e9dd0b1eb8e
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sharon Gratch <sgratch at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Sharon Gratch <sgratch at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list