Skip to content
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 cloning private repos as part of automation API #5333

Merged
merged 2 commits into from
Sep 14, 2020

Conversation

stack72
Copy link
Contributor

@stack72 stack72 commented Sep 11, 2020

Fixes: #5319

@stack72 stack72 self-assigned this Sep 11, 2020
Copy link
Contributor

@EvanBoyle EvanBoyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! Just a few ideas I'm curious to hear your thoughts on.

sdk/go/x/auto/git.go Outdated Show resolved Hide resolved
sdk/go/x/auto/local_workspace.go Outdated Show resolved Hide resolved
sdk/go/x/auto/local_workspace.go Outdated Show resolved Hide resolved
sdk/go/x/auto/example_test.go Outdated Show resolved Hide resolved
sdk/go/x/auto/git.go Outdated Show resolved Hide resolved
sdk/go/x/auto/git.go Outdated Show resolved Hide resolved
sdk/go/x/auto/local_workspace.go Outdated Show resolved Hide resolved
sdk/go/x/auto/git.go Outdated Show resolved Hide resolved
sdk/go/x/auto/git.go Show resolved Hide resolved
Copy link
Contributor

@EvanBoyle EvanBoyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

sdk/go/x/auto/git.go Show resolved Hide resolved
Copy link
Contributor

@metral metral left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: 🎉

fqsn := FullyQualifiedStackName("myOrg", pName, "myStack")

repo := GitRepo{
URL: "https://github.com/pulumi/test-repo.git",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting, I thought you'd have to specify the URL like git@github.com:pulumi/test-repo.git when using SSH for auth, but it works when doing HTTPS?. Do the git@... style URLs work as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

they do AFAICT

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stack72 Just ran into this for the operator, and the URL does in fact need to be git@github.com... as Justin points out.

Using the HTTPS url results in unable to clone repo: invalid auth method

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #5382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automation API - add support for private git repositories (SSH key authentication)
4 participants