- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: enable payload with CDROM
by shavivi＠redhat.com 01 Jan '13

01 Jan '13

Shahar Havivi has submitted this change and it was merged. Change subject: engine: enable payload with CDROM ...................................................................... engine: enable payload with CDROM enable VM to run with iso domain CDROM as well as a payload CDROM Change-Id: Ie1c1a4fc606f67cec90c803d13a052fab9ac1b7e bug-url: https://bugzilla.redhat.com/882651 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/utils/src/test/resources/AppErrors.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 8 files changed, 8 insertions(+), 18 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10422 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1c1a4fc606f67cec90c803d13a052fab9ac1b7e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Renamed event_audit_log_subscriber.java to EventAuditL...
by yzaslavs＠redhat.com 01 Jan '13

01 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Renamed event_audit_log_subscriber.java to EventAuditLogSubscriber.java ...................................................................... core: Renamed event_audit_log_subscriber.java to EventAuditLogSubscriber.java Change-Id: I2be0dc94e27ca51065bfc9c2db601e595d64509e Signed-off-by: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> --- R backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/EventAuditLogSubscriber.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/NotificationService.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/utils/sender/EventSender.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/EventMessageContent.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/EventSenderMailImpl.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/test/java/org/ovirt/engine/core/notifier/utils/mail/MailSenderTest.java 6 files changed, 21 insertions(+), 21 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10051 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2be0dc94e27ca51065bfc9c2db601e595d64509e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Renamed event_notification_methods.java to EventNotifi...
by yzaslavs＠redhat.com 01 Jan '13

01 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Renamed event_notification_methods.java to EventNotificationMethod.java ...................................................................... core: Renamed event_notification_methods.java to EventNotificationMethod.java Change-Id: I583ddb60b305af917db989f2dee0c22f0f7d5a24 Signed-off-by: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddEventSubscriptionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/EventSubscriptionCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveEventSubscriptionCommand.java R backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/EventNotificationMethod.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EventDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EventDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EventDAOWrapperImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/events/NotificationMethodsDAOHibernateImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/BaseDAOTestCase.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/EventDAOTest.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/NotificationService.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/methods/EventMethodFiller.java M backend/manager/tools/engine-notifier/engine-notifier-service/src/main/java/org/ovirt/engine/core/notifier/methods/NotificationMethodMapBuilder.java 13 files changed, 64 insertions(+), 64 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10068 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I583ddb60b305af917db989f2dee0c22f0f7d5a24 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: checkstyle checkstyle
by lhornyak＠redhat.com 01 Jan '13

01 Jan '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: checkstyle checkstyle ...................................................................... engine: checkstyle checkstyle do not import the logger, it is not used Change-Id: Ieb3a57805badbba5978a9c9c7e5b08b468aa0f7f Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M build-tools-root/ovirt-checkstyle-extension/src/main/java/checks/NlsCheck.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10016 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb3a57805badbba5978a9c9c7e5b08b468aa0f7f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: if connect storage pool fails on version mismatch, d...
by yzaslavs＠redhat.com 31 Dec '12

31 Dec '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: engine: if connect storage pool fails on version mismatch, do reconstruct master ...................................................................... engine: if connect storage pool fails on version mismatch, do reconstruct master The following patch will allow to perform reconstruct from InitVdsOnUp, in case that we can not connect to pool because of wrong master version or missing master domain. Using of event queue will protect from parallel reconstructs and from other races. The flow is a following: 1. Try connect to pool 2. If failed on wrong master domain or its version - do reconstruct 3. If success move host to UP Please note: I remove an update of vms at spm after reconstruct, because of it will be removed at the future and that code is actually failed a one host scenario. Change-Id: Ia0d11c068399664daf85a6b03e67d25cd382707e Bug-Url: https://bugzilla.redhat.com/882837 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/eventqueue/EventQueueMonitor.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectHostToStoragePoolServersCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RecoveryStoragePoolCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ReconstructMasterParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/eventqueue/EventType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java 9 files changed, 118 insertions(+), 40 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10103 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0d11c068399664daf85a6b03e67d25cd382707e Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Introducing a queue for failovers event
by yzaslavs＠redhat.com 31 Dec '12

31 Dec '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: engine: Introducing a queue for failovers event ...................................................................... engine: Introducing a queue for failovers event The following patch will introduce a queue for failovers event. The Idea is to use the follwoing queue in order to ensure a correct order of the following operations: reconstruct, domain recovery, domain monitoring, etc... The appropriate event should be submitted to queue and will be executed accorrding to internal logic of the queue. Because of the execution of events is done in a separate thread no deadlock should occurred, also no need for excplicit additional locking This is the first patch it will introduce a queue and all stuff should work as before, but using of queue with small change will allow: to reject unneeded events when some othe event is running, ensure order, ensure appropriate release of quartz threads, fix a races The following patch also introduce a using of queue, from different places, including RecoveryStoragePoolCommand Change-Id: Ic1224feacdcdaaaf0b59d26105805ba7ef2a2fff Bug-Url: https://bugzilla.redhat.com/882837 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MaintananceVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/eventqueue/EventQueueMonitor.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RecoveryStoragePoolCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ReconstructMasterParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RecoveryStoragePoolParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IVdsEventListener.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/eventqueue/Event.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/eventqueue/EventQueue.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/eventqueue/EventResult.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/eventqueue/EventType.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ejb/BeanType.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ejb/EngineEJBUtilsStrategy.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IIrsBroker.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java 17 files changed, 538 insertions(+), 244 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9838 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1224feacdcdaaaf0b59d26105805ba7ef2a2fff Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: liron aravot <liron.aravot(a)gmail.com>

1 0

Change in ovirt-engine[master]: tools: Change default of use_dns_lookup to true
by yzaslavs＠redhat.com 31 Dec '12

31 Dec '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Change default of use_dns_lookup to true ...................................................................... tools: Change default of use_dns_lookup to true Chagning default of this flag to true, as part of the bz handling Change-Id: Ia56ff4afac11803ac0753dbf88fcfbfd24e2e565 Bug-Url: https://bugzilla.redhat.com/868852 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/utils/src/main/resources/engine-manage-domains.conf 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10530 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia56ff4afac11803ac0753dbf88fcfbfd24e2e565 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: ReconstructMasterDomain -connect/disconnect hosts simu...
by tnisan＠redhat.com 31 Dec '12

31 Dec '12

Tal Nisan has submitted this change and it was merged. Change subject: core: ReconstructMasterDomain -connect/disconnect hosts simultaneously ...................................................................... core: ReconstructMasterDomain -connect/disconnect hosts simultaneously When running ReconstructMasterDomain/RecoveryStoragePool commands - any connect/disconnect operations can be done simultaneously by different threads in order to improve the runtime. There's no need to make use here of the runSynchronizeOperation() , it will require to add few classes, handle with parameter passing when this flow is relevant to the Reconstruct flow, in many places in the code we execute runnables using ThreadPoolUtils class so I prefer to keep the implementation simple rather then complicate it and add classes that will be never used again just to use CountDownLatch. Change-Id: I3c8dac368d60eac85a92762dffc5dbb8062b85ef Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java 1 file changed, 52 insertions(+), 39 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10117 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c8dac368d60eac85a92762dffc5dbb8062b85ef Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: add executor for execution of runnables with CountDown...
by tnisan＠redhat.com 31 Dec '12

31 Dec '12

Tal Nisan has submitted this change and it was merged. Change subject: core: add executor for execution of runnables with CountDownLatch ...................................................................... core: add executor for execution of runnables with CountDownLatch This patch adds LatchedRunnableExecuter to be able to execute LatchedRunnableWrapper (pre existing class) instances using CountDownLatch. 1. LatchedRunnableExecuter basically could have been implemented as class whose instances cannot be created and with static methods only - it's has been implemented like this in order to provide proper testing (creating instances of it is rare, so it shouldn't matter in terms of garbage collection), a singleton isn't used as well as it's being used only from Reconstruct flow which is rare, so I prefer to create instance when needed instead of having permanent instance of it. 2. Handle of rejected submission of runnables was removed last patch because its abnormal situation that we shouldn't reach at all in that case according to infra so there's no need to submit runnable again if it failed. 3. Handle of InterruptedException : right now it's caught only as it's being taken care of in other places in the code. Change-Id: Ia56ff4afac01803ac0753dbf88fcfbfd34e2e56d Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/thread/LatchedRunnableExecutor.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/thread/LatchedRunnableWrapperTest.java 2 files changed, 167 insertions(+), 26 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10354 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia56ff4afac01803ac0753dbf88fcfbfd34e2e56d Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: perform connectStorageServer to hosts after during reconstru...
by tnisan＠redhat.com 31 Dec '12

31 Dec '12

Tal Nisan has submitted this change and it was merged. Change subject: perform connectStorageServer to hosts after during reconstruct/recovery(#880180) ...................................................................... perform connectStorageServer to hosts after during reconstruct/recovery(#880180) When performing reconstruct/reinitialize DC, a copy of the pool metadata is being done to a data domain, if hosts doesn't have the needed 'connections' to the new master storage server they will move eventually to non-operational state. This patch adds execution of connectStorageServer to all hosts other then the host that executed the reconstruct/recovery (as this host managed to perform reconstruct successfully, it means that he has the needed 'connections' to the storage server). Change-Id: I3733b732b37f3a687b24f90bfdb1799757434d75 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=880180 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ReconstructMasterDomainCommand.java 1 file changed, 21 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9748 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3733b732b37f3a687b24f90bfdb1799757434d75 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0