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

Upgrade Pulumi and remove scripts/install-pulumi-plugin.js handling #32

Open
t0yv0 opened this issue Sep 12, 2023 · 0 comments
Open

Upgrade Pulumi and remove scripts/install-pulumi-plugin.js handling #32

t0yv0 opened this issue Sep 12, 2023 · 0 comments
Labels
kind/enhancement Improvements or new features

Comments

@t0yv0
Copy link
Member

t0yv0 commented Sep 12, 2023

What happened?

For both native and bridged providers upgrading past
https://github.com/pulumi/pulumi/releases/tag/v3.80.0 introduces an important change (for bridged
providers this starting with
https://github.com/pulumi/pulumi-terraform-bridge/releases/tag/v3.59.0). The change is fully
described in pulumi/pulumi#13800 and affects how Node and Python packages
are generated.
Prior to the change, installing the Node or Python version of a bridged provider via npm or pip
would run a script that ensures that the provider binary itself is installed. After the change, that
is no longer the case, and Pulumi CLI takes over the responsibility for automatic installation.
In practice you may observe files disappearing upon upgrade, for example these files will be removed
from Pulumi bridged providers:
https://github.com/pulumi/pulumi-aws-apigateway/blob/main/sdk/nodejs/scripts/install-pulumi-plugin.js
https://github.com/pulumi/pulumi-azure-native/blob/master/sdk/nodejs/scripts/install-pulumi-plugin.js
If your project is using Makefile targets that deal with these scripts, these require a small
update. For example, in pulumi-azure-native we need to remove the following line from the Makefile:
mkdir -p bin/scripts && cp scripts/install-pulumi-plugin.js bin/scripts
And in pulumi-aws-apigateway this line is removed:
cp -R scripts/ bin &&
https://github.com/pulumi/pulumi-aws-apigateway/pull/85/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52L74

Expected Behavior

Works after upgrade

Steps to reproduce

N/A

Output of pulumi about

N/A

Additional context

N/A

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@mikhailshilkov mikhailshilkov added kind/engineering Work that is not visible to an external user kind/enhancement Improvements or new features and removed kind/engineering Work that is not visible to an external user labels Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Improvements or new features
Projects
None yet
Development

No branches or pull requests

2 participants