-
Notifications
You must be signed in to change notification settings - Fork 78
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
[nodejs] Add resource registration for VpcCni #590
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. |
The `VpcCni` resource can be passed back to the TypeScript implementation from other languages as a resource reference, and needs to be instantiated as an instance of `VpcCni` when deserialized so it can be used in a `dependsOn`.
Does the PR have any schema changes?Looking good! No breaking changes found. |
@@ -20,7 +20,7 @@ provider:: | |||
|
|||
build_nodejs:: VERSION := $(shell pulumictl get version --language javascript) | |||
build_nodejs:: | |||
rm -rf nodejs/eks/bin | |||
rm -rf nodejs/eks/bin/* |
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.
Otherwise, deleting the whole bin
dir was preventing subsequent yarn link
calls in the install_nodejs_sdk
target from succeeding after a prior yarn link
.
Does the PR have any schema changes?Looking good! No breaking changes found. |
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.
Changes LGTM. Do we have any tests that cover this case?
We don't and I'm not sure we can yet because I hit another issue after this is fixed. Will open a new issue to track it. |
@justinvp Is this still needed? |
Yes. |
The
VpcCni
resource can be passed back to the TypeScript implementation from other languages as a resource reference, and needs to be instantiated as an instance ofVpcCni
when deserialized so it can be used in adependsOn
.Part of #566