X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/ee86983a93c335413c508c5717a8707bbced93e6..f2574ec2e4ddef03c9504cd58741e73426ea1b17:/services/api/app/models/job.rb diff --git a/services/api/app/models/job.rb b/services/api/app/models/job.rb index 4af3fa84e9..fbc5640b37 100644 --- a/services/api/app/models/job.rb +++ b/services/api/app/models/job.rb @@ -1,5 +1,5 @@ class Job < ArvadosModel - include AssignUuid + include HasUuid include KindAndEtag include CommonApiTemplate serialize :script_parameters, Hash @@ -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 @@ -69,6 +70,14 @@ class Job < ArvadosModel 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 @@ -77,7 +86,7 @@ class Job < ArvadosModel return true end if new_record? or script_version_changed? - sha1 = Commit.find_commit_range(current_user, nil, nil, self.script_version, nil)[0] rescue nil + sha1 = Commit.find_commit_range(current_user, self.repository, nil, self.script_version, nil)[0] rescue nil if sha1 self.script_version = sha1 else