-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Next.js] Mark Cold Start When Deployed To Vercel #7925
Comments
Generally, this would work by having a global variable (e.g. I think this is a great idea but I still have my doubts about whether we can reliably do this from within the SDK:
Anyhow I think this would be a banger blog post. |
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
Not sure if relevant, but this was recently implemented for the aws-lambda instrumentation here: open-telemetry/opentelemetry-js-contrib#2403 maybe we can find inspiration there. cc @lforst / @chargome |
Looks interesting, I guess we could port this implementation to our instrumentation as well if we can access |
Problem Statement
Customers would like to be able to differentiate events based on whether or not they are associated with a cold-start of a severless function.
The initial request specifically mentions Next.js when deployed to vercel.
Solution Brainstorm
Identifying whether or not the event context is from a cold-start using a tag may be very helpful.
Examples:
cc @lforst
The text was updated successfully, but these errors were encountered: