Michael Kublin has uploaded a new change for review.

Change subject: core: Fixing some race during start up of JBOSS
......................................................................

core: Fixing some race during start up of JBOSS

The following patch will solve unneeded race which
can occurre during start up of JBOSS.
A race is update of status of vdsm and running of monitoring process of
vdsm

Change-Id: Ic440c2f5532ce063af158c3e7d1761950bb9807f
Signed-off-by: Michael Kublin <[email protected]>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/43/8643/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
index f3e16fe..fa04c0e 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
@@ -181,6 +181,7 @@
             vds.setpending_vcpus_count(0);
             vdsManager.UpdateDynamicData(vds.getDynamicData());
         }
+        vdsManager.schedulJobs();
         _vdsManagersDict.put(vds.getId(), vdsManager);
         log.infoFormat("ResourceManager::AddVds - VDS {0} was added to the 
Resource Manager", vds.getId());
 
diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
index 6dd49c7..7399d67 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
@@ -159,11 +159,10 @@
 
     public static VdsManager buildVdsManager(VDS vds) {
         VdsManager vdsManager = new VdsManager(vds);
-        vdsManager.schedulJobs();
         return vdsManager;
     }
 
-    private void schedulJobs() {
+    public void schedulJobs() {
         SchedulerUtil sched = SchedulerUtilQuartzImpl.getInstance();
         duringFailureJobId = sched.scheduleAFixedDelayJob(this, 
"OnVdsDuringFailureTimer", new Class[0],
                     new Object[0], VDS_DURING_FAILURE_TIMEOUT_IN_MINUTES, 
VDS_DURING_FAILURE_TIMEOUT_IN_MINUTES,


--
To view, visit http://gerrit.ovirt.org/8643
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic440c2f5532ce063af158c3e7d1761950bb9807f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to