Skip to content

[FIX] Failing test: Chrome X-Pack Observability UI Functional Tests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts - ObservabilityApp feature controls observability security feature controls observability cases all privileges "before all" hook for "shows observability/cases navlink"#227431

Merged
fkanout merged 1 commit intoelastic:mainfrom
fkanout:155090-enable-cases-nav-link-test
Jul 11, 2025
Merged

Conversation

@fkanout
Copy link
Contributor

@fkanout fkanout commented Jul 10, 2025 •
edited by kibanamachine
Loading

Summary

It resumes/ fixes #155091
It resumes/ fixes #155090
These tests were flaky in 2023, but now, according to FTR, they are working as expected.

@fkanout fkanout requested a review from a team as a code owner July 10, 2025 11:48
@fkanout fkanout self-assigned this Jul 10, 2025
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Jul 10, 2025
@github-actions github-actions bot added the author:obs-ux-management PRs authored by the obs ux management team label Jul 10, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 10, 2025

🤖 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!)

@fkanout fkanout added Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. and removed ci:project-deploy-observability Create an Observability project author:obs-ux-management PRs authored by the obs ux management team labels Jul 10, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 10, 2025

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

@fkanout fkanout added release_note:skip Skip the PR/issue when compiling release notes v9.1.0 v9.2.0 backport:version Backport to applied version labels labels Jul 10, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 10, 2025 •
edited
Loading

💚 Build Succeeded

  • Buildkite Build
  • Commit: 25d841e
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-227431-25d841e2c092

Metrics [docs]

✅ unchanged

History

cc @fkanout

@benakansara
Copy link
Contributor

benakansara commented Jul 10, 2025

@fkanout can you please link flaky test runner results for reference

@fkanout
Copy link
Contributor Author

fkanout commented Jul 10, 2025 •
edited
Loading

@benakansara, sorry! Usually, the FTR posts it automatically in the PR.
https://buildkite.com/elastic/kibana-pull-request/builds/317686/steps/canvas
I'm working on multiple flaky tests, and it seems I mixed things up. Please hold on.

@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 10, 2025

Flaky Test Runner Stats

🟠 Some tests failed. - kibana-flaky-test-suite-runner#8593

[❌] x-pack/test/observability_functional/with_rac_write.config.ts: 43/50 tests passed.

see run history

@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 11, 2025

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#8612

[✅] x-pack/test/observability_functional/with_rac_write.config.ts: 50/50 tests passed.

see run history

@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 11, 2025

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#8619

[✅] x-pack/test/observability_functional/with_rac_write.config.ts: 50/50 tests passed.

see run history

@fkanout
Copy link
Contributor Author

fkanout commented Jul 11, 2025

@benakansara it's ok now. You have 3 FTR, the failed test in the first FTR is not the test I unskipped.
Also, to triple-check it, I ran two other FTRs.

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Jul 11, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 11, 2025

🤖 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!)

@fkanout fkanout merged commit 2008eff into elastic:main Jul 11, 2025
30 checks passed
@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 11, 2025

Starting backport for target branches: 9.1

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 11, 2025
@fkanout
[FIX] Failing test: Chrome X-Pack Observability UI Functional Tests.x…
665198c 
…-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts - ObservabilityApp feature controls observability security feature controls observability cases all privileges "before all" hook for "shows observability/cases navlink" (elastic#227431)

## Summary

It resumes/ fixes elastic#155091
It resumes/ fixes elastic#155090
These tests were flaky in 2023, but now, according to FTR, they are
working as expected.

(cherry picked from commit 2008eff)
@kibanamachine kibanamachine mentioned this pull request Jul 11, 2025
Merged
@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 11, 2025

💚 All backports created successfully

Status Branch Result
✅ 9.1

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 Jul 11, 2025
@kibanamachine @fkanout
[9.1] [FIX] Failing test: Chrome X-Pack Observability UI Functional T…
b4f4e13 
…ests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts - ObservabilityApp feature controls observability security feature controls observability cases all privileges "before all" hook for "shows observability/cases navlink" (#227431) (#227658)

# Backport

This will backport the following commits from `main` to `9.1`:
- [[FIX] Failing test: Chrome X-Pack Observability UI Functional
Tests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts
- ObservabilityApp feature controls observability security feature
controls observability cases all privileges "before all" hook for "shows
observability/cases navlink"
(#227431)](#227431)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Faisal
Kanout","email":"faisal.kanout@elastic.co"},"sourceCommit":{"committedDate":"2025-07-11T13:18:05Z","message":"[FIX]
Failing test: Chrome X-Pack Observability UI Functional
Tests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts
- ObservabilityApp feature controls observability security feature
controls observability cases all privileges \"before all\" hook for
\"shows observability/cases navlink\" (#227431)\n\n## Summary\n\nIt
resumes/ fixes #155091\nIt resumes/ fixes #155090 \nThese tests were
flaky in 2023, but now, according to FTR, they are\nworking as
expected.","sha":"2008eff8d0b0118d9e99846f5f031fbd23fcef93","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","ci:project-deploy-observability","Team:obs-ux-management","backport:version","v9.1.0","v9.2.0"],"title":"[FIX]
Failing test: Chrome X-Pack Observability UI Functional
Tests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts
- ObservabilityApp feature controls observability security feature
controls observability cases all privileges \"before all\" hook for
\"shows observability/cases
navlink\"","number":227431,"url":"https://github.com/elastic/kibana/pull/227431","mergeCommit":{"message":"[FIX]
Failing test: Chrome X-Pack Observability UI Functional
Tests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts
- ObservabilityApp feature controls observability security feature
controls observability cases all privileges \"before all\" hook for
\"shows observability/cases navlink\" (#227431)\n\n## Summary\n\nIt
resumes/ fixes #155091\nIt resumes/ fixes #155090 \nThese tests were
flaky in 2023, but now, according to FTR, they are\nworking as
expected.","sha":"2008eff8d0b0118d9e99846f5f031fbd23fcef93"}},"sourceBranch":"main","suggestedTargetBranches":["9.1"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227431","number":227431,"mergeCommit":{"message":"[FIX]
Failing test: Chrome X-Pack Observability UI Functional
Tests.x-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts
- ObservabilityApp feature controls observability security feature
controls observability cases all privileges \"before all\" hook for
\"shows observability/cases navlink\" (#227431)\n\n## Summary\n\nIt
resumes/ fixes #155091\nIt resumes/ fixes #155090 \nThese tests were
flaky in 2023, but now, according to FTR, they are\nworking as
expected.","sha":"2008eff8d0b0118d9e99846f5f031fbd23fcef93"}}]}]
BACKPORT-->

Co-authored-by: Faisal Kanout <faisal.kanout@elastic.co>
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
@fkanout @kertal
[FIX] Failing test: Chrome X-Pack Observability UI Functional Tests.x…
949af42 
…-pack/test/observability_functional/apps/observability/feature_controls/observability_security·ts - ObservabilityApp feature controls observability security feature controls observability cases all privileges "before all" hook for "shows observability/cases navlink" (elastic#227431)

## Summary

It resumes/ fixes elastic#155091
It resumes/ fixes elastic#155090 
These tests were flaky in 2023, but now, according to FTR, they are
working as expected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment