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

Failing test: Jest Integration Tests.x-pack/solutions/observability/packages/utils_server/es/storage/index_adapter/integration_tests - StorageIndexAdapter when indexing into a clean Elasticsearch instance after rolling over the index manually and indexing the same document puts the document in the new write index #206482

Closed
kibanamachine opened this issue Jan 13, 2025 · 13 comments · Fixed by #208150
Closed

Failing test: Jest Integration Tests.x-pack/solutions/observability/packages/utils_server/es/storage/index_adapter/integration_tests - StorageIndexAdapter when indexing into a clean Elasticsearch instance after rolling over the index manually and indexing the same document puts the document in the new write index #206482

kibanamachine opened this issue Jan 13, 2025 · 13 comments · Fixed by #208150
Assignees
@flash1293
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Team:Observability Team label for Observability Team (for things that are handled across all of observability)

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Jan 13, 2025 •
edited
Loading

A test failed on a tracked branch

Error: expect(received).toMatchObject(expected)

- Expected  -  1
+ Received  + 10

  Object {
    "hits": Object {
      "hits": Array [
        Object {
          "_id": "doc1",
+         "_index": "test_index-000001",
+         "_score": 1,
+         "_source": Object {
+           "foo": "bar",
+         },
+       },
+       Object {
+         "_id": "doc1",
          "_index": "test_index-000002",
+         "_score": 1,
          "_source": Object {
            "foo": "bar",
          },
        },
      ],
      "total": Object {
        "relation": "eq",
-       "value": 1,
+       "value": 2,
      },
    },
  }
    at toMatchObject (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1736787802971935315/elastic/kibana-on-merge/kibana/x-pack/solutions/observability/packages/utils_server/es/storage/index_adapter/integration_tests/index.test.ts:526:28)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.<anonymous> (/opt/buildkite-agent/builds/bk-agent-prod-gcp-1736787802971935315/elastic/kibana-on-merge/kibana/x-pack/solutions/observability/packages/utils_server/es/storage/index_adapter/integration_tests/index.test.ts:190:9)

First failure: kibana-on-merge - main
@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Jan 13, 2025
@botelastic botelastic bot added the needs-team Issues missing a team label label Jan 13, 2025
@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - 8.x

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

@kibanamachine
Copy link
Contributor Author

New failure: kibana-on-merge - main

mistic added a commit that referenced this issue Jan 14, 2025
@mistic
skip flaky suite (#206482)
f047cb3
mistic added a commit that referenced this issue Jan 14, 2025
@mistic
skip flaky suite (#206482)
1235171
@mistic
Copy link
Member

mistic commented Jan 14, 2025

Skipped.

main: f047cb3
8.x: 1235171

@mistic mistic added the Team:Observability Team label for Observability Team (for things that are handled across all of observability) label Jan 14, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/unified-observability (Team:Observability)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 14, 2025
@flash1293 flash1293 self-assigned this Jan 21, 2025
viduni94 pushed a commit to viduni94/kibana that referenced this issue Jan 23, 2025
@mistic @viduni94
skip flaky suite (elastic#206482)
a72af53
@flash1293 flash1293 mentioned this issue Jan 24, 2025
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jan 28, 2025
@flash1293
🌊 Streams - unskip test debug (elastic#208150)
beebbb3 
Fixes elastic#206482
Fixes elastic#206483

I did some tests running the flaky part in a loop and couldn't get it to
fail once both locally and on CI.
I also reviewed the code and I can't tell how this could be flaky. My
guess is that something changed since we skipped this test.

Before spending more time on it, let's try to unskip - if it still
causes problems we can quickly skip again.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 8bb48a4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@flash1293 flash1293

Labels
failed-test A test failure on a tracked branch, potentially flaky-test Team:Observability Team label for Observability Team (for things that are handled across all of observability)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

🌊 Streams - unskip test debug flash1293/kibana
4 participants
@flash1293 @mistic @elasticmachine @kibanamachine