-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
APM UI changes for serverless services / AWS lambda #122775
Conversation
@elasticmachine merge upstream |
5330d2f
to
e34bd5b
Compare
6d14201
to
53a3bf9
Compare
@formgeist shall we add the |
Yes, I think it makes sense similar to the
I think the only place where I think it has the most impact is when the user clicks on the Lambda service icon and looks at the content. So adding a |
@formgeist thanks, I'll get those added |
Pinging @elastic/apm-ui (Team:apm) |
@elasticmachine merge upstream |
@gbamparop - do you know why docker icon is displayed next to lambda icon on some of the screenshots? |
I'd guess it's because the data comes from Synthtrace which in addition to the Lambda fields also generated some docker fields. Shouldn't happen with real data. @gbamparop correct me if I'm wrong. |
On a related note, has anyone opened an issue for adding a Lambda service to our fleet of Opbeans test data for continued testing? |
This is it, I've put as much data as I could in synthrace to help write some tests. |
As discussed we decided to go with the current version for now as there can be several functions with large names and the popup can get quite large. |
💚 Build SucceededMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: cc @gbamparop |
💔 Backport failedThe pull request could not be backported due to the following error: How to fixRe-run the backport manually:
Questions ?Please refer to the Backport tool documentation |
@gbamparop do you think the scenario aws_lambda_with_multiple_functions.ts.zip would be useful as an example in https://github.com/gbamparop/kibana/tree/aws-lambda/packages/elastic-apm-synthtrace/src/scripts/examples ? |
I think that the intention was to just have a few generic examples as a base point in that folder so didn't add that in. Happy to get it added if this is what we aim for. @dgieselaar what do you think? |
Closes #106552
Requires elastic/apm-server#6576 to properly work with metric documents.
Summary
Some APM agents now support AWS lambda instrumentations. In this context some of the data is not available while some additional, serverless-specific information is available that is very valuable for APM use cases. This PR contains UI changes to reflect these aspects for serverless services.
With AWS lambda currently all kind of metric collection are disabled at the agents. Thus, metrics for system, JVM, transaction breakdown are not available.
Screen.Recording.2022-01-28.at.12.18.52.mov
Changes
Added meta-information for AWS lambda services in the service-icons panel
Added
cloud.service.name
andcloud.region
(before only availability zone was shown) as additional info being shown in cloud metainfo sectionReplaced transaction breakdown chart with cold start rate chart for serverless services
In service overview
In transactions overview
In transaction details
Added
cold start
badge to waterfall transaction items (when they represent cold start requests)Added
cold start
badge to transaction flyout (when it represents cold start requests)Hidden metrics-related views (for which metrics are not available)
Testing
Synthtrace
Run the following scenario with synthrace CLI
aws_lambda_with_multiple_functions.ts.zip
Cloud
@gbamparop can help to set this up.
Todo