X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/d65964ad6063b6cd5420004a5cbdc58fe2d0b194..8bd2381e50d3c575cef3bc0f0d4c10a9bafaf292:/services/api/app/models/job.rb diff --git a/services/api/app/models/job.rb b/services/api/app/models/job.rb index 1f0ef758d9..d3c3b1d640 100644 --- a/services/api/app/models/job.rb +++ b/services/api/app/models/job.rb @@ -27,6 +27,7 @@ class Job < ArvadosModel t.add :started_at t.add :finished_at t.add :output + t.add :output_is_persistent t.add :success t.add :running t.add :is_locked_by_uuid @@ -36,6 +37,8 @@ class Job < ArvadosModel t.add :dependencies t.add :log_stream_href t.add :log_buffer + t.add :nondeterministic + t.add :repository end def assert_finished @@ -56,12 +59,25 @@ class Job < ArvadosModel order('priority desc, created_at') end + def self.running + self.where('running = ?', true). + order('priority desc, created_at') + end + protected def foreign_key_attributes super + %w(output log) end + def skip_uuid_read_permission_check + super + %w(cancelled_by_client_uuid) + end + + def skip_uuid_existence_check + super + %w(output log) + end + 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 @@ -70,7 +86,7 @@ class Job < ArvadosModel return true end if new_record? or script_version_changed? - sha1 = Commit.find_by_commit_ish(self.script_version) rescue nil + sha1 = Commit.find_commit_range(current_user, nil, nil, self.script_version, nil)[0] rescue nil if sha1 self.script_version = sha1 else