openstack/fuel-astute
Code Issues Proposed changes

Skip nodes for provision if they already ready/provisioned

Browse Source
This commit is contained in:
Mike Scherbakov 2013-08-05 03:30:16 +04:00
parent e94b6ab378
commit 9bb2edb557
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/astute/orchestrator.rb
View File

@ -85,8 +85,14 @@ module Astute
end
end
def provision(reporter, task_id, nodes)
raise "Nodes to provision are not provided!" if nodes.empty?
def provision(reporter, task_id, nodes_up)
raise "Nodes to provision are not provided!" if nodes_up.empty?
# We need only those which are not ready/provisioned yet
nodes = []
nodes_up.each do |n|
nodes << n unless ['provisioned', 'ready'].include?(n['status'])
end
# Following line fixes issues with uids: it should always be string
nodes.map { |x| x['uid'] = x['uid'].to_s } # NOTE: perform that on environment['nodes'] initialization