Skip to content

[7.x] [RAC] Get o11y alerts in alerts table (#109346)#109705

Merged
weltenwort merged 1 commit intoelastic:7.xfrom
weltenwort:backport/7.x/pr-109346
Aug 23, 2021
Merged

[7.x] [RAC] Get o11y alerts in alerts table (#109346)#109705
weltenwort merged 1 commit intoelastic:7.xfrom
weltenwort:backport/7.x/pr-109346

Conversation

@weltenwort
Copy link
Member

Backports the following commits to 7.x:

* get back index names  in o11y

* testing and integration

* fix types

* Avoid using the rule data client for field list

* Remove left-over index argument

* no needs of alert consumer anymore

Co-authored-by: Felix Stürmer <stuermer@weltenwort.de>
# Conflicts:
#	x-pack/plugins/observability/public/pages/alerts/alerts_table_t_grid.tsx
#	x-pack/plugins/observability/public/pages/alerts/index.tsx
@weltenwort weltenwort added the backport This PR is a backport of another PR label Aug 23, 2021
@weltenwort weltenwort enabled auto-merge (squash) August 23, 2021 17:41
@weltenwort weltenwort self-assigned this Aug 23, 2021
@weltenwort
Copy link
Member Author

This is an attempt to resolve the 7.x backport deadlock by changing the merge order and manually resolving the conflicts.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observability 256 259 +3

Async chunks

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

id before after diff
cases 582.8KB 582.6KB -204.0B
observability 489.6KB 492.1KB +2.6KB
securitySolution 6.5MB 6.5MB -748.0B
timelines 437.1KB 437.0KB -60.0B
total +1.6KB
Unknown metric groups

API count

id before after diff
ruleRegistry 134 136 +2
timelines 960 959 -1
total +1

API count missing comments

id before after diff
ruleRegistry 113 114 +1
timelines 840 839 -1
total -0

Non-exported public API item count

id before after diff
ruleRegistry 6 7 +1

References to deprecated APIs

id before after diff
observability 27 23 -4

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

cc @weltenwort

@weltenwort weltenwort merged commit c788308 into elastic:7.x Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants