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

Failing test: Chrome X-Pack UI Functional Tests.x-pack/test/functional/apps/ml/data_frame_analytics/feature_importance·ts - machine learning data frame analytics total feature importance panel and decision path popover binary classification job should display the total feature importance in the results view #115415

Closed
kibanamachine opened this issue Oct 18, 2021 · 13 comments · Fixed by #115592
Assignees
Labels
failed-test A test failure on a tracked branch, potentially flaky-test :ml

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Oct 18, 2021

A test failed on a tracked branch

Error: expected testSubject(mlTotalFeatureImportanceChart) to exist
    at TestSubjects.existOrFail (/opt/local-ssd/buildkite/builds/kb-cigroup-6-f6ff7d7b1f849a64/elastic/kibana-hourly/kibana/test/functional/services/common/test_subjects.ts:45:13)
    at Object.assertTotalFeatureImportanceEvaluatePanelExists (test/functional/services/ml/data_frame_analytics_results.ts:76:7)
    at Context.<anonymous> (test/functional/apps/ml/data_frame_analytics/feature_importance.ts:201:11)
    at Object.apply (/opt/local-ssd/buildkite/builds/kb-cigroup-6-f6ff7d7b1f849a64/elastic/kibana-hourly/kibana/node_modules/@kbn/test/target_node/functional_test_runner/lib/mocha/wrap_function.js:87:16)

First failure: CI Build - master

@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Oct 18, 2021
@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 18, 2021
@pheyos pheyos self-assigned this Oct 19, 2021
@pheyos pheyos added the :ml label Oct 19, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - 7.16

@spalger
Copy link
Contributor

spalger commented Nov 3, 2021

Looks like the fix made it back to 7.16, first failure in 14 days since the fix, but probably worth taking a look @pheyos

@pheyos
Copy link
Member

pheyos commented Nov 4, 2021

The failure screenshot shows a different area of the page, so it's not helpful here:
image

I'll take a closer look - maybe we need to fold the unused sections or scroll the current section into view, such that failure screenshots are showing relevant details.

@pheyos
Copy link
Member

pheyos commented Nov 4, 2021

With #117503 we're scrolling the feature influence section into view, so the failure screenshot shows the relevant part of the screen and hopefully gives us more details. But maybe the job config change in this PR already fixed this issue. We'll monitor this.

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - main

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - 8.1

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - 8.1

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - 7.17

@kibanamachine
Copy link
Contributor Author

New failure: CI Build - 8.1

@jbudz
Copy link
Member

jbudz commented Feb 17, 2022

/skip

@kibanamachine
Copy link
Contributor Author

Skipped

8.1: 3509301
7.17: aca3d03

@pheyos
Copy link
Member

pheyos commented Apr 27, 2022

The tests have moved to a different location. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
failed-test A test failure on a tracked branch, potentially flaky-test :ml
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants