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.

Sign up for GitHub

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

Jump to bottom

[UA] Fix skipped functional tests #212026

Draft
wants to merge 9 commits into
base: 8.x
Choose a base branch
from
Draft

[UA] Fix skipped functional tests #212026

wants to merge 9 commits into from
+9 −12

Conversation

sabarasaba
Copy link
Member

@sabarasaba sabarasaba commented Feb 21, 2025 •
edited
Loading

Fixes: #167073
Fixes: #167081
Fixes: #167094

This PR fixes navigateToPage method from UA and unskips the tests.

@sabarasaba sabarasaba self-assigned this Feb 21, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 21, 2025 •
edited
Loading

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!

@sabarasaba sabarasaba changed the title [UA] Unskip tests [UA] Fix skipped functional tests Feb 21, 2025
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/test/functional/apps/upgrade_assistant/config.ts: 2/2 tests passed.

see run history

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[❌] x-pack/test/functional/config.upgrade_assistant.ts: 0/50 tests passed.

see run history

@sabarasaba sabarasaba added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more release_note:skip Skip the PR/issue when compiling release notes Feature:Upgrade Assistant backport:version Backport to applied version labels v8.18.0 v8.19.0 labels Mar 3, 2025
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/test/functional/apps/upgrade_assistant/config.ts: 2/2 tests passed.
[❌] x-pack/test/functional/config.upgrade_assistant.ts: 0/2 tests passed.

see run history

@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 3, 2025 •
edited
Loading

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #10 / alerting api integration security and spaces enabled - Group 2 Alerting and Actions Telemetry test telemetry should retrieve telemetry data in the expected format
  • [job] [logs] FTR Configs #1 / Upgrade Assistant feature controls security "before all" hook in "security"
  • [job] [logs] FTR Configs #1 / Upgrade Assistant feature controls security "before all" hook in "security"

Metrics [docs]

✅ unchanged

History

cc @sabarasaba

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kibanamachine kibanamachine Awaiting requested review from kibanamachine kibanamachine will be requested when the pull request is marked ready for review kibanamachine is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@sabarasaba sabarasaba

Labels
backport:version Backport to applied version labels Feature:Upgrade Assistant release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v8.18.0 v8.19.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sabarasaba @elasticmachine @kibanamachine