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

[DataUsage][Serverless] Handle usage metrics errors #197056

Conversation

ashokaditya
Copy link
Member

@ashokaditya ashokaditya commented Oct 21, 2024

Summary

Checklist

@ashokaditya ashokaditya added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 v8.16.0 v8.17.0 labels Oct 21, 2024
@ashokaditya ashokaditya self-assigned this Oct 21, 2024
@ashokaditya ashokaditya force-pushed the task/serverless-data-usage-tests-ux-updates-192966 branch 2 times, most recently from 1642730 to 7c17b6b Compare October 23, 2024 06:50
@ashokaditya ashokaditya force-pushed the task/serverless-data-usage-tests-ux-updates-192966 branch from 7c17b6b to 7ca637b Compare October 24, 2024 11:31
@ashokaditya ashokaditya changed the title [DataUsage][Serverless] Usage metrics improvements/tests [DataUsage][Serverless] Usage metrics improvements Oct 24, 2024
@ashokaditya ashokaditya changed the title [DataUsage][Serverless] Usage metrics improvements [DataUsage][Serverless] Handle usage metrics errors Oct 24, 2024
@ashokaditya ashokaditya marked this pull request as ready for review October 24, 2024 11:32
@ashokaditya ashokaditya requested review from a team as code owners October 24, 2024 11:32
@ashokaditya ashokaditya added the backport:version Backport to applied version labels label Oct 24, 2024
Copy link
Contributor

@neptunian neptunian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we not backport to 8.16?

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
dataUsage 237.6KB 237.6KB +9.0B

cc @ashokaditya

@ashokaditya ashokaditya merged commit 1267bd7 into elastic:main Oct 24, 2024
27 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11500688157

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 24, 2024
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 24, 2024
…197647)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[DataUsage][Serverless] Handle usage metrics errors
(#197056)](#197056)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Ash","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-24T13:48:33Z","message":"[DataUsage][Serverless]
Handle usage metrics errors
(#197056)","sha":"1267bd7129912690d469ae6d359c8242a679dfb8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.17.0"],"title":"[DataUsage][Serverless]
Handle usage metrics
errors","number":197056,"url":"https://github.com/elastic/kibana/pull/197056","mergeCommit":{"message":"[DataUsage][Serverless]
Handle usage metrics errors
(#197056)","sha":"1267bd7129912690d469ae6d359c8242a679dfb8"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/197056","number":197056,"mergeCommit":{"message":"[DataUsage][Serverless]
Handle usage metrics errors
(#197056)","sha":"1267bd7129912690d469ae6d359c8242a679dfb8"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Ash <[email protected]>
@ashokaditya ashokaditya deleted the task/serverless-data-usage-tests-ux-updates-192966 branch October 25, 2024 07:07
ashokaditya added a commit that referenced this pull request Nov 7, 2024
## Summary

Adds missing UX/API tests for changes added in 
- #193966
- #197056
- #195556
- #196559


### Checklist
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

---------

Co-authored-by: kibanamachine <[email protected]>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 7, 2024
## Summary

Adds missing UX/API tests for changes added in
- elastic#193966
- elastic#197056
- elastic#195556
- elastic#196559

### Checklist
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 207894e)
mbondyra pushed a commit to mbondyra/kibana that referenced this pull request Nov 8, 2024
## Summary

Adds missing UX/API tests for changes added in 
- elastic#193966
- elastic#197056
- elastic#195556
- elastic#196559


### Checklist
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

---------

Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Nov 8, 2024
## Summary

Adds missing UX/API tests for changes added in 
- elastic#193966
- elastic#197056
- elastic#195556
- elastic#196559


### Checklist
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed

---------

Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants