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(aws-lambda): lazily initialize aws library #11614

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Conversation

bungle
Copy link
Member

@bungle bungle commented Sep 19, 2023

Summary

Lazily initializes AWS library on a first use, to remove startup delay caused by AWS metadata discovery.

Checklist

  • The Pull Request has tests
  • A changelog file has been added to CHANGELOG/unreleased/kong or adding skip-changelog label on PR if unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

^ this PR does not need any of those, this is on top of unreleased feature.

### Summary

Lazily initializes AWS library on a first use, to remove startup delay caused
by AWS metadata discovery.

Signed-off-by: Aapo Talvensaari <[email protected]>
@chronolaw
Copy link
Contributor

Why don't we need a change log?

@bungle
Copy link
Member Author

bungle commented Sep 20, 2023

Why don't we need a change log?

Because it is fix over to upcoming unreleased feature THAT already has a changelog,
see: #11551

So you can think that there was nothing to fix if this was originally in the original PR. It would be strange if changelog says that we 1) added new feature and 2) fixed it. We can say that we added new feature and be done with it (we didn't fix anything for anyone as nobody is using new feature as it is not released - the audience of changelogs is our users, not us - at least the primary audience).

@hanshuebner hanshuebner merged commit 94fbf7e into master Sep 20, 2023
31 of 32 checks passed
@hanshuebner hanshuebner deleted the fix/aws-lazy-load branch September 20, 2023 08:18
windmgc pushed a commit that referenced this pull request Nov 2, 2023
### Summary

Lazily initializes AWS library on a first use, to remove startup delay caused
by AWS metadata discovery.

Signed-off-by: Aapo Talvensaari <[email protected]>
windmgc pushed a commit that referenced this pull request Nov 3, 2023
### Summary

Lazily initializes AWS library on a first use, to remove startup delay caused
by AWS metadata discovery.

Signed-off-by: Aapo Talvensaari <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants