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

Ability to disable publishing to the Pulumi CDN #1077

Merged
merged 2 commits into from
Sep 14, 2024

Conversation

ringods
Copy link
Member

@ringods ringods commented Sep 10, 2024

Ability to disable publishing to the Pulumi CDN (get.pulumi.com) for non-Pulumi managed providers.

Ran make test-providers which didn't render any changes, but it worked for my pulumiverse/pulumi-acme provider.

@guineveresaenger
Copy link
Contributor

I apologize for the question, but what is Pulumi CDN?

This looks reasonable to me otherwise - I would like to see a third-party provider test in the repo maybe, so we can see the applied changes alongside the PR.

@ringods
Copy link
Member Author

ringods commented Sep 13, 2024

@guineveresaenger I updated the initial comment. Pulumi CDN is get.pulumi.com which is not accessible for 3rd party packages.

This commit uses this setting for the Pulumiverse ACME provider. Besides setting the property, it is a no-op because I had removed these steps manually from the setup before.

pulumiverse/pulumi-acme@687dedc

Should I mimic the acme setup under the make test-providers build target?

Copy link
Member

@danielrbradley danielrbradley left a comment

Choose a reason for hiding this comment

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

Happy this is a nice safe change. I think this is very specifically just an s3 upload ... it's just this s3 bucket happens to be behind a cdn. I think with the expanded comment it's fine though.

@ringods
Copy link
Member Author

ringods commented Sep 13, 2024

@guineveresaenger acme added as test provider.

@ringods ringods added this pull request to the merge queue Sep 14, 2024
Merged via the queue into master with commit a5bd46d Sep 14, 2024
6 checks passed
@ringods ringods deleted the configure-publish-to-cdn branch September 14, 2024 18:46
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.

3 participants