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

Support static file assets. #23

Merged
merged 7 commits into from Apr 25, 2022
Merged

Support static file assets. #23

merged 7 commits into from Apr 25, 2022

Conversation

pgavlin
Copy link
Member

@pgavlin pgavlin commented Apr 22, 2022

Static file assets--that is, file assets that are not generated by
running a command--are translated to S3 bucket object resources. As part
of these changes, a new node is created in the resource tree that
corresponds to the CDK stack itself in order to simplify the dependency
graph. This node depends on any assets required by the stack (rather
than plumbing the assets through to each of the stack's children).

Static file assets--that is, file assets that are not generated by
running a command--are translated to S3 bucket object resources. As part
of these changes, a new node is created in the resource tree that
corresponds to the CDK stack itself in order to simplify the dependency
graph. This node depends on any assets required by the stack (rather
than plumbing the assets through to each of the stack's children).
@pgavlin pgavlin requested a review from viveklak April 22, 2022 06:22
@pgavlin
Copy link
Member Author

pgavlin commented Apr 22, 2022

With these changes, the s3-object-lambda test should pass.

@pgavlin
Copy link
Member Author

pgavlin commented Apr 22, 2022

As it turns out, this is blocked on an AWS Native bug: pulumi/pulumi-aws-native#466

@pgavlin
Copy link
Member Author

pgavlin commented Apr 25, 2022

As it turns out, this is blocked on an AWS Native bug: pulumi/pulumi-aws-native#466

Just to clarify: the test that I was hoping to enable is blocked on a release with that bug fix, but the code changes here are not.

@viveklak
Copy link
Contributor

As it turns out, this is blocked on an AWS Native bug: pulumi/pulumi-aws-native#466

Just to clarify: the test that I was hoping to enable is blocked on a release with that bug fix, but the code changes here are not.

v0.16.0 has just been released.

src/stack.ts Outdated Show resolved Hide resolved
@pgavlin pgavlin merged commit 3fef800 into main Apr 25, 2022
@pulumi-bot pulumi-bot deleted the pgavlin/file-asset branch April 25, 2022 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants