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

[Onboarding] Fix failed Firehose tests #193733

Merged

Conversation

mykolaharmash
Copy link
Contributor

@mykolaharmash mykolaharmash commented Sep 23, 2024

Closes 193294

When the Firehose flow initialized it installs firehose integration and assets for all supported AWS services in the background that might take a while. This change increases the wait time until the loader in the UI is hidden and the test can proceed.

Successful flaky test runner job

@obltmachine
Copy link

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

@mykolaharmash mykolaharmash changed the title 193294 failed firehose tests [Onboarding] Fix failed Firehose tests Sep 23, 2024
@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

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

[✅] x-pack/test_serverless/functional/test_suites/observability/config.ts: 50/50 tests passed.

see run history

@mykolaharmash mykolaharmash marked this pull request as ready for review September 24, 2024 07:02
@mykolaharmash mykolaharmash requested a review from a team as a code owner September 24, 2024 07:02
@mykolaharmash mykolaharmash requested a review from a team September 24, 2024 07:02
@mykolaharmash mykolaharmash added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:obs-ux-logs Observability Logs User Experience Team v8.16.0 labels Sep 24, 2024
@elasticmachine
Copy link
Contributor

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

@mykolaharmash mykolaharmash added backport:skip This commit does not require backporting backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development and removed backport:skip This commit does not require backporting labels Sep 24, 2024
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Sep 24, 2024
@mykolaharmash mykolaharmash force-pushed the 193294-failed-firehose-tests branch from 9871f04 to f0b88d5 Compare September 24, 2024 07:05
@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 24, 2024

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: f0b88d5
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-193733-f0b88d5258a4

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #86 / discover/esql discover esql view ES|QL in Discover should render esql view correctly

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@Kerry350 Kerry350 self-requested a review September 24, 2024 10:35
Copy link
Contributor

@Kerry350 Kerry350 left a comment

Choose a reason for hiding this comment

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

Approving so we can unblock the tests, thanks for looking into this one 👍

Do you think in the future the onboarding tests that rely on these installations should use this approach: #192064? Might be worth a followup ticket.

@mykolaharmash
Copy link
Contributor Author

Approving so we can unblock the tests, thanks for looking into this one 👍

Do you think in the future the onboarding tests that rely on these installations should use this approach: #192064? Might be worth a followup ticket.

Nice, thank you for pointing that out! I've created a follow up maintenance issue.

@mykolaharmash mykolaharmash merged commit ca61209 into elastic:main Sep 24, 2024
23 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 24, 2024
Closes [193294](elastic#193294)

When the Firehose flow initialized it installs firehose integration and
assets for all supported AWS services in the background that might take
a while. This change increases the wait time until the loader in the UI
is hidden and the test can proceed.

[Successful flaky test runner
job](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6998#_)

(cherry picked from commit ca61209)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
8.15 Backport failed because of merge conflicts
8.x

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 193733

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Sep 24, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Onboarding] Fix failed Firehose tests
(#193733)](#193733)

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

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

<!--BACKPORT [{"author":{"name":"Mykola
Harmash","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-24T12:22:59Z","message":"[Onboarding]
Fix failed Firehose tests (#193733)\n\nCloses
[193294](https://github.com/elastic/kibana/issues/193294)\r\n\r\nWhen
the Firehose flow initialized it installs firehose integration
and\r\nassets for all supported AWS services in the background that
might take\r\na while. This change increases the wait time until the
loader in the UI\r\nis hidden and the test can
proceed.\r\n\r\n[Successful flaky test
runner\r\njob](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6998#_)","sha":"ca61209d13fa19d6119521a7276bd0c210bc926c","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","Team:obs-ux-logs","v8.16.0"],"title":"[Onboarding]
Fix failed Firehose
tests","number":193733,"url":"https://github.com/elastic/kibana/pull/193733","mergeCommit":{"message":"[Onboarding]
Fix failed Firehose tests (#193733)\n\nCloses
[193294](https://github.com/elastic/kibana/issues/193294)\r\n\r\nWhen
the Firehose flow initialized it installs firehose integration
and\r\nassets for all supported AWS services in the background that
might take\r\na while. This change increases the wait time until the
loader in the UI\r\nis hidden and the test can
proceed.\r\n\r\n[Successful flaky test
runner\r\njob](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6998#_)","sha":"ca61209d13fa19d6119521a7276bd0c210bc926c"}},"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/193733","number":193733,"mergeCommit":{"message":"[Onboarding]
Fix failed Firehose tests (#193733)\n\nCloses
[193294](https://github.com/elastic/kibana/issues/193294)\r\n\r\nWhen
the Firehose flow initialized it installs firehose integration
and\r\nassets for all supported AWS services in the background that
might take\r\na while. This change increases the wait time until the
loader in the UI\r\nis hidden and the test can
proceed.\r\n\r\n[Successful flaky test
runner\r\njob](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6998#_)","sha":"ca61209d13fa19d6119521a7276bd0c210bc926c"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Mykola Harmash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-logs Observability Logs User Experience Team v8.16.0 v9.0.0
Projects
None yet
6 participants