-
Notifications
You must be signed in to change notification settings - Fork 354
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for specifying a SSH key per git repository #37
Comments
👍 for this behavior. However the last time I played with Github deploy keys I think there had to be some special environment variable magic to do this on a per-repo basis. In addition this would need to be done for the Puppetfile as well as top level sources, right? |
Can't this be managed using an SSH config file? Does it make sense for it Greg On May 28, 2013, at 12:04 AM, Adrien Thebo notifications@github.com wrote: [image: 👍] for this behavior. However the last time I played with Github In addition this would need to be done for the Puppetfile as well as top — |
Ancillas: I think maybe your approach is better. It may be out of r10k's area of responsibility. For instance, sharing the Puppetfile between puppet masters and developers, presents problems if it contains sshkey_paths to keys, which may not exist everywhere, and r10k shouldn't generate keys. |
@larstobi to follow up on this do you think this sort of functionality is still needed, or can we close this issue out? |
@adrienthebo yes, I think this functionality is better served by an SSH config file, as @Ancillas suggested. |
Merge 3.99.x into 4.0.x
Using a Puppetfile with modules in private git repositories using SSH requires using a SSH private key.
Using the "deploy key" feature in Github and Github Enterprise will grant access to repositories for machines. It disallows duplicate SSH keys accross the entire site. Each repository needs to have it's separate deploy key.
This could be solved by generating multiple keys, one per repository, and speficying the SSH key to git when fetching.
Example config:
What do you think?
The text was updated successfully, but these errors were encountered: