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

[jaeger-v2] Add Elasticsearch storage backend integration test #5287

Conversation

akagami-harsh
Copy link
Member

@akagami-harsh akagami-harsh commented Mar 21, 2024

Which problem is this PR solving?

Description of the changes

  • added integration test for elasticsearch
  • not modified the workflow's yet. once tests works i'll modify them too.

How was this change tested?

  • run integration test script
  • export STORAGE=elasticsearch
  • ./scripts/es-integration-test.sh elasticsearch 8.12.0 v2

Checklist

Signed-off-by: Harshvir Potpose <hpotpose62@gmail.com>
@akagami-harsh
Copy link
Member Author

not able to receive trace data

2024-03-21T18:22:44.686+0530    warn    localhostgate/featuregate.go:63 The default endpoints for all servers in components will change to use localhost instead of 0.0.0.0 in a future version. Use the feature gate to preview the new default.     {"feature gate ID": "component.UseLocalHostAsDefaultHost"}
2024/03/21 18:22:45 Starting load generator at 1024 items/sec.
2024/03/21 18:22:47  | Sent:        76 traces (38/sec) | Received:         0 items (0/sec)
2024/03/21 18:22:50  | Sent:       384 traces (77/sec) | Received:         0 items (0/sec)
2024/03/21 18:22:53  | Sent:       384 traces (48/sec) | Received:         0 items (0/sec)
2024/03/21 18:22:56  | Sent:       384 traces (35/sec) | Received:         0 items (0/sec)
2024/03/21 18:22:59  | Sent:       384 traces (27/sec) | Received:         0 items (0/sec)
2024/03/21 18:23:02  | Sent:       460 traces (27/sec) | Received:         0 items (0/sec)
2024/03/21 18:23:02 Stopped generator. Sent:       460 traces (26/sec)
2024/03/21 18:23:05  | Sent:       460 traces (23/sec) | Received:         0 items (0/sec)
2024/03/21 18:23:08  | Sent:       460 traces (20/sec) | Received:         0 items (0/sec)
2024/03/21 18:23:11  | Sent:       460 traces (17/sec) | Received:         0 items (0/sec)
2024/03/21 18:23:12 Time out waiting for all data items received
    test_case.go:289: Time out waiting for all data items received
2024/03/21 18:23:12 Stopping mock backend...
2024/03/21 18:23:12 Stopped backend. Received:         0 items (0/sec)
2024-03-21T18:23:12.953+0530    info    otelcol@v0.96.0/collector.go:280        Received shutdown request
2024-03-21T18:23:12.954+0530    info    service@v0.96.0/service.go:206  Starting shutdown...
2024-03-21T18:23:12.954+0530    info    extensions/extensions.go:59     Stopping extensions...
2024-03-21T18:23:12.954+0530    info    service@v0.96.0/service.go:220  Shutdown complete.
--- ❌ FAIL: TestElasticsearchStorage (28.35s)
=== RUN   TestGRPCStorage
    integration.go:61: Integration test against Jaeger-V2 grpc skipped; set STORAGE env var to grpc to run this
--- 🔕 SKIP: TestGRPCStorage (0.00s)
❌ FAIL
coverage: 3.1% of statements in ./...
❌ FAIL github.com/jaegertracing/jaeger/cmd/jaeger/internal/integration 28.526s
❌ FAIL
make: *** [Makefile:132: jaeger-storage-integration-test] Error 1
T18:23:13 teardown_storage 75189abe49c8dc3fe2be8f5fa21c25be0b0a0e2c8ef8d7858d60265ee88e34d1
T18:23:13 local cid=75189abe49c8dc3fe2be8f5fa21c25be0b0a0e2c8ef8d7858d60265ee88e34d1
T18:23:13 docker kill 75189abe49c8dc3fe2be8f5fa21c25be0b0a0e2c8ef8d7858d60265ee88e34d1
75189abe49c8dc3fe2be8f5fa21c25be0b0a0e2c8ef8d7858d60265ee88e34d1```

@akagami-harsh
Copy link
Member Author

not needed now as e2e tests covers this part #5322

@akagami-harsh akagami-harsh deleted the add-elasticsearch-ingration-tests branch April 4, 2024 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant