-
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] Limit the number of source map artifacts #144963
Conversation
Pinging @elastic/apm-ui (Team:APM) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
…-ref HEAD~1..HEAD --fix'
0a5e037
to
c040ba4
Compare
💚 Build Succeeded
Metrics [docs]Unknown metric groupsESLint disabled in files
ESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
(cherry picked from commit 654d531)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.5`: - [[APM] Limit the number of source map artifacts (#144963)](#144963) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Søren Louv-Jansen","email":"soren.louv@elastic.co"},"sourceCommit":{"committedDate":"2022-11-12T14:25:55Z","message":"[APM] Limit the number of source map artifacts (#144963)","sha":"654d531efda087b5c297d4fef4a8b9aed0a318ca","branchLabelMapping":{"^v8.6.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:APM","apm:test-plan-regression","backport:prev-minor","v8.6.0"],"number":144963,"url":"https://github.com/elastic/kibana/pull/144963","mergeCommit":{"message":"[APM] Limit the number of source map artifacts (#144963)","sha":"654d531efda087b5c297d4fef4a8b9aed0a318ca"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.6.0","labelRegex":"^v8.6.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/144963","number":144963,"mergeCommit":{"message":"[APM] Limit the number of source map artifacts (#144963)","sha":"654d531efda087b5c297d4fef4a8b9aed0a318ca"}}]}] BACKPORT--> Co-authored-by: Søren Louv-Jansen <soren.louv@elastic.co>
Changing |
@mistic - do we know if this case will indeed make it to |
Thanks @simitt for confirming the release. |
Introduced in #132799
This issue fixes a problem where
listArtifacts
will fetch all source maps artifacts without circuit breakers causing Kibana to crash for users with many source map artifacts.TODO:
GET /api/apm/sourcemaps
,'POST /api/apm/sourcemaps'
,DELETE /api/apm/sourcemaps/{id}