Skip to content

Conversation

@SomilJain0112
Copy link
Contributor

@SomilJain0112 SomilJain0112 commented Oct 10, 2025

Which problem is this PR solving?

Resolves #7223

Description of the changes

a. Add IndexSpanAlias and IndexServiceAlias config fields
b. Use explicit aliases when provided instead of prefix pattern

How was this change tested?

Through local testing

Checklist

@codecov
Copy link

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.58%. Comparing base (697356f) to head (f7b4f66).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7550      +/-   ##
==========================================
- Coverage   95.58%   95.58%   -0.01%     
==========================================
  Files         311      311              
  Lines       15519    15545      +26     
==========================================
+ Hits        14834    14858      +24     
- Misses        537      539       +2     
  Partials      148      148              
Flag Coverage Δ
badger_v1 9.86% <0.00%> (-0.04%) ⬇️
badger_v2 2.06% <0.00%> (-0.01%) ⬇️
cassandra-4.x-v1-manual 13.99% <0.00%> (-0.05%) ⬇️
cassandra-4.x-v2-auto 2.05% <0.00%> (-0.01%) ⬇️
cassandra-4.x-v2-manual 2.05% <0.00%> (-0.01%) ⬇️
cassandra-5.x-v1-manual 13.99% <0.00%> (-0.05%) ⬇️
cassandra-5.x-v2-auto 2.05% <0.00%> (-0.01%) ⬇️
cassandra-5.x-v2-manual 2.05% <0.00%> (-0.01%) ⬇️
clickhouse 1.97% <0.00%> (-0.01%) ⬇️
elasticsearch-6.x-v1 18.84% <57.89%> (+0.05%) ⬆️
elasticsearch-7.x-v1 18.87% <57.89%> (+0.05%) ⬆️
elasticsearch-8.x-v1 19.04% <57.89%> (+0.05%) ⬆️
elasticsearch-8.x-v2 2.06% <0.00%> (-0.01%) ⬇️
elasticsearch-9.x-v2 2.06% <0.00%> (-0.01%) ⬇️
grpc_v1 9.69% <0.00%> (-0.03%) ⬇️
grpc_v2 2.06% <0.00%> (-0.01%) ⬇️
memory_v2 2.06% <0.00%> (-0.01%) ⬇️
opensearch-1.x-v1 18.92% <57.89%> (+0.05%) ⬆️
opensearch-2.x-v1 18.92% <57.89%> (+0.05%) ⬆️
opensearch-2.x-v2 2.06% <0.00%> (-0.01%) ⬇️
opensearch-3.x-v2 2.06% <0.00%> (-0.10%) ⬇️
query 2.06% <0.00%> (-0.01%) ⬇️
tailsampling-processor 0.59% <0.00%> (-0.01%) ⬇️
unittests 94.15% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

github-actions bot commented Oct 10, 2025

Metrics Comparison Summary

Total changes across all snapshots: 73

Detailed changes per snapshot

summary_metrics_snapshot_opensearch

📊 Metrics Diff Summary

Total Changes: 73

  • 🆕 Added: 73 metrics
  • ❌ Removed: 0 metrics
  • 🔄 Modified: 0 metrics

🆕 Added Metrics

  • jaeger_storage_latency_seconds (18 variants)
View diff sample
+jaeger_storage_latency_seconds{le="+Inf",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
+jaeger_storage_latency_seconds{le="0",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
+jaeger_storage_latency_seconds{le="10",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
+jaeger_storage_latency_seconds{le="100",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
+jaeger_storage_latency_seconds{le="1000",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
+jaeger_storage_latency_seconds{le="10000",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
+jaeger_storage_latency_seconds{le="25",name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
...
- `jaeger_storage_requests` (1 variants)
View diff sample
+jaeger_storage_requests{name="some_storage",operation="find_traces",otel_scope_name="jaeger-v2",otel_scope_schema_url="",otel_scope_version="",result="err",role="tracestore"}
- `rpc_server_duration_milliseconds` (18 variants)
View diff sample
+rpc_server_duration_milliseconds{le="+Inf",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_duration_milliseconds{le="0",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_duration_milliseconds{le="10",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_duration_milliseconds{le="100",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_duration_milliseconds{le="1000",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_duration_milliseconds{le="10000",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_duration_milliseconds{le="25",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
...
- `rpc_server_requests_per_rpc` (18 variants)
View diff sample
+rpc_server_requests_per_rpc{le="+Inf",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_requests_per_rpc{le="0",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_requests_per_rpc{le="10",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_requests_per_rpc{le="100",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_requests_per_rpc{le="1000",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_requests_per_rpc{le="10000",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_requests_per_rpc{le="25",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
...
- `rpc_server_responses_per_rpc` (18 variants)
View diff sample
+rpc_server_responses_per_rpc{le="+Inf",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_responses_per_rpc{le="0",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_responses_per_rpc{le="10",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_responses_per_rpc{le="100",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_responses_per_rpc{le="1000",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_responses_per_rpc{le="10000",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
+rpc_server_responses_per_rpc{le="25",otel_scope_name="go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc",otel_scope_schema_url="https://opentelemetry.io/schemas/1.37.0",otel_scope_version="0.64.0",rpc_grpc_status_code="2",rpc_method="FindTraces",rpc_service="jaeger.api_v3.QueryService",rpc_system="grpc"}
...

➡️ View full metrics file

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro,
Please review the PR. Please let me know if any changes are required and I am going in the right direction for solving this issue or not.
Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro,
Please review this PR.
Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro ,
Please review this PR as well.
Thanks!

Signed-off-by: Somil Jain <somiljain896@gmail.com>
…ndices

Signed-off-by: Somil Jain <somiljain896@gmail.com>
Signed-off-by: Somil Jain <somiljain896@gmail.com>
Signed-off-by: Somil Jain <somiljain896@gmail.com>
@SomilJain0112 SomilJain0112 force-pushed the fix/Added-fields-for-explicit-alias branch from bcc7b86 to 83eb73c Compare October 18, 2025 05:47
@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro ,
I have resolved all the comments made by you, please review the changes.
Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro , I have resolved comments. Please review here!

Signed-off-by: Somil Jain <somiljain896@gmail.com>
@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro ,
Addressed your above comments and made changes accordingly in the code. Kindly review it.
Thanks!

Signed-off-by: Somil Jain <somiljain896@gmail.com>
@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro
Now we will have complete control over naming that's what we were trying to achieve from this issue.
Please review, Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro
Please review this PR, Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro @mahadzaryab1
Please review this PR.
Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro Are we going in right direction to solve this issue?

Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

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

sorry for delay. Looks good, with a couple of minor comments

@SomilJain0112 SomilJain0112 force-pushed the fix/Added-fields-for-explicit-alias branch from 0d6d3bc to ef82f79 Compare December 9, 2025 18:13
@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro
Removed redundant checks as it was not needed, removed irrelevant comment as well and removed an unnecessary testcase as well. Resolved all of your latest comments. Please have a look again.
Thanks!

@SomilJain0112
Copy link
Contributor Author

Hii @yurishkuro ,Is there any reason why this PR is not merged into main as I can see you have already approved this?

@yurishkuro
Copy link
Member

@SomilJain0112 we only merge PRs that pass CI. This one is failing all ES e2e tests.

@SomilJain0112
Copy link
Contributor Author

I have checked those E2E tests failing are not related to my changes. Correct me please if I am wrong @yurishkuro

@yurishkuro
Copy link
Member

let's rerun then. The se tests are passing regularly on other PRs.

@SomilJain0112
Copy link
Contributor Author

I just went through failed E2E test and this time also they are not failing due to my changes, I verified through logs. @yurishkuro Is there any better way to verify this?
Can we run E2E tests on main branch that can give us more clarity right?

@yurishkuro
Copy link
Member

Kafka failure should be unrelated, I will rerun

@yurishkuro yurishkuro merged commit 5e83f5a into jaegertracing:main Dec 15, 2025
82 of 84 checks passed
SoumyaRaikwar pushed a commit to SoumyaRaikwar/jaeger that referenced this pull request Dec 18, 2025
…ertracing#7550)

## Which problem is this PR solving?
Resolves jaegertracing#7223

## Description of the changes
 a. Add IndexSpanAlias and IndexServiceAlias config fields
 b. Use explicit aliases when provided instead of prefix pattern

## How was this change tested?
Through local testing

## Checklist
- [✅ ] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [ ✅] I have signed all commits
- [ ✅] I have added unit tests for the new functionality
- [✅ ] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Somil Jain <somiljain896@gmail.com>
Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
Signed-off-by: SoumyaRaikwar <somuraik@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Query use index alias for Opensearch/Elasticsearch

2 participants