-
Notifications
You must be signed in to change notification settings - Fork 255
Rewrite and reorganize packages concepts page #16318
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
base: master
Are you sure you want to change the base?
Conversation
- Completely rewrote the packages page content for clarity and better organization - Moved page from Building & Extending guides to IaC Concepts section (after Components, before Assets & Archives) - Removed outdated conceptual diagrams and references to non-existent images - Added practical guidance on choosing between local packages and published SDKs - Clarified the difference between authoring components vs providers - Added alias for previous path to preserve SEO and prevent broken links - Removed unused images (pulumi-package-overview.png and pulumi-package-concepts.png) that are duplicated in blog post 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Documentation ReviewThank you for this comprehensive rewrite! The reorganization significantly improves clarity and organization. I found a few minor issues to address: Issues FoundLine 70: Inconsistent terminology Line 73: Missing nuance Line 76: Sentence clarity Line 79: Passive construction Positive Observations
Verification NeededThe file path remains the same in the repository, but the menu structure has changed (moved from iac-guides-packages to iac-concepts). The alias has been added correctly. However, I recommend verifying that:
Mention @claude if you'd like me to review any updates or help with additional changes. |
Your site preview for commit dcc9387 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-dcc9387c.s3-website.us-west-2.amazonaws.com. |
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
In order to consume a Pulumi package, you must have the runtime installed for the language in which the provider is written. For example, if a package is written in TypeScript, any consumers of the package will require the NodeJS runtime to be installed in order to use the package. | ||
|
||
All Pulumi Packages must include a [schema](/docs/using-pulumi/pulumi-packages/schema/), which defines the resources and functions exposed by the package, and is used to drive the generation of language-specific SDKs and documentation. | ||
Go compiles to a native binary and does not require a runtime, and thus has the lowest overhead for package consumers. Most Pulumi packages in the Pulumi Registry (including all of the packages for the major cloud providers) are written in Go. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is only true for binary based plugins. Source based plugins will still need the Go compiler installed. Also dotnet plugins can be published as either runtime-dependent or runtime-included binaries (we'd recommend the latter). So what you need can get a bit fuzzy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added the gory details. Would be awesome if you could check for accuracy. (Leaving this convo unresolved until I know the info I added is complete and accurate.)
If you are authoring a Pulumi component to be shared within your team or organization, you will need to decide whether to use local packages or publish SDKs. **Most component authors will want consumers to use local packages** for the following reasons: | ||
|
||
- The overhead of publishing SDKs can be significant: your CI/CD process will need to generate SDKs for all Pulumi languages (or at least all the languages your package consumers will use) and you will need package feeds to host those published SDKs. | ||
- Pulumi only offers tooling for writing packages with published SDKs in Go, the [Pulumi Provider SDK](/docs/iac/guides/building-extending/providers/pulumi-provider-sdk/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is less a distinction of local or published SDKs and more of custom or components. Once you've built a provider with a schema in any language our tooling for SDK generation and publishing is the same.
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
Your site preview for commit 52edd6d is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-52edd6dc.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 413898e is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-413898e8.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit aa90482 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-aa904829.s3-website.us-west-2.amazonaws.com. |
…e' from custom resource description.
Your site preview for commit 7f8fe0e is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-7f8fe0e5.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 997519b is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-997519be.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 9bdc01c is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-9bdc01cb.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit b00ce8e is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-b00ce8e6.s3-website.us-west-2.amazonaws.com. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have all the technical context to sanity check all the content, but I've made some readability suggestions for the prose.
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
content/docs/iac/guides/building-extending/packages/packages.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Cam Soper <[email protected]>
Your site preview for commit 686ca46 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-686ca464.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 097ee50 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16318-097ee50a.s3-website.us-west-2.amazonaws.com. |
🤖 Generated with Claude Code