DotCi icon indicating copy to clipboard operation
DotCi copied to clipboard

Only a single private repository per organisation is supported

Open dougjohnson opened this issue 11 years ago • 6 comments
trafficstars

Private repository support was added recently (see #6), which is fantastic - thank you so much! The implementation as far as I can see depends on a single global deploy key being set with an expectation that this can be reused across all private repositories.

Unfortunately, as can be seen in the GitHub documentation here, deploy keys must be unique across an organisation.

As a consequence, adding a new DotCI job for a second private repository in the same organisation results in this error:

screen shot 2014-07-10 at 14 15 34

Is it a mis-configuration by me that is causing this, or is it a genuine issue?

dougjohnson avatar Jul 10 '14 13:07 dougjohnson

@dougjohnson Yes it is a genuine issue. I misunderstood how deploy keys work when I implemented this feature. Current implementation is wrong. Thank you bringing this up.

suryagaddipati avatar Jul 10 '14 13:07 suryagaddipati

For the time being you can get round this issue by creating a jenkins user on github and ensuring that it has appropriate permissions, then when adding a private repo immediately delete the deploy key.

samriley avatar Jul 19 '14 21:07 samriley

Would it be possible to make a release that doesn't try to create the key if the field is empty ?

zimbatm avatar Sep 10 '14 14:09 zimbatm

@zimbatm Its on my roadmap. May be try the the workaround mentioned by @samriley in the meantime.

suryagaddipati avatar Sep 10 '14 15:09 suryagaddipati

+1 to get this fixed.

@samriley, to clarify, why do you need to delete the deploy key, and are you deleting it from github or Jenkins?

ronwsmith avatar Dec 31 '14 04:12 ronwsmith

You delete them from github.

If you have a user set up that can access the repo then you dont need the deploy key. It's then just a case of deleting the deploy key on github so that dotci can add the key to another repo.

I think I spotted a fix committed recently-ish, so this might not be needed on the latest version.

samriley avatar Dec 31 '14 07:12 samriley