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

[ResponseOps][Rules] Move Rule Form code into @kbn/response-ops-rule-form #201729

Closed

Conversation

maryam-saeidi
Copy link
Member

Summary

Testing CI for #198725

Zacqary and others added 30 commits October 4, 2024 12:18
# Conflicts:
#	x-pack/plugins/security_solution/public/detection_engine/rule_management/api/api.ts
kibanamachine and others added 10 commits November 20, 2024 18:29
…/kibana into 195211-rule-form-package

# Conflicts:
#	x-pack/plugins/fleet/cypress.config.space_awareness.d.ts
#	x-pack/plugins/fleet/cypress.config.space_awareness.js
…ckage

# Conflicts:
#	packages/response-ops/rule_form/src/rule_actions/rule_actions_system_actions_item.tsx
#	x-pack/plugins/observability_solution/observability/tsconfig.json
#	x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/system_action_type_form.tsx
@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 26, 2024

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!

@maryam-saeidi maryam-saeidi force-pushed the 195211-rule-form-package branch from aa0afdd to da4012a Compare November 26, 2024 14:50
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7491

[❌] x-pack/test/observability_functional/with_rac_write.config.ts: 0/1 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7493

[❌] x-pack/test/functional/apps/dashboard/group1/config.ts: 0/1 tests passed.
[❌] x-pack/test/observability_functional/with_rac_write.config.ts: 0/1 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7500

[❌] x-pack/test/observability_functional/with_rac_write.config.ts: 0/3 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7506

[❌] x-pack/test/observability_functional/with_rac_write.config.ts: 0/1 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#7507

[✅] x-pack/test/observability_functional/with_rac_write.config.ts: 3/3 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#7513

[❌] x-pack/test/observability_functional/with_rac_write.config.ts: 0/3 tests passed.

see run history

@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 27, 2024

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #79 / APM specs correlations failed transactions correlations space with no features disabled navigates to the alerts tab
  • [job] [logs] FTR Configs #79 / APM specs correlations failed transactions correlations space with no features disabled navigates to the alerts tab
  • [job] [logs] FTR Configs #102 / ObservabilityApp Observability alerts > Alerts table Filtering Autocompletion works
  • [job] [logs] FTR Configs #102 / ObservabilityApp Observability alerts > Alerts table Filtering Autocompletion works
  • [job] [logs] Jest Tests #15 / useColumns check if initial width for the last column does not exist
  • [job] [logs] Jest Tests #15 / useColumns check if initial width for the last column does not exist
  • [job] [logs] Jest Tests #15 / useColumns columns should changes the column list when defaultColumns has been updated
  • [job] [logs] Jest Tests #15 / useColumns columns should changes the column list when defaultColumns has been updated
  • [job] [logs] Jest Tests #15 / useColumns does not fetch alerts fields if they're overridden through the alertsFields prop
  • [job] [logs] Jest Tests #15 / useColumns does not fetch alerts fields if they're overridden through the alertsFields prop
  • [job] [logs] Jest Tests #15 / useColumns onColumnResize
  • [job] [logs] Jest Tests #15 / useColumns onColumnResize
  • [job] [logs] Jest Tests #15 / useColumns onResetColumns should restore visible columns defaults
  • [job] [logs] Jest Tests #15 / useColumns onResetColumns should restore visible columns defaults
  • [job] [logs] Jest Tests #15 / useColumns onToggleColumns should update the column details in the storage when onToggleColumn is called
  • [job] [logs] Jest Tests #15 / useColumns onToggleColumns should update the column details in the storage when onToggleColumn is called
  • [job] [logs] Jest Tests #15 / useColumns onToggleColumns should update the list of columns when on Toggle Columns is called
  • [job] [logs] Jest Tests #15 / useColumns onToggleColumns should update the list of columns when on Toggle Columns is called
  • [job] [logs] Jest Tests #15 / useColumns onToggleColumns should update the list of visible columns when onToggleColumn is called
  • [job] [logs] Jest Tests #15 / useColumns onToggleColumns should update the list of visible columns when onToggleColumn is called
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns hide all columns with onChangeVisibleColumns
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns hide all columns with onChangeVisibleColumns
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns should change visibleColumns if provided defaultColumns change
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns should change visibleColumns if provided defaultColumns change
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns should populate visibleColumns correctly
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns should populate visibleColumns correctly
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns show all columns with onChangeVisibleColumns
  • [job] [logs] Jest Tests #15 / useColumns visibleColumns show all columns with onChangeVisibleColumns

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
alerting 248 246 -2
observability 1074 1072 -2
securitySolution 6259 6257 -2
triggersActionsUi 862 870 +8
total +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
@kbn/alerts-ui-shared 304 296 -8
@kbn/response-ops-rule-form - 156 +156
@kbn/triggers-actions-ui-types 15 17 +2
total +150

Async chunks

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

id before after diff
alerting 94.1KB 102.8KB +8.7KB
cases 492.7KB 492.9KB +257.0B
observability 472.7KB 480.8KB +8.2KB
securitySolution 13.3MB 13.4MB +8.7KB
triggersActionsUi 1.7MB 1.7MB +15.0KB
total +40.8KB

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
@kbn/alerts-ui-shared 8 4 -4
@kbn/response-ops-rule-form - 7 +7
total +3

Page load bundle

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

id before after diff
alerting 25.6KB 25.6KB +56.0B
triggersActionsUi 128.8KB 127.6KB -1.2KB
total -1.2KB
Unknown metric groups

API count

id before after diff
@kbn/alerts-ui-shared 320 312 -8
@kbn/response-ops-rule-form - 157 +157
@kbn/triggers-actions-ui-types 15 17 +2
total +151

ESLint disabled line counts

id before after diff
@kbn/alerts-ui-shared 8 6 -2
@kbn/response-ops-rule-form - 4 +4
ruleRegistry 2 3 +1
triggersActionsUi 131 132 +1
total +4

References to deprecated APIs

id before after diff
@kbn/alerts-ui-shared 8 0 -8
@kbn/response-ops-rule-form - 8 +8
total -0

Total ESLint disabled count

id before after diff
@kbn/alerts-ui-shared 9 7 -2
@kbn/response-ops-rule-form - 4 +4
ruleRegistry 4 5 +1
triggersActionsUi 137 138 +1
total +4

History

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.

7 participants