feat(slo): group by instance id

[kdelemme]

Resolves #186125

Summary

This PR adds the group by "SLO instance id" option.
I included the slo.groupings flattened field on the group summary response, so we can format the group name in a more user friendly way than just showing the slo.instanceId field which is a coma separated string without the group keys.

[obltmachine]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

• /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
• run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

[kdelemme]

/ci

[elasticmachine]

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

[mgiota]

LGTM! @kdelemme For some reason when I create an SLO with group by I get 0 SLO instances created. While I'm looking into this, could you make sure that the group by instance id option is added to the Group Overview Embeddable as well?

I know it's not good how it is implemented at the moment, but we need to manually add the group by options in the embeddable as well. You can check how it was done when we added the Group by remote option as part of this issue, or feel free to refactor, so that it automatically picks up the group by options in the embeddable.

[mgiota]

@kdelemme I was able to generate some group by SLOs and grouping by instance id works well

Shall we add the group by instance id in the Group Overview embeddable in a follow up PR?

[kibana-ci]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: 7df68b2
• Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-186131-7df68b284fc6
• Observability Deployment

Failed CI Steps

• FTR Configs #14

Test Failures

• [job] [logs] FTR Configs #14 / Cases Attachment framework Attachment hooks Modal filters with multiple selection

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
slo	854	855	+1

Async chunks

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

id	before	after	diff
slo	862.6KB	863.3KB	+777.0B

Page load bundle

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

id	before	after	diff
observability	123.0KB	123.1KB	+71.0B

History

• 💛 Build #216117 was flaky 8f1ca83
• 💛 Build #215871 was flaky 8aff591
• 💔 Build #215791 failed 13fa56b

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

cc @kdelemme

[mgiota]

LGTM! As discussed I approve this PR and we can handle the group by instance id in the Group embeddable in a follow up PR.