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

[8.14] [Entity Analytics] Flaky Cypress test fixes (#186081) #187472

Merged
merged 3 commits into from
Jul 4, 2024
Merged

[8.14] [Entity Analytics] Flaky Cypress test fixes (#186081) #187472

merged 3 commits into from
Jul 4, 2024

Conversation

hop-dev
Copy link
Contributor

@hop-dev hop-dev commented Jul 3, 2024 •
edited
Loading

Backport

closes #187472

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

@hop-dev
[Entity Analytics] Flaky Cypress test fixes (elastic#186081)
aae98fb 
### Fix 1: Press refresh button after using date picker on dashboard

Closes elastic#183662
Closes elastic#183635
Closes elastic#183982

We shouldnt have to do this, but there is some condition where clicking
apply gets the UI in a stuck state and I cant figure it out.

### Fix 2: Use more specific selector for search bar on risk score
preview page

Closes elastic#184133

We were using a generic selector `[data-test-subj="unifiedQueryInput"]
textarea` which was bringing back two elements sometimes (no idea why),
I have given the box its own dedicated selector to hopefully prevent
this from happening

---------

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit c34bb46)
@hop-dev hop-dev enabled auto-merge (squash) July 3, 2024 12:43
@hop-dev hop-dev mentioned this pull request Jul 3, 2024
Merged
@hop-dev hop-dev self-assigned this Jul 3, 2024
@hop-dev hop-dev requested a review from machadoum July 3, 2024 12:46
@hop-dev hop-dev mentioned this pull request Jul 3, 2024
Closed
@hop-dev
Copy link
Contributor Author

hop-dev commented Jul 3, 2024

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment initAction includes the _meta data of the indices in the response
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment initAction resolves left when cluster.routing.allocation.enabled is incompatible
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment initAction resolves right empty record if no indices were found
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment initAction resolves right record with found indices
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment initAction resolves right when cluster.routing.allocation.enabled=all
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment reindex & waitForReindexTask resolves left incompatible_mapping_exception if all reindex failures are due to a strict_dynamic_mapping_exception
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment reindex & waitForReindexTask resolves right and excludes all documents not matching the excludeOnUpgradeQuery
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment reindex & waitForReindexTask resolves right and proceeds to add missing documents if there are some existing docs conflicts
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment reindex & waitForReindexTask resolves right when reindex succeeds with reindex script
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment reindex & waitForReindexTask resolves right when reindex succeeds without reindex script
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment reindex & waitForReindexTask resolves right, ignores version conflicts and does not update existing docs when reindex multiple times
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment removeWriteBlock rejects if there is a non-retryable error
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment removeWriteBlock resolves right if successful when an index already has a write block
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment removeWriteBlock resolves right if successful when an index does not have a write block
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment setWriteBlock once resolved, prevents further writes to the index
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment setWriteBlock resolves left index_not_found_exception when the index does not exist
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment setWriteBlock resolves right when setting a write block on an index that already has one
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment setWriteBlock resolves right when setting the write block succeeds
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment waitForIndexStatus resolves left with "index_not_green_timeout" after waiting for an index status to be green timeout
  • [job] [logs] Jest Integration Tests #3 / Migration actions - serverless environment waitForIndexStatus resolves left with "index_not_yellow_timeout" after waiting for an index status to be yellow timeout

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 15.2MB 15.2MB +51.0B

History

cc @hop-dev

@hop-dev
Copy link
Contributor Author

hop-dev commented Jul 4, 2024

@elasticmachine merge upstream

@hop-dev hop-dev requested review from machadoum and removed request for machadoum July 4, 2024 07:53
@hop-dev hop-dev merged commit cfd36e1 into elastic:8.14 Jul 4, 2024
6 checks passed
@hop-dev hop-dev deleted the backport/8.14/pr-186081 branch July 4, 2024 09:10
@szaffarano szaffarano mentioned this pull request Jul 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@machadoum machadoum machadoum approved these changes

Assignees

@hop-dev hop-dev

Labels
backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hop-dev @elasticmachine @machadoum