From: Jan Provaznik <jprov...@redhat.com>

For now condor supports only 'on-demand' state syncing, so
we have to sync state at least when listing instances.
condormatic_instances_sync_states method is already called when
listing instances in a pool, but we should sync state also when
listing all instances, shouldn't we?
---
 src/app/controllers/instance_controller.rb |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/app/controllers/instance_controller.rb 
b/src/app/controllers/instance_controller.rb
index fcb237e..e94eea9 100644
--- a/src/app/controllers/instance_controller.rb
+++ b/src/app/controllers/instance_controller.rb
@@ -32,6 +32,9 @@ class InstanceController < ApplicationController
   def index
     require_privilege(Privilege::INSTANCE_VIEW)
 
+    # go to condor and sync the database to the real instance states
+    condormatic_instances_sync_states
+
     @order_dir = params[:order_dir] == 'desc' ? 'desc' : 'asc'
     @order = params[:order] || 'name'
     @instances = Instance.search_filter(params[:search], 
Instance::SEARCHABLE_COLUMNS).paginate(
-- 
1.7.0.1

_______________________________________________
deltacloud-devel mailing list
deltacloud-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to