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

[Response Ops] Onweek APM WIP #129901

Closed
wants to merge 18 commits into from
Closed

Conversation

chrisronline
Copy link
Contributor

@chrisronline chrisronline commented Apr 11, 2022

@kibana-ci
Copy link
Collaborator

kibana-ci commented Apr 26, 2022

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Default CI Group #12 / alerting api integration spaces only Alerting inMemoryMetrics should count executions
  • [job] [logs] Default CI Group #12 / alerting api integration spaces only Alerting inMemoryMetrics should count executions
  • [job] [logs] Jest Integration Tests #2 / Fleet setup preconfiguration with multiple instances Kibana preconfiguration setup sets up Fleet correctly when multiple Kibana instaces are started in serial
  • [job] [logs] Jest Integration Tests #2 / Fleet setup preconfiguration with multiple instances Kibana preconfiguration setup sets up Fleet correctly when multiple Kibana instances are started at the same time
  • [job] [logs] Jest Integration Tests #2 / Fleet setup preconfiguration with multiple instances Kibana preconfiguration setup sets up Fleet correctly with single Kibana instance
  • [job] [logs] Jest Integration Tests #1 / migration from 7.13 to 7.14+ with many failed action_tasks filters out all outdated action_task_params and action tasks
  • [job] [logs] Jest Integration Tests #1 / migration v2 completes the migration even when a full batch would exceed ES http.max_content_length
  • [job] [logs] Jest Integration Tests #1 / migration v2 fails with a descriptive message when a single document exceeds maxBatchSizeBytes

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
triggersActionsUi 360 362 +2

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
alerting 316 325 +9
monitoring 9 11 +2
monitoringCollection 9 21 +12
total +23

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
triggersActionsUi 711.9KB 714.0KB +2.1KB

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
alerting 19 20 +1
monitoringCollection 0 1 +1
total +2

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
triggersActionsUi 99.4KB 99.5KB +121.0B
Unknown metric groups

API count

id before after diff
alerting 325 334 +9
monitoring 11 13 +2
monitoringCollection 9 21 +12
total +23

async chunk count

id before after diff
triggersActionsUi 77 78 +1

ESLint disabled in files

id before after diff
apm 15 14 -1
securitySolution 69 68 -1
synthetics 7 6 -1
total -3

ESLint disabled line counts

id before after diff
apm 92 89 -3
enterpriseSearch 9 7 -2
monitoringCollection 2 1 -1
synthetics 53 47 -6
triggersActionsUi 167 168 +1
total -11

Total ESLint disabled count

id before after diff
apm 107 103 -4
enterpriseSearch 9 7 -2
monitoringCollection 2 1 -1
securitySolution 511 510 -1
synthetics 60 53 -7
triggersActionsUi 173 174 +1
total -14

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@legrego legrego closed this Jul 17, 2024
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.

3 participants