Skip to content

Fix:Initialize the correct otel-grpc exporter #2666

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
Merged
merged 1 commit into from
Apr 23, 2024
Merged

Fix:Initialize the correct otel-grpc exporter #2666

merged 1 commit into from
Apr 23, 2024

Conversation

snowleopard331
Copy link
Contributor

@snowleopard331 snowleopard331 commented Apr 23, 2024 •
edited
Loading

Fix the problem that the following configurations do not take effect

otel:
trace:
enable: true
exporter: otlp-grpc

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

FoghostCn
FoghostCn approved these changes Apr 23, 2024
View reviewed changes
Copy link
Contributor

@FoghostCn FoghostCn left a comment

Choose a reason for hiding this comment

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

LGTM

@AlexStocks AlexStocks merged commit 1bd6abc into apache:main Apr 23, 2024
@FoghostCn
Copy link
Contributor

Plz backport this to release-3.1 branch @snowleopard331

FoghostCn pushed a commit to FoghostCn/dubbo-go that referenced this pull request May 18, 2024
@snowleopard331 @FoghostCn
Fix:Initialize the correct otel-grpc exporter (apache#2666)
333ac97 
Co-authored-by: yunfei <qiuyunfei@kiwiing.com>
(cherry picked from commit 1bd6abc)
@FoghostCn FoghostCn mentioned this pull request May 18, 2024
Merged
FoghostCn added a commit that referenced this pull request May 18, 2024
@FoghostCn @snowleopard331
Fix:Initialize the correct otel-grpc exporter (#2666) (#2679)
3439258 
Co-authored-by: yunfei <qiuyunfei@kiwiing.com>
(cherry picked from commit 1bd6abc)

Co-authored-by: Jovi <snowleopard331@163.com>
@AlexStocks AlexStocks mentioned this pull request Mar 22, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FoghostCn FoghostCn FoghostCn approved these changes

@AlexStocks AlexStocks AlexStocks approved these changes

Assignees
No one assigned
Labels
backport-needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@snowleopard331 @FoghostCn @AlexStocks