-class Job < OrvosModel
+class Job < ArvadosModel
include AssignUuid
include KindAndEtag
include CommonApiTemplate
serialize :tasks_summary, Hash
before_create :ensure_unique_submit_id
before_create :ensure_script_version_is_commit
+ before_update :ensure_script_version_is_commit
has_many :commit_ancestors, :foreign_key => :descendant, :primary_key => :script_version
t.add :script_parameters
t.add :script_version
t.add :cancelled_at
- t.add :cancelled_by_client
- t.add :cancelled_by_user
+ t.add :cancelled_by_client_uuid
+ t.add :cancelled_by_user_uuid
t.add :started_at
t.add :finished_at
t.add :output
t.add :success
t.add :running
- t.add :is_locked_by
+ t.add :is_locked_by_uuid
t.add :log
t.add :resource_limits
t.add :tasks_summary
running: false)
end
+ def self.queue
+ self.where('started_at is ? and is_locked_by_uuid is ? and cancelled_at is ?',
+ nil, nil, nil).
+ order('priority desc, created_at')
+ end
+
protected
def ensure_script_version_is_commit
+ if self.is_locked_by_uuid and self.started_at
+ # Apparently client has already decided to go for it. This is
+ # needed to run a local job using a local working directory
+ # instead of a commit-ish.
+ return true
+ end
sha1 = Commit.find_by_commit_ish(self.script_version) rescue nil
if sha1
self.script_version = sha1
self.script_parameters.values.each do |v|
next unless v.is_a? String
v.match(/^(([0-9a-f]{32})\b(\+[^,]+)?,?)*$/) do |locator|
- bare_locator = locator[0].gsub(/\+[^,]+/,'')
- deps[bare_locator] = true
+ deps[locator] = true
end
end
deps.keys
elsif is_locked_by_was and is_locked_by_was != current_user.uuid
logger.warn "User #{current_user.uuid} tried to steal lock on #{self.class.to_s} #{uuid_was} from #{is_locked_by_was}"
false
- elsif !is_locked_by.nil? and is_locked_by != current_user.uuid
- logger.warn "User #{current_user.uuid} tried to lock #{self.class.to_s} #{uuid_was} with uuid #{is_locked_by}"
+ elsif !is_locked_by_uuid.nil? and is_locked_by_uuid != current_user.uuid
+ logger.warn "User #{current_user.uuid} tried to lock #{self.class.to_s} #{uuid_was} with uuid #{is_locked_by_uuid}"
false
else
super