-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 plugin download url metadata to package spec #4947
Conversation
Diff for pulumi-azuread with merge commit a857464 |
Diff for pulumi-random with merge commit a857464 |
c2d3c78
to
f310c21
Compare
Diff for pulumi-kubernetes with merge commit a857464 |
Diff for pulumi-azuread with merge commit f18e16f |
Diff for pulumi-random with merge commit f18e16f |
Diff for pulumi-kubernetes with merge commit f18e16f |
Diff for pulumi-azure with merge commit a857464 |
Diff for pulumi-aws with merge commit a857464 |
Diff for pulumi-gcp with merge commit a857464 |
Diff for pulumi-aws with merge commit f18e16f |
Diff for pulumi-azure with merge commit f18e16f |
Diff for pulumi-gcp with merge commit f18e16f |
After a discussion about terminology, the better phrasing is to use pluginDownloadURL
908e92c
to
c25cbf0
Compare
Diff for pulumi-random with merge commit bd6da18 |
Diff for pulumi-azuread with merge commit bd6da18 |
Diff for pulumi-kubernetes with merge commit bd6da18 |
Diff for pulumi-gcp with merge commit bd6da18 |
Diff for pulumi-azure with merge commit bd6da18 |
Diff for pulumi-aws with merge commit bd6da18 |
Diff for pulumi-azuread with merge commit 2ba5f5d |
Diff for pulumi-random with merge commit 2ba5f5d |
Diff for pulumi-kubernetes with merge commit 2ba5f5d |
Diff for pulumi-gcp with merge commit 2ba5f5d |
Diff for pulumi-aws with merge commit 2ba5f5d |
Diff for pulumi-azure with merge commit 2ba5f5d |
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.
Will this approach only work for python and NodeJS? What can be done about Go/DotNet - or is that a different conversation?
The code looks fine here - have you been able to hook this up to a provider to get it working?
@stack72 yes it'll only work for those languages. I have a successful test working with the rke provider which i'll send over! |
This optional field allows providers and plugins to specify a URL to download a provider plugin from. This will allow community plugins to use automatic plugin acquisition
Currently this is just a noop, the same changes need to be passed through to
pulumi-terraform-bridge
as well.