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

Clear artifacts per function #1162

Merged
merged 1 commit into from
Dec 17, 2021
Merged

Clear artifacts per function #1162

merged 1 commit into from
Dec 17, 2021

Conversation

thdxr
Copy link
Contributor

@thdxr thdxr commented Dec 16, 2021

No description provided.

@thdxr thdxr added the bug Something isn't working label Dec 16, 2021
@thdxr thdxr merged commit aab9b1d into master Dec 17, 2021
@thdxr thdxr deleted the fix-wiping-artifacts branch December 17, 2021 01:14
@revmischa
Copy link
Contributor

A little hard to tell for me, does this mean it won't delete my lambda layer artifacts between deployments?

@thdxr
Copy link
Contributor Author

thdxr commented Dec 17, 2021

The generated layer is done through CDK and that ends up being deleted by CDK on every run so I don't think that it'll stick around

@revmischa
Copy link
Contributor

The generated layer is done through CDK and that ends up being deleted by CDK on every run so I don't think that it'll stick around

If I use CDK without SST it doesn't delete any of the assets. If my layer assetHash doesn't change then it doesn't rebuild it, if it changes or I manually delete my cdk.out directory then it rebuilds the layer.

jayair added a commit that referenced this pull request Dec 27, 2021
* updated mongodb example with serverless option

* Add a missing new line in code sample (#1110)

* Removed redundant config for Thundra integration (#1119)

* Function: allow setDefaultFunctionProps before Functions are created (#1126)

* Function: allow imported IConstructs in attachPermissions (#1125)

* Fixed bug with adding multiple topic subscribers in same stack (#1123)

* Pin version in update test (#1129)

* v0.53.4

* Fix linting

* Fix prettier

* Script create user function protected (#1128)

* Console improvements (#1130)

* Add Function to stack metadata

* Invoke Function

* Invoke EventBridge

* Do not show Functions used in other constructs

* Refactor PayloadForm

* Show full REQUEST/RESPONSE payload

* Fix lint

* Doc: document `Auth.attachPermissionsForTriggers`

* Refactor building + watcher system (#1044)

* v0.54.0

* cli: respect region in local aws config if region is not provided (#1143)

* cli: respect aws cli region if region is not provided

* Remove AWS_SDK_LOAD_CONFIG

* Queue: cannot call addConsumer multiple times from the same scope (#1146)

* Fix crash when function not found (#1149)

* Fixed empty functions file (#1148)

* Fix debug bucket (#1150)

* Fix zipped payload issue

* v0.54.1

* Fix issue with python bundling (#1152)

* v0.54.2

* Log error from sync build

* Do not watch duplicate paths + fix stuck state machine when stack build fails

* Remove Watcher dependency on Handler

* Update include property into ts.config (#1158)

* Block public access for DebugStack S3 bucket (#1092)

* Clear artifacts per function (#1162)

* Various performance fixes (#1164)

* v0.54.3

* Remove optional chaining

* Fix a typo in CONTRIBUTING.md (#1171)

* Function: functionName takes callback (#1172)

* Bucket: allow multiple addNotifications calls within the same scope (#1173)

* Fix esbuild treating warnings as errors (#1167)

* v0.54.4

* Fix reload functionBuilder (#1178)

* Updating new DocSearch config

* Update Queue.md

* Update Queue.md

* Editing

Co-authored-by: Jakob Fix <170240+jfix@users.noreply.github.com>
Co-authored-by: Serkan ÖZAL <serkan@thundra.io>
Co-authored-by: Frank <wangfanjie@gmail.com>
Co-authored-by: Dax Raad <d@ironbay.co>
Co-authored-by: Mischa Spiegelmock <me@mish.dev>
Co-authored-by: Davide Ungari <ungarida@gmail.com>
Co-authored-by: Akos Krivachy <krivachy@users.noreply.github.com>
Co-authored-by: Dmitriy Fishman <fishman.code@gmail.com>
Co-authored-by: Jay <jvelayud@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants