Skip to content

[APM] Synchronous Anomaly detection jobs creation#145969

Merged
yngrdyn merged 9 commits intoelastic:mainfrom
yngrdyn:create-anomaly-detection-jobs-synchronously
Dec 13, 2022
Merged

[APM] Synchronous Anomaly detection jobs creation#145969
yngrdyn merged 9 commits intoelastic:mainfrom
yngrdyn:create-anomaly-detection-jobs-synchronously

Conversation

@yngrdyn
Copy link
Contributor

@yngrdyn yngrdyn commented Nov 22, 2022

closes #145781.

When creating an anomaly job we are now waiting for yellow status in the cluster.
Additionally, job creation is synchronous now.

@yngrdyn yngrdyn requested a review from a team November 22, 2022 10:46
@botelastic botelastic bot added the Team:APM - DEPRECATED Use Team:obs-ux-infra_services. label Nov 22, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:APM)

@yngrdyn yngrdyn assigned yngrdyn and unassigned yngrdyn Nov 22, 2022
@yngrdyn yngrdyn added backport:skip This PR does not require backporting v8.7.0 release_note:fix labels Nov 22, 2022
@yngrdyn yngrdyn changed the title [APM] Create anomaly detection jobs synchronously [APM] Anomaly detection jobs creation synchronously Nov 22, 2022
@yngrdyn yngrdyn changed the title [APM] Anomaly detection jobs creation synchronously [APM] Synchronous Anomaly detection jobs creation Nov 22, 2022
@yngrdyn yngrdyn force-pushed the create-anomaly-detection-jobs-synchronously branch from 570e729 to 13083b9 Compare November 22, 2022 11:00
@yngrdyn yngrdyn requested a review from sorenlouv November 28, 2022 17:59
@kibana-ci
Copy link

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 60 66 +6
osquery 109 115 +6
securitySolution 445 451 +6
total +20

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 69 75 +6
osquery 110 117 +7
securitySolution 521 527 +6
total +21

History

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

@yngrdyn yngrdyn merged commit c591b4d into elastic:main Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:fix Team:APM - DEPRECATED Use Team:obs-ux-infra_services. v8.7.0

Projects

None yet

4 participants