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
codegen/{go,js,.net,py}: Generate retainOnDelete #12306
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
Changelog[uncommitted] (2023-03-06)Features
|
707ae8f
to
f0a76d5
Compare
Temporarily pins to the branch for pulumi/pulumi#12306 so that we can wire up the codegen changes.
It most certainly does. I don't know where this is getting the .NET SDK, but it might just be a super old version. |
f0a76d5
to
4bac333
Compare
5ede5bd
to
a3a8172
Compare
4bac333
to
50372c4
Compare
Yup it's linking to Pulumi.Random 4.2.0 which is from like 2021. |
Adds the most recent relese of pulumi-random to included schemas. We need something newer than 4.2.0 or 4.3.1 (both from 2021) to test options added since then, like retainOnDelete. Refs #12306 (comment)
Adds the most recent relese of pulumi-random to included schemas. We need something newer than 4.2.0 or 4.3.1 (both from 2021) to test options added since then, like retainOnDelete. Refs #12306 (comment)
12326: testdata/schema: Add random 4.11.2 r=abhinav a=abhinav Adds the most recent relese of pulumi-random to included schemas. We need something newer than 4.2.0 or 4.3.1 (both from 2021) to test options added since then, like retainOnDelete. Refs #12306 (comment) Co-authored-by: Abhinav Gupta <abhinav@pulumi.com>
a3a8172
to
7d1a9ec
Compare
50372c4
to
95f76c0
Compare
Thanks! Fixed in #12326. |
Uses the newly added `options.retainOnDelete` in PCL to generate the RetainOnDelete resource option for Go, NodeJS, .NET, and Python. Java support cannot be added without a release of pulumi/pulumi because the new field in pcl.ResourceOptions is not yet visible to Java. Resolves #12304
95f76c0
to
407e695
Compare
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.
Nice and simple!
bors merge |
Build succeeded: |
Uses the newly added
options.retainOnDelete
in PCLto generate the RetainOnDelete resource option
for Go, NodeJS, .NET, and Python.
Java support cannot be added without a release of pulumi/pulumi
because the new field in pcl.ResourceOptions is not yet visible
to Java.
The change has been staged at pulumi/pulumi-java#1005
pending merge and release of #12305.
Resolves #12304