Skip to content

feat(otlp-exporter): add span factory base structure#504

Open
arjun-rajappa wants to merge 12 commits into
otlp-exporterfrom
span-convertor-base
Open

feat(otlp-exporter): add span factory base structure#504
arjun-rajappa wants to merge 12 commits into
otlp-exporterfrom
span-convertor-base

Conversation

@arjun-rajappa

Copy link
Copy Markdown
Contributor

No description provided.

@arjun-rajappa arjun-rajappa self-assigned this May 21, 2026
@arjun-rajappa arjun-rajappa requested a review from a team May 21, 2026 09:04

@CagriYonca CagriYonca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me

@pvital pvital left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks good to me, but why are the tests failing?

Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
@arjun-rajappa arjun-rajappa force-pushed the span-convertor-base branch from 2a6884c to 41b2efb Compare June 2, 2026 08:48
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
…truct

Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
@arjun-rajappa arjun-rajappa force-pushed the span-convertor-base branch from 72ed149 to 9ee586a Compare June 4, 2026 15:36
@arjun-rajappa arjun-rajappa requested a review from pvital June 11, 2026 06:06

@pvital pvital left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please fix the SonarQube-reported issues.

Comment thread lib/instana/exporter/otlp/base_converter.rb
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Comment thread lib/instana/exporter/otlp/resource.rb Outdated
Signed-off-by: Arjun Rajappa <arjun.rajappa@ibm.com>
@sonarqubecloud

Copy link
Copy Markdown

@pvital pvital left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks good to me.

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.

3 participants