Project

General

Profile

Actions

Bug #11121

closed

[Crunch] Set output collection owner_uuid to match job

Added by Tom Clegg over 5 years ago. Updated over 5 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 1 (0 open1 closed)

Task #11125: Review 11121-crunch-output-collection-ownerResolved02/16/2017

Actions
Actions #1

Updated by Tom Clegg over 5 years ago

  • Description updated (diff)
  • Category set to Crunch
Actions #2

Updated by Tom Clegg over 5 years ago

  • Description updated (diff)
Actions #3

Updated by Lucas Di Pentima over 5 years ago

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

Updated by Lucas Di Pentima over 5 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.

Actions #6

Updated by Tom Clegg over 5 years ago

LGTM, thanks!

Actions #7

Updated by Tom Clegg over 5 years ago

  • Target version set to 2017-03-01 sprint
Actions #8

Updated by Lucas Di Pentima over 5 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:d0ba6eaac1387ad817de1c2df2d7f4f00800aaa5.

Actions

Also available in: Atom PDF