Project

General

Profile

Actions

Bug #3785

closed

[Crunch] Job log collections should be stored with the same owner_uuid (and therefore permissions) as the job

Added by Brett Smith over 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Story points:
0.5

Description

A user has permission to see a Job through a Project. They can visit the Job page just fine. However, they do not have permission to read the Job's log. Crunch should arrange for logs to be readable by anyone with read permission to the owning Project.

This won't address the related problem that log/output (and their permissions) stay in project A when the relevant job/pipeline moves from project A to project B.


Subtasks 2 (0 open2 closed)

Task #5271: Fix crunch-jobResolvedPeter Amstutz09/02/2014Actions
Task #5295: Review 3785-job-log-collection-ownerResolvedRadhika Chippada09/02/2014Actions
Actions #1

Updated by Ward Vandewege over 9 years ago

  • Target version set to Arvados Future Sprints
Actions #2

Updated by Tom Clegg about 9 years ago

  • Subject changed from [Crunch] Project Pipeline Job Logs should be readable by Project participants to [Crunch] Job log collections should be stored with the same owner_uuid (and therefore permissions) as the job
  • Description updated (diff)
  • Target version changed from Arvados Future Sprints to 2015-03-11 sprint
Actions #3

Updated by Ward Vandewege about 9 years ago

  • Story points set to 0.5
Actions #4

Updated by Peter Amstutz about 9 years ago

  • Assigned To set to Peter Amstutz
Actions #5

Updated by Radhika Chippada about 9 years ago

Peter,
The update to set the owner_uuid for the log itself looks good.

Just a couple of questions.

  • Is it possible to test this update?
  • Now that we are setting the owner_uuid of the job log correctly, it is desirable that we no longer "hide" the Log tab for anonymous users. Would you update workbench/app/controllers/jobs_controller -> show_pane_list accordingly? There might a couple tests that need to be updated as well.

Thanks.

Actions #6

Updated by Peter Amstutz about 9 years ago

Radhika Chippada wrote:

Peter,
The update to set the owner_uuid for the log itself looks good.

Just a couple of questions.

  • Is it possible to test this update?

As has been often discussed, unfortunately we don't have a testing framework for crunch-job, so there is no way to easily add an automated test.

  • Now that we are setting the owner_uuid of the job log correctly, it is desirable that we no longer "hide" the Log tab for anonymous users. Would you update workbench/app/controllers/jobs_controller -> show_pane_list accordingly? There might a couple tests that need to be updated as well.

Fixed show_pane_list. Ran all the workbench tests, no failures.

Thanks.

Actions #7

Updated by Radhika Chippada about 9 years ago

Peter, I understand about the testing crunch jobs. LGTM

Actions #8

Updated by Peter Amstutz about 9 years ago

  • Status changed from New to Resolved

Applied in changeset arvados|commit:18ddae75beb91b0792ad34a9b4996316715e67b7.

Actions

Also available in: Atom PDF