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

[Alerting] Remove legacy ES client usage in the alerting functional tests #95332

Closed
YulNaumenko opened this issue Mar 24, 2021 · 3 comments · Fixed by #102349
Closed

[Alerting] Remove legacy ES client usage in the alerting functional tests #95332

YulNaumenko opened this issue Mar 24, 2021 · 3 comments · Fixed by #102349
Assignees
Labels
Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@YulNaumenko
Copy link
Contributor

Functional tests
x-pack/test/alerting_api_integration/security_and_spaces/tests/actions/builtin_action_types/es_index.ts:16:
x-pack/test/alerting_api_integration/security_and_spaces/tests/actions/builtin_action_types/es_index_preconfigured.ts:18:
x-pack/test/alerting_api_integration/security_and_spaces/tests/actions/execute.ts:26:
x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/alerts.ts:29:
x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/create.ts:23:
x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/delete.ts:22:
x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/disable.ts:23:
x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/enable.ts:23:
x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/rbac_legacy.ts:17:
x-pack/test/alerting_api_integration/spaces_only/tests/actions/builtin_action_types/es_index.ts:16:
x-pack/test/alerting_api_integration/spaces_only/tests/actions/execute.ts:25:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/alerts_base.ts:26:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/builtin_alert_types/es_query/alert.ts:35:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/builtin_alert_types/index_threshold/alert.ts:34:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/builtin_alert_types/index_threshold/fields_endpoint.ts:20:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/builtin_alert_types/index_threshold/indices_endpoint.ts:21:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/builtin_alert_types/index_threshold/time_series_query_endpoint.ts:54:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/create.ts:22:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/delete.ts:16:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/disable.ts:21:
x-pack/test/alerting_api_integration/spaces_only/tests/alerting/enable.ts:21:
x-pack/test/plugin_api_integration/test_suites/event_log/service_api_integration.ts:14:

@YulNaumenko YulNaumenko added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Mar 24, 2021
@elasticmachine
Copy link
Contributor

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

@mikecote
Copy link
Contributor

This needs to be done by 7.15 feature freeze. If we can't, we should notify the core team (#83910 (comment)).

@mikecote
Copy link
Contributor

We should check event log and task manager as well.

@ymao1 ymao1 self-assigned this Jun 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
Labels
Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants