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

Create empty ssh key for git client to avoid creating real deploy keys when using dry-run. #1054

Merged
merged 2 commits into from Nov 17, 2021

Conversation

josecordaz
Copy link
Contributor

Closes: #1050

What changed?
Avoid creating deploy key when running add app using --dry-run flag

Why?
To avoid the error "deploy key already exists" when adding the app without dry-run flag

How did you test it?
Manually and adding a unit test.

Release notes

Documentation Changes

@josecordaz josecordaz added the bug Something isn't working label Nov 10, 2021
@josecordaz josecordaz marked this pull request as ready for review November 10, 2021 15:29
Copy link
Contributor

@J-Thompson12 J-Thompson12 left a comment

Choose a reason for hiding this comment

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

LGTM. It would be nice to handle the dry run just inside factory.go so you dont need to pass it around but I think its more unclean to move makePublicKey somewhere that factory can access it.

@jpellizzari jpellizzari merged commit 2f4c835 into main Nov 17, 2021
@jpellizzari jpellizzari deleted the 1050-fix-deploy-key-on-dry-run branch November 17, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add app --dry-run breaks add app by creating the deploy key
3 participants