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

fix(lambda-at-edge): static regeneration while running in ISR mode #1729 #1731

Conversation

cmaerz
Copy link
Contributor

@cmaerz cmaerz commented Sep 22, 2021

No description provided.

@cmaerz cmaerz changed the title fix(lambda-at-edge): static regeneration while running in ISR mode fix(lambda-at-edge): static regeneration while running in ISR mode #1729 Sep 22, 2021
@dphang
Copy link
Collaborator

dphang commented Sep 23, 2021

Looks like some unit tests failed

@cmaerz
Copy link
Contributor Author

cmaerz commented Sep 23, 2021

@dphang Yeah.. I can‘t get the build running currently. See: #1729
Maybe you can help me there, than I can fix the tests :D

@dphang
Copy link
Collaborator

dphang commented Sep 23, 2021

Hm, are archiver types added properly? You can also try to delete the yarn lock file temporarily and regenerate it, sometimes it helped me when dependencies are having issues

@cmaerz
Copy link
Contributor Author

cmaerz commented Sep 23, 2021

Hm.. deleting yarn.lock didn't help. Tbh I don't even know what archiver is, I didn't add that package :D

@slsnextbot
Copy link
Collaborator

Handler Size Report

There are changes to handler sizes. Please review.

Base Handler Sizes (kB) (commit 2612f08)

{
    "Lambda@Edge": {
        "Default Lambda": {
            "Standard": 1560,
            "Minified": 550
        },
        "API Lambda": {
            "Standard": 101,
            "Minified": 44
        },
        "Image Lambda": {
            "Standard": 933,
            "Minified": 371
        },
        "Regeneration Lambda": {
            "Standard": 655,
            "Minified": 237
        }
    }
}

New Handler Sizes (kB) (commit 0f5cc5d)

{
    "Lambda@Edge": {
        "Default Lambda": {
            "Standard": 1290,
            "Minified": 461
        },
        "API Lambda": {
            "Standard": 101,
            "Minified": 44
        },
        "Image Lambda": {
            "Standard": 943,
            "Minified": 375
        },
        "Regeneration Lambda": {
            "Standard": 665,
            "Minified": 242
        }
    }
}

@codecov
Copy link

codecov bot commented Sep 23, 2021

Codecov Report

Merging #1731 (b25e18c) into master (2612f08) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head b25e18c differs from pull request most recent head 0f5cc5d. Consider uploading reports for the commit 0f5cc5d to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1731   +/-   ##
=======================================
  Coverage   83.85%   83.86%           
=======================================
  Files          98       98           
  Lines        3494     3496    +2     
  Branches     1048     1048           
=======================================
+ Hits         2930     2932    +2     
  Misses        500      500           
  Partials       64       64           
Impacted Files Coverage Δ
...ackages/libs/lambda-at-edge/src/default-handler.ts 87.00% <ø> (ø)
...ambda-at-edge/src/lib/triggerStaticRegeneration.ts 95.65% <ø> (ø)
...es/libs/lambda-at-edge/src/regeneration-handler.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2612f08...0f5cc5d. Read the comment docs.

@dphang
Copy link
Collaborator

dphang commented Sep 24, 2021

@dphang dphang merged commit b32c0e8 into serverless-nextjs:master Sep 24, 2021
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

3 participants