1 class Job < ArvadosModel
4 include CommonApiTemplate
5 attr_protected :arvados_sdk_version, :docker_image_locator
6 serialize :script_parameters, Hash
7 serialize :runtime_constraints, Hash
8 serialize :tasks_summary, Hash
9 before_create :ensure_unique_submit_id
10 after_commit :trigger_crunch_dispatch_if_cancelled, :on => :update
11 before_validation :set_priority
12 before_validation :update_state_from_old_state_attrs
13 validate :ensure_script_version_is_commit
14 validate :find_arvados_sdk_version
15 validate :find_docker_image_locator
16 validate :validate_status
17 validate :validate_state_change
18 validate :ensure_no_collection_uuids_in_script_params
19 before_save :update_timestamps_when_state_changes
21 has_many :commit_ancestors, :foreign_key => :descendant, :primary_key => :script_version
22 has_many(:nodes, foreign_key: :job_uuid, primary_key: :uuid)
24 class SubmitIdReused < StandardError
27 api_accessible :user, extend: :common do |t|
31 t.add :script_parameters
34 t.add :cancelled_by_client_uuid
35 t.add :cancelled_by_user_uuid
42 t.add :is_locked_by_uuid
44 t.add :runtime_constraints
47 t.add :nondeterministic
49 t.add :supplied_script_version
50 t.add :arvados_sdk_version
51 t.add :docker_image_locator
57 # Supported states for a job
60 (Running = 'Running'),
61 (Cancelled = 'Cancelled'),
63 (Complete = 'Complete'),
67 update_attributes(finished_at: finished_at || Time.now,
68 success: success.nil? ? false : success,
77 self.where('state = ?', Queued).order('priority desc, created_at')
81 Job::queue.each_with_index do |job, index|
82 if job[:uuid] == self.uuid
90 self.where('running = ?', true).
91 order('priority desc, created_at')
94 def lock locked_by_uuid
97 unless self.state == Queued and self.is_locked_by_uuid.nil?
98 raise AlreadyLockedError
101 self.is_locked_by_uuid = locked_by_uuid
108 def foreign_key_attributes
109 super + %w(output log)
112 def skip_uuid_read_permission_check
113 super + %w(cancelled_by_client_uuid)
116 def skip_uuid_existence_check
117 super + %w(output log)
121 if self.priority.nil?
127 def ensure_script_version_is_commit
128 if self.state == Running
129 # Apparently client has already decided to go for it. This is
130 # needed to run a local job using a local working directory
131 # instead of a commit-ish.
134 if new_record? or script_version_changed?
135 sha1 = Commit.find_commit_range(current_user, self.repository, nil, self.script_version, nil)[0] rescue nil
137 self.supplied_script_version = self.script_version if self.supplied_script_version.nil? or self.supplied_script_version.empty?
138 self.script_version = sha1
140 self.errors.add :script_version, "#{self.script_version} does not resolve to a commit"
146 def ensure_unique_submit_id
148 if Job.where('submit_id=?',self.submit_id).first
149 raise SubmitIdReused.new
155 def resolve_runtime_constraint(key, attr_sym)
156 if ((runtime_constraints.is_a? Hash) and
157 (search = runtime_constraints[key]))
158 ok, result = yield search
160 ok, result = true, nil
163 send("#{attr_sym}=".to_sym, result)
165 errors.add(attr_sym, result)
170 def find_arvados_sdk_version
171 resolve_runtime_constraint("arvados_sdk_version",
172 :arvados_sdk_version) do |git_search|
173 commits = Commit.find_commit_range(current_user, "arvados",
174 nil, git_search, nil)
175 if commits.nil? or commits.empty?
176 [false, "#{git_search} does not resolve to a commit"]
177 elsif not runtime_constraints["docker_image"]
178 [false, "cannot be specified without a Docker image constraint"]
180 [true, commits.first]
185 def find_docker_image_locator
186 resolve_runtime_constraint("docker_image",
187 :docker_image_locator) do |image_search|
188 image_tag = runtime_constraints['docker_image_tag']
189 if coll = Collection.for_latest_docker_image(image_search, image_tag)
190 [true, coll.portable_data_hash]
192 [false, "not found for #{image_search}"]
199 queue = self.script_parameters.values
200 while not queue.empty?
201 queue = queue.flatten.compact.collect do |v|
205 v.match(/^(([0-9a-f]{32})\b(\+[^,]+)?,?)*$/) do |locator|
206 deps[locator.to_s] = true
215 def permission_to_update
216 if is_locked_by_uuid_was and !(current_user and
217 (current_user.uuid == is_locked_by_uuid_was or
218 current_user.uuid == system_user.uuid))
219 if script_changed? or
220 script_parameters_changed? or
221 script_version_changed? or
222 (!cancelled_at_was.nil? and
223 (cancelled_by_client_uuid_changed? or
224 cancelled_by_user_uuid_changed? or
225 cancelled_at_changed?)) or
226 started_at_changed? or
227 finished_at_changed? or
232 tasks_summary_changed? or
234 logger.warn "User #{current_user.uuid if current_user} tried to change protected job attributes on locked #{self.class.to_s} #{uuid_was}"
238 if !is_locked_by_uuid_changed?
242 logger.warn "Anonymous user tried to change lock on #{self.class.to_s} #{uuid_was}"
244 elsif is_locked_by_uuid_was and is_locked_by_uuid_was != current_user.uuid
245 logger.warn "User #{current_user.uuid} tried to steal lock on #{self.class.to_s} #{uuid_was} from #{is_locked_by_uuid_was}"
247 elsif !is_locked_by_uuid.nil? and is_locked_by_uuid != current_user.uuid
248 logger.warn "User #{current_user.uuid} tried to lock #{self.class.to_s} #{uuid_was} with uuid #{is_locked_by_uuid}"
256 def update_modified_by_fields
257 if self.cancelled_at_changed?
258 # Ensure cancelled_at cannot be set to arbitrary non-now times,
259 # or changed once it is set.
260 if self.cancelled_at and not self.cancelled_at_was
261 self.cancelled_at = Time.now
262 self.cancelled_by_user_uuid = current_user.uuid
263 self.cancelled_by_client_uuid = current_api_client.andand.uuid
264 @need_crunch_dispatch_trigger = true
266 self.cancelled_at = self.cancelled_at_was
267 self.cancelled_by_user_uuid = self.cancelled_by_user_uuid_was
268 self.cancelled_by_client_uuid = self.cancelled_by_client_uuid_was
274 def trigger_crunch_dispatch_if_cancelled
275 if @need_crunch_dispatch_trigger
276 File.open(Rails.configuration.crunch_refresh_trigger, 'wb') do
277 # That's all, just create/touch a file for crunch-job to see.
282 def update_timestamps_when_state_changes
283 return if not (state_changed? or new_record?)
287 self.started_at ||= Time.now
288 when Failed, Complete
289 self.finished_at ||= Time.now
291 self.cancelled_at ||= Time.now
294 # TODO: Remove the following case block when old "success" and
295 # "running" attrs go away. Until then, this ensures we still
296 # expose correct success/running flags to older clients, even if
297 # some new clients are writing only the new state attribute.
305 when Cancelled, Failed
312 self.running ||= false # Default to false instead of nil.
314 @need_crunch_dispatch_trigger = true
319 def update_state_from_old_state_attrs
320 # If a client has touched the legacy state attrs, update the
321 # "state" attr to agree with the updated values of the legacy
324 # TODO: Remove this method when old "success" and "running" attrs
326 if cancelled_at_changed? or
331 self.state = Cancelled
332 elsif success == false
334 elsif success == true
335 self.state = Complete
336 elsif running == true
346 if self.state.in?(States)
349 errors.add :state, "#{state.inspect} must be one of: #{States.inspect}"
354 def validate_state_change
356 if self.state_changed?
357 ok = case self.state_was
359 # state isn't set yet
362 # Permit going from queued to any state
365 # From running, may only transition to a finished state
366 [Complete, Failed, Cancelled].include? self.state
367 when Complete, Failed, Cancelled
368 # Once in a finished state, don't permit any more state changes
371 # Any other state transition is also invalid
375 errors.add :state, "invalid change from #{self.state_was} to #{self.state}"
381 def ensure_no_collection_uuids_in_script_params
382 # recursive_hash_search searches recursively through hashes and
383 # arrays in 'thing' for string fields matching regular expression
384 # 'pattern'. Returns true if pattern is found, false otherwise.
385 def recursive_hash_search thing, pattern
388 return true if recursive_hash_search v, pattern
390 elsif thing.is_a? Array
392 return true if recursive_hash_search k, pattern
394 elsif thing.is_a? String
395 return true if thing.match pattern
400 # Fail validation if any script_parameters field includes a string containing a
401 # collection uuid pattern.
402 if self.script_parameters_changed?
403 if recursive_hash_search(self.script_parameters, Collection.uuid_regex)
404 self.errors.add :script_parameters, "must use portable_data_hash instead of collection uuid"