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

Migrate Alerting plugins (including Task Manager and Event Log) to TS project references #81052

Closed
2 of 6 tasks
YulNaumenko opened this issue Oct 19, 2020 · 3 comments · Fixed by #81557 or #88556
Closed
2 of 6 tasks
Assignees
Labels
blocked Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@YulNaumenko
Copy link
Contributor

YulNaumenko commented Oct 19, 2020

Summary

Now that the Kibana build system supports using TypeScript project references (#46773), the infra plugin must be migrated to participate. This is part of the overall effort tracked at #80508.

Plugins to migrate:

  • alerts
  • actions
  • event_log
  • task_manager
  • triggers_actions_ui
  • stack_alerts

Blockers

Plugin dependencies blocking full migration:
task_manager depends on migration of the following first:
apm (#81003)

event_log depends on migration of the following first:
home

actions and alerts plugins depends on migration of the following first:
security_oss
home

Acceptance criteria

Each of the listed plugins can be referred to using a project reference, which means it...

  • has a tsconfig.json file which conforms to the documented form
  • references the dependencies
  • emits type definitions for its public API
  • compiles in tsc's build mode

Related information

@YulNaumenko YulNaumenko added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Project:Alerting labels Oct 19, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@smith
Copy link
Contributor

smith commented Jan 12, 2021

Plugin dependencies blocking full migration:
task_manager depends on migration of the following first:
apm (#81003)

This was fixed in #83081 and is no longer a blocker.

@YulNaumenko
Copy link
Contributor Author

event_log migration still blocked by migrating of spaces plugin.

@YulNaumenko YulNaumenko reopened this Jan 16, 2021
@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment