cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kelv...@apache.org
Subject git commit: updated refs/heads/4.3 to 1acdbf8
Date Thu, 16 Jan 2014 00:21:48 GMT
Updated Branches:
  refs/heads/4.3 5c7772509 -> 1acdbf836


CLOUDSTACK-5696: fix a regression caused by out-of-band sync changes


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/1acdbf83
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/1acdbf83
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/1acdbf83

Branch: refs/heads/4.3
Commit: 1acdbf8369c68fc99d8ea06204895cb55965d49a
Parents: 5c77725
Author: Kelven Yang <kelveny@gmail.com>
Authored: Wed Jan 15 16:21:05 2014 -0800
Committer: Kelven Yang <kelveny@gmail.com>
Committed: Wed Jan 15 16:21:29 2014 -0800

----------------------------------------------------------------------
 api/src/com/cloud/vm/VirtualMachine.java           | 17 +++++++++--------
 .../framework/jobs/impl/AsyncJobManagerImpl.java   |  4 +++-
 2 files changed, 12 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/1acdbf83/api/src/com/cloud/vm/VirtualMachine.java
----------------------------------------------------------------------
diff --git a/api/src/com/cloud/vm/VirtualMachine.java b/api/src/com/cloud/vm/VirtualMachine.java
index 3400898..b88df22 100755
--- a/api/src/com/cloud/vm/VirtualMachine.java
+++ b/api/src/com/cloud/vm/VirtualMachine.java
@@ -29,7 +29,7 @@ import com.cloud.utils.fsm.StateObject;
 
 /**
  * VirtualMachine describes the properties held by a virtual machine
- * 
+ *
  */
 public interface VirtualMachine extends RunningOn, ControlledEntity, Identity, InternalIdentity,
StateObject<VirtualMachine.State> {
 
@@ -117,7 +117,7 @@ public interface VirtualMachine extends RunningOn, ControlledEntity, Identity,
I
             s_fsm.addTransition(State.Expunging, VirtualMachine.Event.ExpungeOperation, State.Expunging);
             s_fsm.addTransition(State.Error, VirtualMachine.Event.DestroyRequested, State.Expunging);
             s_fsm.addTransition(State.Error, VirtualMachine.Event.ExpungeOperation, State.Expunging);
-            			
+
             s_fsm.addTransition(State.Stopping, VirtualMachine.Event.FollowAgentPowerOnReport,
State.Running);
             s_fsm.addTransition(State.Stopped, VirtualMachine.Event.FollowAgentPowerOnReport,
State.Running);
             s_fsm.addTransition(State.Running, VirtualMachine.Event.FollowAgentPowerOnReport,
State.Running);
@@ -126,8 +126,9 @@ public interface VirtualMachine extends RunningOn, ControlledEntity, Identity,
I
             s_fsm.addTransition(State.Stopping, VirtualMachine.Event.FollowAgentPowerOffReport,
State.Stopped);
             s_fsm.addTransition(State.Running, VirtualMachine.Event.FollowAgentPowerOffReport,
State.Stopped);
             s_fsm.addTransition(State.Migrating, VirtualMachine.Event.FollowAgentPowerOffReport,
State.Stopped);
+            s_fsm.addTransition(State.Stopped, VirtualMachine.Event.FollowAgentPowerOffReport,
State.Stopped);
         }
-        
+
         public static boolean isVmStarted(State oldState, Event e, State newState) {
             if (oldState == State.Starting && newState == State.Running) {
                 return true;
@@ -194,10 +195,10 @@ public interface VirtualMachine extends RunningOn, ControlledEntity,
Identity, I
         AgentReportMigrated,
         RevertRequested,
         SnapshotRequested,
-        
+
         // added for new VMSync logic
         FollowAgentPowerOnReport,
-        FollowAgentPowerOffReport,        
+        FollowAgentPowerOffReport,
     };
 
     public enum Type {
@@ -214,7 +215,7 @@ public interface VirtualMachine extends RunningOn, ControlledEntity, Identity,
I
          * VM with this type. UserBareMetal should treat exactly as User.
          */
         UserBareMetal(false),
-        
+
         /*
          * General VM type for queuing VM orchestration work
          */
@@ -274,7 +275,7 @@ public interface VirtualMachine extends RunningOn, ControlledEntity, Identity,
I
 
     /**
      * returns the guest OS ID
-     * 
+     *
      * @return guestOSId
      */
     long getGuestOSId();
@@ -313,7 +314,7 @@ public interface VirtualMachine extends RunningOn, ControlledEntity, Identity,
I
     Date getCreated();
 
     long getServiceOfferingId();
-    
+
     Long getDiskOfferingId();
 
     Type getType();

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/1acdbf83/framework/jobs/src/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java
----------------------------------------------------------------------
diff --git a/framework/jobs/src/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java
b/framework/jobs/src/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java
index cf7da81..a77f864 100644
--- a/framework/jobs/src/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java
+++ b/framework/jobs/src/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java
@@ -516,7 +516,9 @@ public class AsyncJobManagerImpl extends ManagerBase implements AsyncJobManager,
                         if (jobDispatcher != null) {
                             jobDispatcher.runJob(job);
                         } else {
-                            s_logger.error("Unable to find a wakeup dispatcher from the joined
job: " + job);
+                            // TODO, job wakeup is not in use yet
+                            if (s_logger.isTraceEnabled())
+                                s_logger.trace("Unable to find a wakeup dispatcher from the
joined job: " + job);
                         }
                     } else {
                         AsyncJobDispatcher jobDispatcher = getDispatcher(job.getDispatcher());


Mime
View raw message