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

[processor/servicegraph] consume traces even metric count is equal to 0 #23029

Merged
merged 3 commits into from
Jun 2, 2023

Conversation

tuhao1020
Copy link

Description:
consume traces even metric count is equal to 0

Link to tracking Issue:
#23028

@tuhao1020 tuhao1020 requested a review from a team June 2, 2023 07:25
@tuhao1020 tuhao1020 requested a review from jpkrohling as a code owner June 2, 2023 07:25
@github-actions github-actions bot added the processor/servicegraph Service graph processor label Jun 2, 2023
@github-actions github-actions bot requested a review from mapno June 2, 2023 07:25
Copy link
Contributor

@mapno mapno left a comment

Choose a reason for hiding this comment

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

Good catch

Signed-off-by: 涂浩 <haotu@alauda.io>
@tuhao1020 tuhao1020 requested a review from mapno June 2, 2023 10:53
@songy23 songy23 changed the title consume traces even metric count is equal to 0 [processor/servicegraph] consume traces even metric count is equal to 0 Jun 2, 2023
@songy23 songy23 added the ready to merge Code review completed; ready to merge by maintainers label Jun 2, 2023
@dmitryax dmitryax merged commit cd9c1f0 into open-telemetry:main Jun 2, 2023
@github-actions github-actions bot added this to the next release milestone Jun 2, 2023
Caleb-Hurshman pushed a commit to observIQ/opentelemetry-collector-contrib that referenced this pull request Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
processor/servicegraph Service graph processor ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants