-
Notifications
You must be signed in to change notification settings - Fork 225
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
Pulumi Import Bucket V2 example fix #13663
Conversation
@@ -38,7 +38,7 @@ Both approaches allow you to adopt and begin managing existing cloud resources w | |||
|
|||
Import uses the selected stack's configured [provider](/docs/concepts/resources/providers/) to look up the desired resource in the cloud provider, read its current configuration, and add the resource to the stack to bring it under management by Pulumi from that point forward. For this, it requires two pieces of information: | |||
|
|||
* The _type_ of cloud resource to import --- either as a type _token_ (a string that uniquely identifies a Pulumi resource type) when using the CLI or as a resource declaration when importing in code. The type token of an Amazon S3 Bucket resource, for example, is `aws:s3/bucketv2:BucketV2`. | |||
* The _type_ of cloud resource to import --- either as a type _token_ (a string that uniquely identifies a Pulumi resource type) when using the CLI or as a resource declaration when importing in code. The type token of an Amazon S3 Bucket resource, for example, is `aws:s3/bucketV2:BucketV2`. |
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.
Your site preview for commit 27dbfdf is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-13663-27dbfdf9.s3-website.us-west-2.amazonaws.com. |
@t0yv0 I can just merge this or do we also need to update: https://github.com/pulumi/pulumi-aws/blob/df65048bb98916e5381bfc196653dd7e17db4664/provider/cmd/pulumi-resource-aws/schema.json#L348909 |
You can just merge. The ref'd schema is the source of truth and it has the correct form just as your change does. |
Fixes: #13655
Proposed changes
Unreleased product version (optional)
Related issues (optional)