Bug #11121

[Crunch] Set output collection owner_uuid to match job

Added by Tom Clegg over 4 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
02/16/2017
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-

Description

Currently, if a job is started with owner_uuid set to a shared project, the job and its log are shared, but its output is not.

We do this with the log collection

  my $log_coll = api_call(
    "collections/create", ensure_unique_name => 1, collection => {
      manifest_text => $log_manifest,
      owner_uuid => $Job->{owner_uuid},
      name => sprintf("Log from %s job %s", $Job->{script}, $Job->{uuid}),
    });

but we need to do it with the output collection too.

  my $pid = open2($child_out, $child_in, 'python', '-c', q{
import arvados
import sys
print (arvados.api("v1").collections().
       create(body={"manifest_text": sys.stdin.read()}).
       execute(num_retries=int(sys.argv[1]))["portable_data_hash"])
}, retry_count());

source:sdk/cli/bin/crunch-job


Subtasks

Task #11125: Review 11121-crunch-output-collection-ownerResolved

Associated revisions

Revision d0ba6eaa
Added by Lucas Di Pentima over 4 years ago

Merge branch '11121-crunch-output-collection-owner'
Closes #11121

History

#1 Updated by Tom Clegg over 4 years ago

  • Description updated (diff)
  • Category set to Crunch

#2 Updated by Tom Clegg over 4 years ago

  • Description updated (diff)

#3 Updated by Lucas Di Pentima over 4 years ago

  • Status changed from New to In Progress
  • Assigned To set to Lucas Di Pentima

#4 Updated by Lucas Di Pentima over 4 years ago

Branch 11121-crunch-output-collection-owner - Update at: 0403a0c
Running tests: https://ci.curoverse.com/job/developer-run-tests/168/

Code update only, test case still pending.

#6 Updated by Tom Clegg over 4 years ago

LGTM, thanks!

#7 Updated by Tom Clegg over 4 years ago

  • Target version set to 2017-03-01 sprint

#8 Updated by Lucas Di Pentima over 4 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:d0ba6eaac1387ad817de1c2df2d7f4f00800aaa5.

Also available in: Atom PDF