Skip to content

Conversation

@darnautov
Copy link
Contributor

@darnautov darnautov commented Oct 9, 2024

Summary

Removes assertions for success toast messages that were deleted in #194614

Checklist

@darnautov darnautov requested a review from a team as a code owner October 9, 2024 10:30
@darnautov darnautov self-assigned this Oct 9, 2024
@darnautov darnautov added :ml release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:ML Team label for ML (also use :ml) t// v8.16.0 backport:version Backport to applied version labels labels Oct 9, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #29 / machine learning - anomaly detection anomaly explorer with influencers with farequote based multi metric job renders swim lanes correctly on the time bounds change

Metrics [docs]

✅ unchanged

cc @darnautov

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 9, 2024
## Summary

Removes assertions for success toast messages that were deleted in
elastic#194614

### 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

(cherry picked from commit a406acb)
@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

@darnautov darnautov deleted the ml-fix-trained-models-toast-assertion branch October 9, 2024 12:31
@kibanamachine kibanamachine mentioned this pull request Oct 9, 2024
2 tasks
kibanamachine added a commit that referenced this pull request Oct 9, 2024
…5587)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ML] Fix assertion in the Trained Model UI tests
(#195566)](#195566)

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

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

<!--BACKPORT [{"author":{"name":"Dima
Arnautov","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-09T12:24:31Z","message":"[ML]
Fix assertion in the Trained Model UI tests (#195566)\n\n##
Summary\r\n\r\nRemoves assertions for success toast messages that were
deleted
in\r\nhttps://github.com//pull/194614\r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"a406acba31ed05588ae21eed5d380722e32fbbb0","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":[":ml","release_note:skip","v9.0.0","Team:ML","v8.16.0","backport:version"],"title":"[ML]
Fix assertion in the Trained Model UI tests
","number":195566,"url":"https://github.com/elastic/kibana/pull/195566","mergeCommit":{"message":"[ML]
Fix assertion in the Trained Model UI tests (#195566)\n\n##
Summary\r\n\r\nRemoves assertions for success toast messages that were
deleted
in\r\nhttps://github.com//pull/194614\r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"a406acba31ed05588ae21eed5d380722e32fbbb0"}},"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/195566","number":195566,"mergeCommit":{"message":"[ML]
Fix assertion in the Trained Model UI tests (#195566)\n\n##
Summary\r\n\r\nRemoves assertions for success toast messages that were
deleted
in\r\nhttps://github.com//pull/194614\r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"a406acba31ed05588ae21eed5d380722e32fbbb0"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Dima Arnautov <[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 :ml release_note:skip Skip the PR/issue when compiling release notes Team:ML Team label for ML (also use :ml) t// v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants