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

Clean the template dir if the remote has changed #6784

Merged
merged 2 commits into from Apr 16, 2021

Conversation

svangordon-fruit
Copy link
Contributor

If the remote of the ~/.pulumi/templates repo is different than the remote provided through the workspace.pulumiTemplateGitRepository, remove that directory. Without this change, if the user has templates cached locally (extremely likely), the client will attempt to clone the directory in ~/.pulumi; when that fails because the dir already exists, it will drop into the templates dir and do a pull. This results in the client continuing to use the previous templates source, instead of the new one set by pulumiTemplateGitRepository.

This change makes #6545 usable.

@github-actions
Copy link

PR is now waiting for a maintainer to run the acceptance tests.

Note for the maintainer: To run the acceptance tests, please comment /run-acceptance-tests on the PR

@github-actions
Copy link

PR is now waiting for a maintainer to run the acceptance tests.

Note for the maintainer: To run the acceptance tests, please comment /run-acceptance-tests on the PR

@leezen leezen requested review from justinvp and stack72 April 15, 2021 22:36
@github-actions
Copy link

PR is now waiting for a maintainer to run the acceptance tests.

Note for the maintainer: To run the acceptance tests, please comment /run-acceptance-tests on the PR

@justinvp
Copy link
Member

/run-acceptance-tests

@github-actions
Copy link

Please view the results of the PR Build + Acceptance Tests Run Here

Copy link
Member

@justinvp justinvp left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@justinvp justinvp merged commit 5e495e8 into pulumi:master Apr 16, 2021
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.

None yet

2 participants