Project

General

Profile

Actions

Bug #5318

closed

[Workbench] Make note of lag time when changing git permissions

Added by Bryan Cosca about 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Story points:
-

Description

I created a new git repo bcosc.gatk2 using the admin account for , which has manage access.

git clone :bcosc.gatk2.git
Cloning into 'bcosc.gatk2'...
R access for bcosc.gatk2 DENIED to qr1hi-tpzed-rv6f0l8lbvdzkog
(Or there may be no repository at the given path. Did you spell it correctly?)
fatal: The remote end hung up unexpectedly

Then I just granted them owner access and the git clone worked.


Subtasks 1 (0 open1 closed)

Task #5505: Review 5318-git-sharing-noteResolvedPeter Amstutz02/26/2015Actions
Actions #1

Updated by Brett Smith about 9 years ago

  • Target version set to Bug Triage

Bryan,

Can you please clarify what you mean when you say you "granted them owner access?" If you created the repository with your BU account, I believe it would've been owned by that same user from the time it was created.

How quickly did all this happen? It can take a minute or so for changes to git repositories in Arvados to be reflected on the git server. Like the error message indicates, it's possible that the repository simply didn't exist when you first tried to clone it. Is that a reasonable possibility?

Thanks.

Actions #2

Updated by Tom Clegg about 9 years ago

  • Category set to API
  • Status changed from New to Feedback
Actions #3

Updated by Tom Clegg about 9 years ago

  • Target version changed from Bug Triage to 2015-04-01 sprint
Actions #4

Updated by Peter Amstutz about 9 years ago

  • Subject changed from Creating a new git repo and granting manage access denied a git clone to [Workbench] Make note of lag time when changing git permissions
  • Status changed from Feedback to In Progress

Updating this story to add a note to Workbench that there is a lag time on updating git permissions.

Actions #5

Updated by Peter Amstutz about 9 years ago

  • Assigned To set to Peter Amstutz
Actions #6

Updated by Brett Smith about 9 years ago

The code at 4e72d37 is fine, but the text seems kind of easy to miss. I wonder if we could maybe hook into the link modification JavaScript to show a Bootstrap alert after the user makes a change? That seems like it would be much more effective. But if it's too much trouble, this is good to merge. Thanks.

Actions #7

Updated by Peter Amstutz about 9 years ago

I get what you're saying, but I don't really have time to fiddle with it more, so I'm merging as is. Thanks.

Actions #8

Updated by Peter Amstutz about 9 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:23721458d7ec8c44c4b54f45eb7d2a4e6d7cdce1.

Actions

Also available in: Atom PDF